Closed
Bug 1249038
Opened 9 years ago
Closed 8 years ago
In Facebook Event "[Invite|...]" button, the "..." part is a bit too tall (unless we spoof a Chrome user agent string)
Categories
(Web Compatibility :: Site Reports, defect)
Web Compatibility
Site Reports
Tracking
(platform-rel +, firefox47 affected)
RESOLVED
WORKSFORME
People
(Reporter: dholbert, Unassigned)
Details
(Whiteboard: [platform-rel-Facebook])
Attachments
(2 files)
STR:
1. View a Facebook Event page (where you have privileges to invite people)
2. Look at the "Invite|..." button towards the upper-right.
ACTUAL RESULTS:
The "..." part is a bit taller than the "Invite" part. See attached screenshot.
EXPECTED RESULTS:
The two parts should be the same height.
Firefox Nightly (47) and release (44) give ACTUAL RESULTS. So does Firefox 31 (a random old build that I tried). So, not a Firefox regression, or not a recent one at least.
Chrome 50 dev gives EXPECTED RESULTS.
Reporter | ||
Comment 1•9 years ago
|
||
Reporter | ||
Comment 2•9 years ago
|
||
Actually, this seems to be a Facebook bug, in their Firefox-specific CSS.
In particular:
- If I save an affected Facebook page in Firefox & load it in Chrome, then I get ACTUAL RESULTS. (i.e. Chrome agrees with us on the brokenness)
- If I use Firefox Nightly with a spoofed Chrome user agent string, then I get EXPECTED RESULTS. (i.e. we render the buttons nicely, if Facebook sends us their Chrome-specific code instead of their Firefox-specific code)
Component: Layout → Desktop
Product: Core → Tech Evangelism
Reporter | ||
Updated•9 years ago
|
Summary: In Facebook Event "[Invite|...]" button, the "..." part is a bit too tall → In Facebook Event "[Invite|...]" button, the "..." part is a bit too tall (unless we spoof a Chrome user agent string)
Reporter | ||
Comment 3•9 years ago
|
||
(If it matters, my OS is Ubuntu 15.10, 64-bit.)
Comment 4•9 years ago
|
||
Markup structure in Firefox:
div
a - "invite"
div
a - "..."
Both As and DIVs have display:inline-block, the As also have a line-height: 26px applied. If I remove either the display:inline-block or the line-height from the A elements, the extra height (1px or so) goes away.
Updated•8 years ago
|
Whiteboard: [platform-rel-Facebook]
Updated•8 years ago
|
platform-rel: --- → ?
Updated•8 years ago
|
platform-rel: ? → +
Comment 5•8 years ago
|
||
Sorry for letting this slip through the cracks - setting to [contactready]. I don't think we need much more of an explanation -- serving the same CSS that Chrome receives would fix it. If they would prefer to fix the Firefox CSS, see Comment #4.
Whiteboard: [platform-rel-Facebook] → [platform-rel-Facebook][contactready]
Reporter | ||
Comment 6•8 years ago
|
||
Hmm, actually I can't reproduce this anymore! (using current Firefox Nightly 51 on Ubuntu 16.04, if it matters) I'm seeing results like attachment 8720351 [details] -- expected/nice results.
Perhaps Facebook fixed whatever quirk they had in their Firefox-specific CSS that was causing this? Since we don't know when/why/how this was fixed, I guess WORKSFORME is appropriate. (Feel free to reopen if anyone else is still hitting this.)
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → WORKSFORME
Whiteboard: [platform-rel-Facebook][contactready] → [platform-rel-Facebook]
Comment 7•8 years ago
|
||
Thanks for checking Daniel!
Assignee | ||
Updated•6 years ago
|
Product: Tech Evangelism → Web Compatibility
You need to log in
before you can comment on or make changes to this bug.
Description
•