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)

enhancement
Not set
normal

Tracking

(Not tracked)

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.
Blocks: 1381969
Blocks: 939540
You need to log in before you can comment on or make changes to this bug.