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)

defect

Tracking

()

VERIFIED FIXED
mozilla1.9

People

(Reporter: jtd, Unassigned)

References

()

Details

(4 keywords)

Attachments

(3 files, 1 obsolete file)

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?
Attached file better testcase
Here's a simplified testcase.
Attachment #311973 - Attachment is obsolete: true
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.
Severity: critical → major
Keywords: regression, testcase
Keywords: top100
The patch for bug 424631 fixes this for me.
Depends on: 424631
Blocks: 423130
Hardware: PC → All
(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
Flags: blocking1.9? → blocking1.9+
Priority: -- → P1
Component: Layout → Layout: BiDi Hebrew & Arabic
QA Contact: layout → layout.bidi
Target Milestone: --- → mozilla1.9beta5
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 ;)
I don't use the Internet as much as you think I do :p
Was this fixed by the checkin for bug 424631 @ noonish today?
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.
Attached file smaller testcase
This has only two levels of nested inlines. Hover to see the effect.
(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
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.
(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?
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.
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
Flags: in-testsuite+
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.

Attachment

General

Created:
Updated:
Size: