Closed
Bug 425338
Opened 17 years ago
Closed 17 years ago
hovering over text on wikipedia causes jiggling text
Categories
(Core :: Layout: Text and Fonts, defect, P1)
Core
Layout: Text and Fonts
Tracking
()
VERIFIED
FIXED
mozilla1.9
People
(Reporter: jtd, Unassigned)
References
()
Details
(4 keywords)
Attachments
(3 files, 1 obsolete file)
993 bytes,
text/html
|
Details | |
275 bytes,
text/html
|
Details | |
2.37 KB,
patch
|
Details | Diff | Splinter Review |
Steps: 1. Go to wikipedia 2. Move the mouse over the list of languages listed under 10,000+ Result: the layout of the line suddenly switches back and forth between two states. Occurs over multiple lines, starting with the language after Azərbaycan in the list.
Flags: blocking1.9?
Reporter | ||
Comment 1•17 years ago
|
||
![]() |
||
Comment 2•17 years ago
|
||
Here's a simplified testcase.
Attachment #311973 -
Attachment is obsolete: true
![]() |
||
Comment 3•17 years ago
|
||
some things of note, re: my testcase - remove the line "a { text-decoration: none; }" and the issue doesn't occur. - remove the line "a:hover { text-decoration: underline; }" and the issue doesn't occur. - remove the tag "<span>" a line under the body tag (and the corresponding </span>) and the issue doesn't occur.
Updated•17 years ago
|
Severity: critical → major
Keywords: regression,
testcase
Updated•17 years ago
|
Hardware: PC → All
Comment 5•17 years ago
|
||
(In reply to comment #4) > The patch for bug 424631 fixes this for me. > confirmed, the patch also fixes this problem for me on mac with the testcase and also the real wiki site
Updated•17 years ago
|
Flags: blocking1.9? → blocking1.9+
Priority: -- → P1
Updated•17 years ago
|
Component: Layout → Layout: BiDi Hebrew & Arabic
QA Contact: layout → layout.bidi
Target Milestone: --- → mozilla1.9beta5
Comment 6•17 years ago
|
||
Do we know how common this pattern (see comment #3) is on the web? I've been browsing some bidi websites that aren't Wikipedia, and things seem to work fine. Some examples: http://mozilla.org.il/evangel.shtml http://www.ynet.co.il/home/0,7340,L-8,00.html We've run across limited examples where it doesn't work: http://he.wikipedia.org/wiki/%D7%A2%D7%9E%D7%95%D7%93_%D7%A8%D7%90%D7%A9%D7%99 but so far they're all on Wikipedia. Anyone in the bidi community able to comment? I would imagine that Mano would have screamed if this was more common ;)
Comment 7•17 years ago
|
||
I don't use the Internet as much as you think I do :p
Comment 8•17 years ago
|
||
Was this fixed by the checkin for bug 424631 @ noonish today?
![]() |
||
Comment 9•17 years ago
|
||
Consolidating IRC discussion / discoveries for possible related jiggles: http://www.google.ae/ - near the middle, click and hold on "English". http://ma7room.com/ - highlight email address near top, webmaster @ ... http://study4uae.com/vb/index.php - near the bottom, hover. All of the above are Alexa Top100 sites, some Arabic etc.
Comment 10•17 years ago
|
||
This has only two levels of nested inlines. Hover to see the effect.
Comment 11•17 years ago
|
||
(In reply to comment #8) > Was this fixed by the checkin for bug 424631 @ noonish today? > Yes, this is fixed by that checkin (tested locally. I'll try to also pull an "hourly" build and verify).
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Comment 12•17 years ago
|
||
Verified fixed with Mozilla/5.0 (Macintosh; U; PPC Mac OS X 10.4; en-US; rv:1.9pre) Gecko/2008032714 Minefield/3.0pre. I'll leave it for someone else to double-check and mark this VERIFIED. Not sure whether there is a decision on respinning b5 for this. I think it makes sense to do so.
Comment 13•17 years ago
|
||
(In reply to comment #12) > I'll leave it for someone else to double-check and mark this VERIFIED. Will do that when the build is ready. > Not sure whether there is a decision on respinning b5 for this. I think it > makes sense to do so. Otherwise adding a relnote?
Comment 14•17 years ago
|
||
These reftest use height instead of underline, since height can be changed without affecting the rendering when everything is working correctly. Only 'b' fails in a build with the bug. 'a' is included for sanity. Both pass in a build with the patch.
Comment 15•17 years ago
|
||
Verified with Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9pre) Gecko Minefield/3.0pre ID:2008032723 Until this isn't definitely fixed for beta5 lets set the TM to mozilla1.9.
Status: RESOLVED → VERIFIED
Target Milestone: mozilla1.9beta5 → mozilla1.9
Updated•17 years ago
|
Flags: in-testsuite+
Updated•17 years ago
|
Attachment #312141 -
Attachment description: possible reftest → possible reftest [checked in]
Attachment #312141 -
Attachment is patch: true
Component: Layout: BiDi Hebrew & Arabic → Layout: Text
QA Contact: layout.bidi → layout.fonts-and-text
You need to log in
before you can comment on or make changes to this bug.
Description
•