Closed Bug 1491529 Opened Last year Closed Last year

Restore footer image after bug 1471305

Categories

(Thunderbird :: Theme, enhancement)

enhancement
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 64.0

People

(Reporter: Paenglab, Assigned: Paenglab)

References

Details

Attachments

(1 file)

Bug 1471305 removed the footer image we still use.

Maybe we can resurrect it or we need to work around with mirroring the main image.
Does Thunderbird really need footer images ? additional_backgrounds apply to the footer as well according to bug 1486202. Can't that be an alternative ?
It used it until now. additional_backgrounds would only work when there are additional backgrounds. Normal LW-themes have no such.
(In reply to Richard Marti (:Paenglab) from comment #2)
> It used it until now. additional_backgrounds would only work when there are
> additional backgrounds. Normal LW-themes have no such.

LWT will be migrated to static WebExtension themes on AMO, and LWT support will get removed.
This patch uses now the --lwt-header-image instead of the --lwt-footer-image. I positioned the image right bottom to show the bottom of the header image which is almost never seen and we can say, the bottom of the window can also show the bottom of the image. Like I implemented it, it works for multiple background images too.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9011193 - Flags: review?(jorgk)
How do I test this? The WE theme bgtest@paenglab.ch.xpi or LW theme?
Both, to be sure the multi backgrounds still work and how it looks with a normal LW-theme. "Japanese Tattoo" is a good LW-theme to see the statusbar image (the text colour isn't optimal but doesn't matter for this test).
Comment on attachment 9011193 [details] [diff] [review]
statusbar-image.patch

Hmm, I don't see a difference with the WE theme bgtest@paenglab.ch.xpi, It that expected? The "Tattoo" LW theme now shows something in the statusbar again.
Attachment #9011193 - Flags: review?(jorgk) → review+
(In reply to Jorg K (GMT+2) from comment #7)
> Comment on attachment 9011193 [details] [diff] [review]
> statusbar-image.patch
> 
> Hmm, I don't see a difference with the WE theme bgtest@paenglab.ch.xpi, It
> that expected? 

Yes, when you see all three TB icons then it is working as it should.
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/185f455ddd94
Use differently positioned header image for the statusbar after the footer image removal in bug 1471305. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: Last year
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 64.0
You need to log in before you can comment on or make changes to this bug.