The default bug view has changed. See this FAQ.

hovering over text on wikipedia causes jiggling text

VERIFIED FIXED in mozilla1.9

Status

()

Core
Layout: Text
P1
major
VERIFIED FIXED
9 years ago
9 years ago

People

(Reporter: jtd, Unassigned)

Tracking

(4 keywords)

Trunk
mozilla1.9
regression, relnote, testcase, top100
Points:
---
Dependency tree / graph
Bug Flags:
blocking1.9 +
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(3 attachments, 1 obsolete attachment)

(Reporter)

Description

9 years ago
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

9 years ago
Created attachment 311973 [details]
testcase, reduced with all styles included
Created attachment 311974 [details]
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.

Updated

9 years ago
Severity: critical → major
Keywords: regression, testcase

Updated

9 years ago
Keywords: top100
The patch for bug 424631 fixes this for me.
Depends on: 424631

Updated

9 years ago
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

Updated

9 years ago
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.
Created attachment 312130 [details]
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
Last Resolved: 9 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?

Comment 14

9 years ago
Created attachment 312141 [details] [diff] [review]
possible reftest [checked in]

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

Updated

9 years ago
Flags: in-testsuite+

Updated

9 years ago
Attachment #312141 - Attachment description: possible reftest → possible reftest [checked in]
Attachment #312141 - Attachment is patch: true
Keywords: relnote

Updated

9 years ago
Duplicate of this bug: 427327

Updated

9 years ago
Duplicate of this bug: 427326

Updated

9 years ago
Duplicate of this bug: 429825
Duplicate of this bug: 431896
Duplicate of this bug: 429265

Updated

9 years ago
Duplicate of this bug: 433623

Updated

9 years ago
Duplicate of this bug: 437498

Updated

9 years ago
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.