Extra menu border is rendered to "Show your bookmarks" toolbar button menu




11 days ago
9 days ago


(Reporter: arai, Unassigned)


Firefox Tracking Flags

(Not tracked)



(2 attachments)



11 days ago
Created attachment 8950199 [details]
Extra border around the bookmark menu

Steps to reproduce:
  1. Run Nightly 60.0a1 (2018-02-12) (64-bit) with clean profile on macOS
  2. Start Customize and place "Show your bookmarks" button next to "View history, bookmarkds, and more" button.
  3. Open https://www.amazon.com/ in tab
  4. Drag and drop the tab into "Show your bookmarks" to add bookmark directly under "Bookmarks Menu"
  5. Repeat step 4 about 50 times, in order to make the bookmarks menu overflows vertically from the screen
  6. Shutdown Nightly
  7. Run Nightly again
  8. Click "Show your bookmarks" to open bookmarks menu

Actual result:
  Extra border is rendered around the menu
  See attachment

  For me, this issue happens only once after startup

Expected result:
  No extra border is rendered
Created attachment 8950371 [details]

I get something a little different on Linux (Ubuntu). There is some artifact around/behind the arrow on the bookmarks menu. I'm not sure this is the same problem. This is a relatively minor issue but needs further investigation to figure out what's going on.
Mike, can you suggest who might be able to shed some light on what's going on here. :jaws was not able to reproduce this on windows, so it seems like this might exist somewhere between our front-end and the platform/graphics layer?
Flags: needinfo?(mconley)
Maybe Enn could shed some light?
Flags: needinfo?(mconley) → needinfo?(enndeakin)

Comment 4

9 days ago
I don't see this myself, but the bookmarks menu has its own implementation in browser/components/places/content/menu.xml 

It would be a good start to identify if an existing element is contributing to the extraneous appearance, or whether it is just a graphics artifact left behind, perhaps during the animation.
Flags: needinfo?(enndeakin)
I'll give this a priority to get it off the triage list. Comment #4 has good next steps.
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.