![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#30739 closed defect (bug) (fixed)
Twenty Fifteen: Menu icon is not properly centered
| Reported by: | tenpura | Owned by: | lancewillett |
|---|---|---|---|
| Priority: | low | Milestone: | 4.1 |
| Component: | Bundled Theme | Version: | 4.1 |
| Severity: | minor | Keywords: | commit fixed-major |
| Cc: | Focuses: | rtl |
Description
The menu icon on smaller screens does not seem to be properly centered.
Adding display: block; to .secondary-toggle:before fix this for me.
.secondary-toggle:before {
color: #333;
content: "\f419";
line-height: 42px;
position: relative;
top: -1px;
left: 0;
display: block;
}
Attachments (3)
Change History (20)
#2
@
12 years ago
- Resolution → worksforme
- Status new → closed
I'm sorry. The problem won't be reproduced with the clean install of latest trunk and branches/4.1. Something was wrong with my old test environment.
#3
@
12 years ago
- Resolution worksforme
- Status closed → reopened
Ok. I found out the way to reproduce the problem. This occurs only when twentyfifteen-ja.mo is loaded. Do you think of any section that can cause this in the language file?
#4
@
12 years ago
- Keywords reporter-feedback removed
#30743 also mentioned this problem when the locale is RTL
#5
@
12 years ago
- Milestone Awaiting Review
- Resolution → duplicate
- Status reopened → closed
Thanks for the report tenpura. Let's fix this as part of #30743.
This ticket was mentioned in Slack in #core by helen. View the logs.
12 years ago
#8
@
12 years ago
- Focuses rtl added
- Keywords needs-patch added
- Milestone 4.1 → Future Release
- Priority normal → low
- Severity normal → minor
This appears to only be reproducible if you zoom in and then zoom back out again.
Bumping for now, unless anyone comes up with a rock solid reason why this happens and an accompanying patch.
Only affects RTL.
#9
@
12 years ago
While I was looking at #30743, I've managed to reproduce the bug without zooming.
Specifying width for the pseudo element will fix the issue.
Patch incoming shortly.
Hi tenpura, thanks for this ticket. Can you please add a bit more information? Which small screens did you see this with? Can you add device and OS info so we can repeat it?
Is it WP trunk and latest version of Twenty Fifteen from trunk?
Can you repeat the bug on the Twenty Fifteen demo site: https://twentyfifteendemo.wordpress.com/ (screenshot: https://cloudup.com/cYVXHmE6iQm).