There are a bunch of CSS/image files in toolkit/themes/winstripe/help and toolkit/themes/pinstripe/help that aren't needed when the help viewer isn't built. We should package them only when MOZ_HELP_VIEWER is defined.
Hi Gavin, you asked for it, you review it ;-)
Assignee: jwalden+fxhelp → steffen.wilberg
Status: NEW → ASSIGNED
Attachment #333110 - Flags: review?(gavin.sharp)
Let's not ship another release with a bunch of unused images.
Comment on attachment 333110 [details] [diff] [review] add some "ifdef MOZ_HELP_VIEWER" Thanks, Steffen :) Can you ifdef http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/toolkit/locales/jar.mn&rev=1.48&mark=110-114#110 too while you're at it? Or file a new bug...
Attachment #333110 - Flags: review?(gavin.sharp) → review+
Summary: Don't package help viewer CSS/image files ifndef MOZ_HELP_VIEWER → Don't package help viewer theme/locale files
Status: ASSIGNED → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a2
You need to log in before you can comment on or make changes to this bug.