Closed Bug 1708481 Opened 5 months ago Closed 5 months ago

Header separator from Proton spec missing in the Bookmarks Panel

Categories

(Firefox :: Bookmarks & History, task, P2)

task

Tracking

()

RESOLVED FIXED
90 Branch
Tracking Status
firefox89 --- fixed
firefox90 --- fixed

People

(Reporter: emalysz, Assigned: emalysz)

References

(Blocks 1 open bug)

Details

(Keywords: helpwanted, Whiteboard: [proton-door-hangers][priority:2b] [proton-uplift])

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1702990 +++

I had missed that we also wanted to add the separator directly below the edit bookmark heading as well. I'm copying over the priority level from the previous bug

Pushed by emalysz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0bb236e351f1
add separator beneath bookmarks panel header r=mconley
Status: ASSIGNED → RESOLVED
Closed: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch

Comment on attachment 9219256 [details]
Bug 1708481, add separator beneath bookmarks panel header

Beta/Release Uplift Approval Request

  • User impact if declined: Required for MR1 / Proton
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): A pretty small, CSS-only change that only impacts the bookmarks panel.
  • String changes made/needed: None.
Attachment #9219256 - Flags: approval-mozilla-beta?
Whiteboard: [proton-door-hangers][priority:2b] → [proton-door-hangers][priority:2b] [proton-uplift]

Comment on attachment 9219256 [details]
Bug 1708481, add separator beneath bookmarks panel header

Approved for 89 beta 7, thanks.

Attachment #9219256 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.