281725 - Remove some obsolete images in en-US/images

Status: VERIFIED FIXED

(SeaMonkey :: Help Documentation, defect)

Description

[Stefan [:stefanh]]

There are some old gif images in the image dir that are not used:

key.gif
newmail.gif
pen.gif
sick.gif
smile.gif

Also, pen.gif and newmail.gif are in jar.mn ...

Comment 1

[Stefan [:stefanh]]

Attachment: jar.mn image reference removal

At least this will save some footprint. The images listed above has to be cvs
removed as well.

Comment 2

[Ian Neal]

Comment on attachment 173896 [details] [diff] [review]
jar.mn image reference removal

>-        locale/en-US/help/images/help_print.gif         (locale/en-US/images/help_print.gif)	
>+        locale/en-US/help/images/help_print.gif         (locale/en-US/images/help_print.gif)    
Trailing spaces at the end of this entry could be removed too.

>-        locale/en-US/help/images/newmail.gif            (locale/en-US/images/newmail.gif)
>-	locale/en-US/help/images/mail_flag_column.png         (locale/en-US/images/mail_flag_column.png)
>-	locale/en-US/help/images/mail_flag.png                (locale/en-US/images/mail_flag.png)
>+        locale/en-US/help/images/mail_flag_column.png         (locale/en-US/images/mail_flag_column.png)
>+        locale/en-US/help/images/mail_flag.png                (locale/en-US/images/mail_flag.png)
The second entries could be lined up too.

>@@ -79,7 +77,7 @@
>         locale/en-US/help/images/table.gif                     (locale/en-US/images/table.gif)
>         locale/en-US/help/images/task_mail.png             (locale/en-US/images/task_mail.png)
>         locale/en-US/help/images/task_newmail.png          (locale/en-US/images/task_newmail.png)
>-	locale/en-US/help/images/taskbar-ab.png                  (locale/en-US/images/taskbar-ab.png)
>+        locale/en-US/help/images/taskbar-ab.png                  (locale/en-US/images/taskbar-ab.png)
Again the second entries could be lined up too.

r= with those changes

Comment 3

[Stefan [:stefanh]]

Attachment: New version

New version addressing Ians comments.

Comment 4

[Stefan [:stefanh]]

Comment on attachment 173995 [details] [diff] [review]
New version

Simple, small footprint-reducer. Patch+removal of obsolete images.

Comment 5

[Christopher Aillon (sabbatical, not receiving bugmail)]

(In reply to comment #0)
> Also, pen.gif and newmail.gif are in jar.mn ...

newmail.gif still exists in jar.mn with your patch....

Comment 6

[Stefan [:stefanh]]

Attachment: Correct version (Checked in)

(In reply to comment #5)
> newmail.gif still exists in jar.mn with your patch....

Here's the correct version. Sorry.

Comment 7

[Stefan [:stefanh]]

Comment on attachment 173995 [details] [diff] [review]
New version

Clearing obsolete approval request.

Comment 8

[Christopher Aillon (sabbatical, not receiving bugmail)]

Comment on attachment 174133 [details] [diff] [review]
Correct version (Checked in)

a=caillon for 1.8b

Comment 9

[Stefan [:stefanh]]

Ian,

can you check in the patch and remove the obsolete images, please.

Comment 10

[Ian Neal]

Comment on attachment 174133 [details] [diff] [review]
Correct version (Checked in)

Checking in jar.mn;
/cvsroot/mozilla/extensions/help/resources/jar.mn,v  <--  jar.mn
new revision: 1.65; previous revision: 1.64
done
Removing locale/en-US/images/key.gif;
/cvsroot/mozilla/extensions/help/resources/locale/en-US/images/key.gif,v  <-- 
key.gif
new revision: delete; previous revision: 1.1
done
Removing locale/en-US/images/newmail.gif;
/cvsroot/mozilla/extensions/help/resources/locale/en-US/images/newmail.gif,v 
<--  newmail.gif
new revision: delete; previous revision: 1.6
done
Removing locale/en-US/images/pen.gif;
/cvsroot/mozilla/extensions/help/resources/locale/en-US/images/pen.gif,v  <-- 
pen.gif
new revision: delete; previous revision: 1.2
done
Removing locale/en-US/images/sick.gif;
/cvsroot/mozilla/extensions/help/resources/locale/en-US/images/sick.gif,v  <-- 
sick.gif
new revision: delete; previous revision: 1.2
done
Removing locale/en-US/images/smile.gif;
/cvsroot/mozilla/extensions/help/resources/locale/en-US/images/smile.gif,v  <--
 smile.gif
new revision: delete; previous revision: 1.2
done

Comment 11

[Stefan [:stefanh]]

--> Fixed

Comment 12

[Ian Neal]

Really marking as fixed for Stefan

Comment 13

[Stefan [:stefanh]]

v