Opened 3 years ago
Closed 5 months ago
#6153 closed defect (bug) (fixed)
In accessibility page, Show Welcome Box issue.
Reported by: | smit08 | Owned by: | |
---|---|---|---|
Milestone: | Priority: | low | |
Component: | Make (Get Involved) / P2 | Keywords: | has-testing-info has-patch |
Cc: |
Description
Steps to reproduce:
- Visit the accessibility page (URL: https://make.wordpress.org/accessibility).
- Then click the Show Welcome Box. And view this in responsive device below 1000px.
- there is an issue of margin/padding between the <p> tag and Hide Welcome Box text.
Attachments (4)
Change History (13)
#2
@
3 years ago
Instead of decreasing the margin and padding of the button, It is preferable to add the margin-top to teh content on teh smaller devices from 1024px.
@
3 years ago
After applying this solution the spacing between the show welcome box and hide welcome box is looking same so might be that will be relevant for this issue.
#3
@
3 years ago
- Component changed from General to Make (Get Involved) / P2
- Priority changed from normal to low
#5
@
3 years ago
I have some quick & small points to mention for review on similar pages if they are valid.
3 columns on .make-welcome
has width:30%
for min-width:1024px
if that's not intentionally then can we make those columns wider and fix them to parent wrapper using
@media screen and (min-width: 1024px) { .make-welcome .entry-content .column { width: 33.3333333333%; padding: 1.5%; float: left; } }
.make-welcome .entry-content
has max-width:940px;
currently,
can't this section has max-width: 1024px;
same as other sections and look properly aligned?
New block header using 889px
for responsive while rest content after that using 876px
.
https://make.wordpress.org/docs/style-guide/
handbook breadcrumbs look too close to the handbook container name. Try checking 900px
top: 7rem;
working perfectly on all screens so far.
handbook breadcrumbs using left: 280px;
currently,
but not vertically aligned with entry content. we can try using left: 25%; margin-left: 4rem;
#9
@
5 months ago
- Resolution set to fixed
- Status changed from new to closed
Thanks for the report! This was fixed in [13125] (via a ticket that was a duplicate of this, but was seen first):
Breathe: Improve spacing between welcome box content and its toggle link.
Supplemental note: #6356 just had its resolution changed to be a duplicate of this, but had been considered the original report for this issue and thus had numerous other tickets closed as duplicates of it.
As you can clearly see the padding issue between the Hide Welcome Box text and <p> tag description.