Closed Bug 1365663 Opened 6 years ago Closed 6 years ago

Update fixed items at top of synced tabs item in library panel to have icons, correct indenting


(Firefox :: Toolbars and Customization, defect, P1)

53 Branch



Firefox 56
56.1 - Jun 26
Tracking Status
firefox56 --- verified


(Reporter: Gijs, Assigned: Gijs)



(Whiteboard: [photon-structure])


(1 file)

The spec has an item for the synced tabs. It should do the same thing the current synced tabs button in the hamburger panel does.
Flags: qe-verify+
Whiteboard: [photon-structure]
Priority: -- → P2
QA Contact: gwimberly
Aaron, can you test the current version of the synced tabs item in the library panel and check if we still need to do anything here (besides the width of the library panel and this on current nightly - should be fixed by bug 1370580, you can test with the main hamburger panel's library entry for now) ?

Per comment #0 (and looking at the spec), I believe this should be done now, as we're just reusing the same thing we already had. The only thing I see is some remaining styling issues (mostly missing icons for the fixed items at the top, and the indenting of those items). If that's right, I can update the summary here to clarify.
Flags: needinfo?(abenson)
Yeah, this looks great! :)
Flags: needinfo?(abenson)
Summary: Add a synced tabs item to the library panel → Update fixed items at top of synced tabs item in library panel to have icons, correct indenting
Assignee: nobody → gijskruitbosch+bugs
Iteration: --- → 56.1 - Jun 26
Priority: P2 → P1
Comment on attachment 8878473 [details]
Bug 1365663 - update synced tabs panel view iconography and alignment,

LGTM! Thanks!

::: browser/themes/shared/customizableui/
(Diff revision 1)
>  photonpanelmultiview .subviewbutton:focus {
>    outline: 0;
>  }
> +photonpanelmultiview .subviewbutton:not(.subviewbutton-back) > .toolbarbutton-text {

Hmm, perhaps the back buttons' .toolbarbutton-text should just be display: none?
Feel free to include that in this patch, if it makes sense.
Attachment #8878473 - Flags: review?(mdeboer) → review+
Pushed by
update synced tabs panel view iconography and alignment, r=mikedeboer
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Depends on: 1374650
I can see "a synced tabs item is add in the library panel and have icon," in latest Nightly 56.0a1 on ubuntu 16.04 , 64bit.

Build ID 	20170620100236
User Agent 	Mozilla/5.0 (X11; Linux x86_64; rv:56.0) Gecko/20100101 Firefox/56.0
QA Whiteboard: [bugday-20170621]
I have reproduced this Bug on Nightly 55.0a1 (2017-05-17) on Windows 10, 64 Bit!

The bug's fix is now verified on latest  Nightly 56.0a1

Build ID    :	20170620030208
User Agent  : 	Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0

Verified on Windows, Mac, and Ubuntu.
Flags: qe-verify+
Blocks: 1387512
You need to log in before you can comment on or make changes to this bug.