Closed
Bug 1342029
Opened 7 years ago
Closed 7 years ago
remove button in about:home appears over text
Categories
(Snippets :: Campaign, task)
Snippets
Campaign
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: maruf.rahman.95, Assigned: jcollings)
Details
Attachments
(3 files)
Build ID 20170221004019 UserAgent Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:53.0) Gecko/20100101 Firefox/53.0 Steps to reproduce : 1. open Firefox developer edition (aurora) 2. go to about:home 3. under the search bar there is "Fun Fact" Actual Result : The remove icon in of the "Fun Fact" is appearing over the text Expected Result : The remove icon should appear in a vacant palce [bugday-20170222]
Updated•7 years ago
|
Component: General → Layout
Product: Firefox → Core
Comment 1•7 years ago
|
||
This isn't a Layout bug unless our CSS is somehow not working. But this really needs some way to reproduce with more details. Cosmin, were you able to reproduce?
Component: Layout → Untriaged
Flags: needinfo?(cosmin.muntean)
Product: Core → Firefox
Comment 2•7 years ago
|
||
I was able to reproduce the issue, but now the RDM message is no longer displayed on "about:home" page. The issue was only reproducible for the RDM message from "about:home" page (the massage from the provided screenshot). When the text was hovered the "[x]" button was displayed over the text. I am not sure if the RDM message will be displayed in the future, if not we can close this issue?
Flags: needinfo?(cosmin.muntean)
Comment 3•7 years ago
|
||
(In reply to Cosmin Muntean [:CosminMCG] from comment #2) > I was able to reproduce the issue, but now the RDM message is no longer > displayed on "about:home" page. > > The issue was only reproducible for the RDM message from "about:home" page > (the massage from the provided screenshot). When the text was hovered the > "[x]" button was displayed over the text. > > I am not sure if the RDM message will be displayed in the future, if not we > can close this issue? I don't know. Arcadio, I'm told you might know more about this snippet and who made it?
Assignee: nobody → jcollings
Component: Untriaged → Campaign
Flags: needinfo?(alainez)
Product: Firefox → Snippets
Hi...yes. The Life Cycle Marketing team. Adding them here.
Flags: needinfo?(wwinton)
Flags: needinfo?(dvincent)
Flags: needinfo?(alainez)
Comment 5•7 years ago
|
||
Thanks, Arcadio. It looks like the overlap happens when there are more than two lines of text. Flagging Giorgos, our master snippet mechanic. Giorgos, is there any way we can get some extra spacing between the block button and the snippet text?
Flags: needinfo?(wwinton)
Flags: needinfo?(giorgos)
Flags: needinfo?(dvincent)
Comment 6•7 years ago
|
||
First of all something that's not visible in the screenshot: The remove button appears only when the mouse is over the snippet. The whole UI and UX follows guidelines we received back in the day to match the way Tiles look and thus the button is same and it appears on hover above content. That said if you still want to change this we can, it's a snippet template change.
Flags: needinfo?(giorgos)
I personally believe we should not fix this. It's not happening with all the three-line snippets (see attachment).
Assignee | ||
Comment 9•7 years ago
|
||
Apologies for the delay. Giorgos, I just spoke with the Firefox Product and UX team and they would like the template fixed so the "X" doesn't hover over text. Do you have time to work on this soon and do you need any design assistance from UX? Please let me know, thanks!
Flags: needinfo?(giorgos)
Comment 10•7 years ago
|
||
I'll be in a workweek next week, so this will be worked from the week of 27th. It would be valuable to get UX pointers during next week, i.e. place the button over the text or on the right of the text?
Flags: needinfo?(giorgos)
Assignee | ||
Comment 11•7 years ago
|
||
(In reply to Giorgos Logiotatidis [:giorgos] from comment #10) > I'll be in a workweek next week, so this will be worked from the week of > 27th. It would be valuable to get UX pointers during next week, i.e. place > the button over the text or on the right of the text? Great thanks Giorgos! This is the feedback I got from Aaron Benson, "Ideally, both. The icon could be floated in the upper-right corner of that "box" where the snippet lives. Example: https://cl.ly/3X0w3e0A3I0P"
Flags: needinfo?(giorgos)
Comment 12•7 years ago
|
||
This is WIP. Here's the pull request https://github.com/mozmar/snippets/pull/106
Comment 13•7 years ago
|
||
This is now in production
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(giorgos)
Resolution: --- → FIXED
Assignee | ||
Comment 14•7 years ago
|
||
Giorgos, thank you for fixing this. Can you check snippets 6455 and 6453? I cannot tell in preview if the fix is working. I checked on about:home about 4 hours after you pushed to production and I'm still getting the "x" covering some text as seen in the attachment. Let me know, thanks!
Flags: needinfo?(giorgos)
Comment 15•7 years ago
|
||
jean I cannot reproduce the issue. I's probably a stable cache, now it seems to be working. Feel free to re-open the bug if it doesn't for you.
Flags: needinfo?(giorgos)
Assignee | ||
Comment 16•7 years ago
|
||
(In reply to Giorgos Logiotatidis [:giorgos] from comment #15) > jean I cannot reproduce the issue. I's probably a stable cache, now it seems > to be working. Feel free to re-open the bug if it doesn't for you. All good, the changes finally rolled out. Thank you again!
You need to log in
before you can comment on or make changes to this bug.
Description
•