Closed Bug 1174521 Opened 9 years ago Closed 9 years ago

no click is working om autoscout24.de

Categories

(Core :: Layout, defect)

41 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
firefox40 --- unaffected
firefox41 + fixed

People

(Reporter: micha.postbox, Assigned: ehsan.akhgari)

References

Details

(Keywords: regression)

Attachments

(2 files)

User Agent: Mozilla/5.0 (Windows NT 6.3; rv:41.0) Gecko/20100101 Firefox/41.0 Build ID: 20150613030206 Steps to reproduce: open Link: http://fahrzeuge.autoscout24.de/?atype=C&make=9&mmvco=1&model=1624&mmvmk0=9&mmvmd0=1624&fuel=B&pricefrom=1000&priceto=100000&kmto=50000&fregfrom=2013&cy=D&doorfrom=4&doorto=5&ustate=N%2CU&sort=price&eq=39&dtr=s According Arai from IRC, mozregression tells it's a regression from bug 1143570. He confirmed that backing out https://hg.mozilla.org/mozilla-central/rev/f6a0792e6259 fixes this. Actual results: No link is clickable. Expected results: Links should work.
Blocks: 1143570
Component: Untriaged → Layout
Keywords: regression
Product: Firefox → Core
[Tracking Requested - why for this release]:
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: needinfo?(ehsan)
This seems to be a minimum set. So, clickable item inside floating box, another box with "position: absolute" style, followed by another (maybe inline) node. (maybe the bottom-most line of the last element matter?) If I hover mouse on the text "B", the dropdown menu become clickable. In the autoscout24.de page, most contents are inside floating boxes, and there are several "position: absolute" boxes, followed by an image element with width="2" and height="2", which is placed around top-right corner of the page.
I can confirm that on a daily compiled just now the following page http://fahrzeuge.autoscout24.de/?dtr=s does not work. The links (Fahrzeugsuche, Werkstattsuche, Verkaufen, etc. ) in the menu don't respond and all the other links on the page don't work either. Backing out https://hg.mozilla.org/mozilla-central/rev/f6a0792e6259 fixes this. Sorry, this was hard to believe, so I had to try it myself ;-)
roc, are you OK with a backout of https://hg.mozilla.org/mozilla-central/rev/f6a0792e6259 plus a patch that adds a test for it?
Assignee: nobody → ehsan
Flags: needinfo?(ehsan) → needinfo?(roc)
Yeah, sure
Flags: needinfo?(roc)
Flags: needinfo?(ehsan)
I fixed the test. The key was changing the 10px padding to 20px.
Flags: needinfo?(ehsan)
Adding a tracking flag for FF41. I tried to open the STR link on FF40.0a2 2015-06-03 and do not see the problem, therefore setting FF40 status as "unaffected".
https://hg.mozilla.org/mozilla-central/rev/587e2801a239 (sorry to interfere, why wasn't this marked fixed?)
Flags: needinfo?(cbook)
Confirmed, its fixed.
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(cbook)
Resolution: --- → FIXED
Comment on attachment 8623463 [details] [diff] [review] Backout the second part of bug 1143570 to fix the regression Approval Request Comment [Feature/regressing bug #]: Bug 1143570 (part 2 patch) [User impact if declined]: Comment 0, also see bug 1174834. [Describe test coverage new/current, TreeHerder]: Has an automated test [Risks and why]: Not risky, this is backing out an optimization patch landed in bug 1143570. [String/UUID change made/needed]: None.
Attachment #8623463 - Flags: approval-mozilla-aurora?
Sorry to interfere: Why does this have an Aurora (Mozilla40) request? The original patch was never landed on Mozilla40, so it would be hard to back it out ;-)
Flags: needinfo?(ehsan)
(In reply to Jorg K from comment #18) > Sorry to interfere: Why does this have an Aurora (Mozilla40) request? The > original patch was never landed on Mozilla40, so it would be hard to back it > out ;-) My mistake!
Flags: needinfo?(ehsan)
Attachment #8623463 - Flags: approval-mozilla-aurora?
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: