Closed Bug 1698367 Opened 2 months ago Closed 2 months ago

Have specific icons for feeds rather than re-using news ones.

Categories

(SeaMonkey :: Feed Discovery and Preview, enhancement)

enhancement

Tracking

(seamonkey2.53+ fixed, seamonkey2.57esr? affected)

RESOLVED FIXED
seamonkey 2.86
Tracking Status
seamonkey2.53 + fixed
seamonkey2.57esr ? affected

People

(Reporter: iann_bugzilla, Assigned: iann_bugzilla)

Details

(Whiteboard: SM2.53.7)

Attachments

(1 file)

At the moment SeaMonkey re-uses the icons for news for feed folders / menus, we should use specific ones.

Checked under both classic and modern on linux, will need checking on macOS and Windows.

Attachment #9208965 - Flags: review?(frgrahl)
Attachment #9208965 - Flags: approval-comm-release?
Attachment #9208965 - Flags: approval-comm-esr60?

Comment on attachment 9208965 [details] [diff] [review]
1698367-new-rss-icons-2538.patch

LGTM

The icons are no longer in comm-central and need to be copied into the patch for c-c.

Attachment #9208965 - Flags: review?(frgrahl)
Attachment #9208965 - Flags: review+
Attachment #9208965 - Flags: approval-comm-release?
Attachment #9208965 - Flags: approval-comm-release+
Attachment #9208965 - Flags: approval-comm-esr60?
Attachment #9208965 - Flags: approval-comm-esr60+

I think it's possible to "revive" old versions of files by hg updating to when they existed and hg copying them into a new commit before hg merging them back to tip, although hg does like to merge subsequent changes made to the files, so you might have to manually revert those changes before committing the merge.

Pushed by frgrahl@gmx.net:
https://hg.mozilla.org/comm-central/rev/b223e15cc862
Have specific icons for feeds rather than re-using news ones. r=frg

Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED

https://gitlab.com/seamonkey-project/seamonkey-2.53-comm/-/commit/18c983f95c14c30b0d346cb4d6bf5ec9263c5ee5
Have specific icons for feeds rather than re-using news ones. r=frg a=frg

Whiteboard: SM2.53.7
Target Milestone: --- → seamonkey 2.86
You need to log in before you can comment on or make changes to this bug.