Closed Bug 1365663 Opened 3 years ago Closed 2 years ago

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

Categories

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

53 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 56
Iteration:
56.1 - Jun 26
Tracking Status
firefox56 --- verified

People

(Reporter: Gijs, Assigned: Gijs)

References

(Blocks 1 open bug)

Details

(Whiteboard: [photon-structure])

Attachments

(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)
Thanks!
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
Status: NEW → ASSIGNED
Iteration: --- → 56.1 - Jun 26
Priority: P2 → P1
Comment on attachment 8878473 [details]
Bug 1365663 - update synced tabs panel view iconography and alignment,

https://reviewboard.mozilla.org/r/149828/#review155084

LGTM! Thanks!

::: browser/themes/shared/customizableui/panelUI.inc.css:1259
(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 gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/909c1f2a6f21
update synced tabs panel view iconography and alignment, r=mikedeboer
https://hg.mozilla.org/mozilla-central/rev/909c1f2a6f21
Status: ASSIGNED → RESOLVED
Closed: 2 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

[bugday-20170621]
Verified on Windows, Mac, and Ubuntu.
Status: RESOLVED → VERIFIED
Flags: qe-verify+
Blocks: 1387512
You need to log in before you can comment on or make changes to this bug.