Closed
Bug 1457787
Opened 7 years ago
Closed 6 years ago
Remove feedIcon PNGs in favor of feed.svg
Categories
(Firefox :: Theme, enhancement, P3)
Firefox
Theme
Tracking
()
RESOLVED
DUPLICATE
of bug 1477669
People
(Reporter: ntim, Assigned: jason)
References
(Blocks 1 open bug)
Details
(Keywords: good-first-bug)
https://searchfox.org/mozilla-central/search?q=feedIcon&case=false®exp=false&path=
The jar.mn references + the files themselves can be removed.
The other references (CSS and JS) can be replaced with chrome://browser/skin/feed.svg
Updated•7 years ago
|
Priority: -- → P3
Updated•7 years ago
|
Assignee: nobody → 1991manish.kumar
Comment 1•6 years ago
|
||
what about these lines?
https://searchfox.org/mozilla-central/source/browser/components/preferences/in-content/main.js#3207,3216,3225
I don't think so 'chrome://browser/skin/feed16.svg' will work?
Flags: needinfo?(ntim.bugs)
Reporter | ||
Comment 2•6 years ago
|
||
(In reply to Manish Kumar [:manishkk] from comment #1)
> what about these lines?
> https://searchfox.org/mozilla-central/source/browser/components/preferences/
> in-content/main.js#3207,3216,3225
>
> I don't think so 'chrome://browser/skin/feed16.svg' will work?
You can just use chrome://browser/skin/feed.svg
Flags: needinfo?(ntim.bugs)
Comment 3•6 years ago
|
||
(In reply to Tim Nguyen :ntim from comment #2)
> (In reply to Manish Kumar [:manishkk] from comment #1)
> > what about these lines?
> > https://searchfox.org/mozilla-central/source/browser/components/preferences/
> > in-content/main.js#3207,3216,3225
> >
> > I don't think so 'chrome://browser/skin/feed16.svg' will work?
>
> You can just use chrome://browser/skin/feed.svg
but I think the correct path of 'feed.svg' is this-
https://searchfox.org/mozilla-central/search?q=path%3Afeed.svg&case=false®exp=false&path=
Flags: needinfo?(ntim.bugs)
Reporter | ||
Comment 4•6 years ago
|
||
When Firefox is compiled, the url is changed to chrome://browser/skin/feed.svg.
If you type that link into your urlbar, you will see the correct icon appear.
Flags: needinfo?(ntim.bugs)
Comment 5•6 years ago
|
||
How can I test for these kinds of bugs? How can I ensure that what changes I did those are correct?
Any command?
Flags: needinfo?(ntim.bugs)
Reporter | ||
Comment 6•6 years ago
|
||
(In reply to Manish Kumar [:manishkk] from comment #5)
> How can I test for these kinds of bugs? How can I ensure that what changes I
> did those are correct?
> Any command?
The icon is used at different places in the UI:
- https://www.reddit.com/r/firefox/.rss
- about:preferences#general -> scroll down to "Applications"
You need to manually check if the icon is correctly replaced in both of these links.
Flags: needinfo?(ntim.bugs)
Updated•6 years ago
|
Assignee: 1991manish.kumar → nobody
Keywords: good-first-bug
Assignee | ||
Comment 7•6 years ago
|
||
Hi Tim,
Can I take this one too?
Thanks!
Flags: needinfo?(ntim.bugs)
Reporter | ||
Comment 8•6 years ago
|
||
Yeah, sure :) Please let me know if you have any questions.
Assignee: nobody → jason
Flags: needinfo?(ntim.bugs)
Assignee | ||
Comment 9•6 years ago
|
||
So just to confirm, I should replace audoFeedIcon and videoFeedIcon with feed.svg as well?
Flags: needinfo?(ntim.bugs)
Reporter | ||
Comment 10•6 years ago
|
||
(In reply to Jason Chapin from comment #9)
> So just to confirm, I should replace audoFeedIcon and videoFeedIcon with
> feed.svg as well?
Yes.
Flags: needinfo?(ntim.bugs)
Assignee | ||
Comment 11•6 years ago
|
||
I hit a snag, as part of bug 1491243 feed.svg was removed. Should I add it back, or how should I proceed if not?
thanks!
-jason
Flags: needinfo?(ntim.bugs)
Reporter | ||
Comment 12•6 years ago
|
||
(In reply to Jason Chapin from comment #11)
> I hit a snag, as part of bug 1491243 feed.svg was removed. Should I add it
> back, or how should I proceed if not?
> thanks!
> -jason
Hmm, it looks like bug 1477669 would also remove feedIcon.png/audioFeedIcon.png/videoFeedIcon.png, so I'm going to dupe to that one.
Let me know if you need help finding another good-first-bug. Sorry :(
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(ntim.bugs)
Resolution: --- → DUPLICATE
Assignee | ||
Comment 13•6 years ago
|
||
NP, it was a learning experience at least :)
If you know of another good-first-bug please send it my way thanks!
Flags: needinfo?(ntim.bugs)
Reporter | ||
Comment 14•6 years ago
|
||
(In reply to Jason Chapin from comment #13)
> NP, it was a learning experience at least :)
> If you know of another good-first-bug please send it my way thanks!
You can try bug 1491614, bug 1457782 or bug 1451715.
You can look at this list if none of those 3 bugs interest you: https://codetribute.mozilla.org/projects/ff
Flags: needinfo?(ntim.bugs)
You need to log in
before you can comment on or make changes to this bug.
Description
•