#5286 closed defect (bug) (fixed)
Found Responsive issue on about page
Reported by: | jatindevani | Owned by: | ryelle |
---|---|---|---|
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
@
4 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
@
4 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
@
4 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
@
4 years ago
I have Commits on Github
https://github.com/rkradadiya/wordpress.org/commit/f0b23ffc8f3f82fb072b7d03e79926ee4d8387f3
#6
in reply to:
↑ 5
@
4 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.
4 years ago
#7
- Keywords has-patch added; needs-patch removed
Responsive Media query CSS add to about page
#8
@
4 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.
4 years ago
#15
@
4 years ago
- Owner set to ryelle
- Resolution set to fixed
- Status changed from assigned to closed
In 10441:
4 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
rem
units?