Open
Bug 1382133
Opened 7 years ago
Updated 4 years ago
Port bug 534083 to SM: Provide image scaling for embedded images
Categories
(SeaMonkey :: MailNews: Message Display, enhancement)
SeaMonkey
MailNews: Message Display
Tracking
(Not tracked)
NEW
People
(Reporter: thomas8, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [tb31features])
+++ This bug was initially created as a clone of Bug #534083 +++
See TB's patch in attachment 830240 [details] [diff] [review].
This is about images originally embedded in HTML content, as opposed to attachments displayed inline, which are already auto-scaling.
So after this bug, *all* images will be "shrink-to-fit", and it's unlikely and nonsensical for users to want otherwise. So we can remove this SeaMonkey-only pref which is currently still handled in shared code:
mail.enable_automatic_image_resizing
(In reply to Ian Neal from bug 534083 comment #32)
> I would say yes
(In reply to Thomas D. (currently busy elsewhere; needinfo?me) from bug 534083 comment #31)
> Ian, does SeaMonkey want this? Nice feature, simple patch...
> Which would allow us to do the cleanup mentioned below...
>
> (In reply to alta88 from bug 534083 comment #27)
> > by the way, if suite takes this patch, they should remove the pref and also
> > clean up the code here:
> > http://mxr.mozilla.org/comm-central/source/mailnews/mime/src/mimemoz2.cpp#1217
>
> https://dxr.mozilla.org/comm-central/source/mailnews/mime/src/mimemoz2.
> cpp#1107-1115
> ...and some other spots related to that.
>
> > imo, the class can be kept as it can't hurt to identify img attachments for
> > whoever might need it.
You need to log in
before you can comment on or make changes to this bug.
Description
•