Closed
Bug 1365663
Opened 8 years ago
Closed 7 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)
Tracking
()
Tracking | Status | |
---|---|---|
firefox56 | --- | verified |
People
(Reporter: Gijs, Assigned: Gijs)
References
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.
Assignee | ||
Updated•8 years ago
|
Flags: qe-verify+
Whiteboard: [photon-structure]
Updated•8 years ago
|
Priority: -- → P2
QA Contact: gwimberly
Assignee | ||
Comment 1•7 years ago
|
||
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)
Assignee | ||
Comment 3•7 years ago
|
||
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 | ||
Updated•7 years ago
|
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Iteration: --- → 56.1 - Jun 26
Priority: P2 → P1
Comment hidden (mozreview-request) |
Comment 5•7 years ago
|
||
mozreview-review |
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+
Comment hidden (mozreview-request) |
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/909c1f2a6f21
update synced tabs panel view iconography and alignment, r=mikedeboer
Comment 8•7 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
status-firefox56:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
Comment 9•7 years ago
|
||
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]
Comment 10•7 years ago
|
||
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]
Comment 11•7 years ago
|
||
Verified on Windows, Mac, and Ubuntu.
You need to log in
before you can comment on or make changes to this bug.
Description
•