| Submitter | Diff | Changes | Open Issues | Last Updated |
|---|---|---|---|---|
| Loading... | ||||
| Error loading review requests: | ||||
Created attachment 8836276 [details]
sidebar.xpi
Install an add-on with a sidebar and no icon. For the first tab, the icon does not appear in the sidebar menu. Then open a new tab, icon appears.
|
(Reporter) | |
Comment 1•4 months ago
|
||
Created attachment 8836277 [details]
no-icon-first-tab.mov
|
(Assignee) | |
Comment 2•4 months ago
|
||
Looks like the generic icon used when no icon defined. I'm guessing (based on my test addon which I didn't see that with) that defining an icon doesn't have the same issue.
|
|
(Assignee) | |
Updated•4 months ago
|
||
|
|
(Assignee) | |
Comment 3•4 months ago
|
||
STR: - install attached xpi - open new window - check for icon in toolbarbutton This turned out to be from opening an new window, which didn't go through the code path that added the style for the icon.
| Comment hidden (mozreview-request) |
|
||
Comment 5•4 months ago
|
||
| mozreview-review | ||
Comment on attachment 8836889 [details] Bug 1338726 fix icon style for new windows, https://reviewboard.mozilla.org/r/112208/#review113472 Looks good, but please add a test.
| Comment hidden (mozreview-request) |
|
|
||
Comment 7•4 months ago
|
||
| mozreview-review | ||
Comment on attachment 8836889 [details] Bug 1338726 fix icon style for new windows, https://reviewboard.mozilla.org/r/112208/#review114272
Pushed by mixedpuppy@gmail.com: https://hg.mozilla.org/integration/autoland/rev/6a03b96cc5ba fix icon style for new windows, r=kmag
|
||
Comment 9•4 months ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/6a03b96cc5ba
Description
•