#5286 closed defect (bug) (fixed)
Found Responsive issue on about page
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | Priority: | normal | |
| Component: | WordPress.org Site | Keywords: | needs-design has-patch needs-design-feedback 2nd-opinion |
| Cc: |
Description
I found the Horizontal Scroll Issue in below page.
Page URL :: https://wordpress.org/about/
In one of the class padding is writter twice. => https://prnt.sc/t51db0
Possible solution is created based on inspect element ::
.parallelogram {
color: #fff;
display: block;
margin: 0 auto;
opacity: .9;
padding: 75.2px 0;
padding: 4.7rem 0;
position: absolute;
text-align: center;
-webkit-transform: skew(0deg);
-ms-transform: skew(0deg);
transform: skew(0deg);
}
.parallelogram p {
-webkit-transform: skew(0deg);
-ms-transform: skew(0deg);
transform: skew(0deg);
}
.parallelogram>*{
-webkit-transform: skew(0deg);
-ms-transform: skew(0deg);
transform: skew(0deg);
}
body.page-about .shapes .parallelogram.technology{
-webkit-transform: skew(0deg);
-ms-transform: skew(0deg);
transform: skew(0deg);
}
.parallelogram:before{
-webkit-transform: skew(0deg);
-ms-transform: skew(0deg);
transform: skew(0deg);
width: 20px;
}
body.page-about .shapes .parallelogram.community{
margin-top:20px;
}
Attachments (1)
Change History (18)
#2
follow-up:
↓ 3
@
6 years ago
Hey @jatindevani! Thanks for filing this issue.
As @joyously mentions, the "double" padding is a fallback for Internet Explorer <= 8.
In terms of the scroll, we can probably fix this by setting an overflow-x: hidden on the parallelogram container for smaller viewports.
#3
in reply to:
↑ 2
;
follow-up:
↓ 4
@
6 years ago
@dufresnesteven
overflow-x: hidden
This is not a good solution for the design.
Please check Five for the Future page for a relevant design solution.
URL - https://wordpress.org/five-for-the-future/
Thanks.
#4
in reply to:
↑ 3
@
6 years ago
Sure. I'm not a big fan of the overlap on that page but I'm pretty indifferent overall.
Feel free to upload a patch!
Replying to rkradadiya:
@dufresnesteven
overflow-x: hiddenThis is not a good solution for the design.
Please check Five for the Future page for a relevant design solution.
URL - https://wordpress.org/five-for-the-future/
Thanks.
#5
follow-up:
↓ 6
@
6 years ago
I have Commits on Github
https://github.com/rkradadiya/wordpress.org/commit/f0b23ffc8f3f82fb072b7d03e79926ee4d8387f3
#6
in reply to:
↑ 5
@
6 years ago
Replying to rkradadiya:
I have Commits on Github
https://github.com/rkradadiya/wordpress.org/commit/f0b23ffc8f3f82fb072b7d03e79926ee4d8387f3
Thanks! Feel free to open a pull request against WordPress/wordpress.org, just mention https://meta.trac.wordpress.org/ticket/5286 in the PR description to link it to this ticket.
Just a note, your commits there are mixing spaces and tabs, we prefer tabs for indentation.
This ticket was mentioned in PR #13 on WordPress/wordpress.org by rkradadiya.
6 years ago
#7
- Keywords has-patch added; needs-patch removed
Responsive Media query CSS add to about page
#8
@
6 years ago
Responsive Media query CSS add to about page
Props: @jatindevani @dufresnesteven @rkradadiya
This ticket was mentioned in Slack in #design by rkradadiya. View the logs.
6 years ago
#15
@
5 years ago
- Owner set to ryelle
- Resolution set to fixed
- Status changed from assigned to closed
In 10441:
5 years ago
#16
This ticket was fixed with a different approach, but thanks for working on this patch.
Isn't that a fallback for browsers that don't support
remunits?