Opened 10 years ago
Closed 10 years ago
#30743 closed defect (bug) (fixed)
RTL styles bug with tabs on About page
Reported by: | alex-ye | Owned by: | nacin |
---|---|---|---|
Milestone: | 4.1 | Priority: | normal |
Severity: | minor | Version: | 3.8 |
Component: | Help/About | Keywords: | has-patch fixed-major commit |
Focuses: | rtl | Cc: |
Description
Some RTL styles bugs in 4.1 admin and Twenty Fifteen theme, Tested on WordPress 4.1 RC2 (Arabic Language), Google Chrome 39, Firefox 34.
Some of them are urgent.
See the attachments.
Attachments (12)
Change History (39)
#5
@
10 years ago
- Keywords needs-patch added
- Milestone changed from Awaiting Review to 4.1
- Severity changed from normal to minor
#6
@
10 years ago
The bug in 1.jpg happens on Google Chrome 39.0.2171.95 m (64-bit) and Firefox Developer Edition 36.0a2 (2014-12-15) with Microsoft Windows 8.1 64bit. The user avatar in a separated line. I couldn't determine what is really happen! .. I will tell you if I got any further information.
The bug in 3.jpg only happens on Firefox Developer Edition 36.0a2 (2014-12-15), maybe it's a bug in the browser!!
#7
follow-up:
↓ 11
@
10 years ago
The fix-4.patch needs someone to review it. The idea is to set a font-size, but I don't know if 32px is the proper value.
#8
follow-up:
↓ 10
@
10 years ago
fix-2.patch will fix the bug in 2.jpg, at least for Twenty Fifteen theme!. maybe we could do it in a better way?.
In addition, The font-size is not the same on the front-end and Visual Editor using a 1366x768 screen. I think the font-size should be 19px instead of 17px. who knows?!
#11
in reply to:
↑ 7
;
follow-up:
↓ 13
@
10 years ago
Replying to alex-ye:
The fix-4.patch needs someone to review it. The idea is to set a font-size, but I don't know if 32px is the proper value.
That's not correct. It should be 16px
for .secondary-toggle:before
, 32px
for .secondary-toggle.toggled-on:before
. They need to be different size. But I don't understand why we need to specify font size again for .secondary-toggle:before
when it's been set in L:188.
I can't reproduce the issue.
#12
in reply to:
↑ 10
@
10 years ago
Replying to iamtakashi:
Replying to alex-ye:
In addition, The font-size is not the same on the front-end and Visual Editor using a 1366x768 screen. I think the font-size should be 19px instead of 17px. who knows?!
17px is the right size for the editor style. See: #29986
Thanks, this explain why it's 17px :)
#13
in reply to:
↑ 11
@
10 years ago
Replying to iamtakashi:
Replying to alex-ye:
The fix-4.patch needs someone to review it. The idea is to set a font-size, but I don't know if 32px is the proper value.
That's not correct. It should be
16px
for.secondary-toggle:before
,32px
for.secondary-toggle.toggled-on:before
. They need to be different size. But I don't understand why we need to specify font size again for.secondary-toggle:before
when it's been set in L:188.
I can't reproduce the issue.
The bug occurs when I zoom in the page (from 150% to 170%), Thanks for pointing to L:188. if you think it's an edge case which will never happens, ignore it.
#14
@
10 years ago
Just to summarize..
1.jpg, still occurs with me on Google Chrome 39.0.2171.95 m (64-bit) and Firefox Developer Edition 36.0a2 (2014-12-15) with Microsoft Windows 8.1 64bit.
@yoavf can't reproduce it, so I will try to test it on other computers.
--
2.jpg, the fonts are still different (Tahoma on the Visual Editor and "Noto Serif" on the front-end). the fix-2.patch resolve it but it seems like a hack for me!
--
3.jpg, it looks like it's a bug in the browser, only happens on Firefox Developer Edition 36.0a2 (2014-12-15).
--
4.jpg, it only happens when I zoom in (from 150% to 170%). and no one can repoduce it!
--
5.jpg, @yoavf had upload the 30743.diff patch to reslove it.
--
6.jpg, needs tests and patch!
#15
follow-up:
↓ 17
@
10 years ago
The <code>
tags on the About page have a direction: ltr
rule applied in common.css. Surely an ltr marker isn't required for every piece of text which needs to remain ltr?
#16
@
10 years ago
I'm going to split this ticket up so we can better manage the separate issues. Stand by.
#17
in reply to:
↑ 15
@
10 years ago
Replying to johnbillion:
The
<code>
tags on the About page have adirection: ltr
rule applied in common.css. Surely an ltr marker isn't required for every piece of text which needs to remain ltr?
LTR marker is required for RTL sites.
#18
@
10 years ago
- Component changed from General to Help/About
- Focuses ui administration template removed
- Summary changed from RTL styles bugs in 4.1 admin and Twenty Fifteen to RTL styles bug with tabs on About page
#19
@
10 years ago
The About page tab text wrapping is only reproducible in Chrome on Windows. Chrome on OS X is fine, as is Firefox and IE9.
#22
follow-up:
↓ 26
@
10 years ago
@helen Thanks for the changeset, check 7.jpg and 8.jpg. The same proplem exist in some other texts!.. maybe it's related to #30745 ? this only happens on Google Chrome.
#23
@
10 years ago
@alex-ye, I've added a patch that intends to fix the icon issue in #30739. Can you test the 30739.diff, please?
This ticket was mentioned in Slack in #core-themes by iamtakashi. View the logs.
10 years ago
#25
@
10 years ago
- Version changed from trunk to 3.8
The wrapping text issue is reproducible back to at least 3.8. Not a regression in 4.1.
#26
in reply to:
↑ 22
@
10 years ago
Replying to alex-ye:
@helen Thanks for the changeset, check 7.jpg and 8.jpg. The same proplem exist in some other texts!.. maybe it's related to #30745 ? this only happens on Google Chrome.
I would suggest opening a new ticket for those, as this ticket is being used specifically for the about page tabs. Also, if you could, please see if you can find whether or not that exists in older versions or if it's new to 4.1.
takes care of the code example in the about page