99514 - arrow-right.gif vs. arrow-rit.gif

Status: VERIFIED FIXED

(SeaMonkey :: Themes, defect)

Description

[risto.kankkunen]

Classic skin has chrome://global/skin/arrow-right.gif while Modern has
chrome://global/skin/arrow/arrow-rit.gif. However, even though files

themes/modern/messenger/threadpane.css
themes/modern/editor/editordialog.css

refer to arrow-rit.gif, themes/modern/global/wizardoverlay.css refers to the
non-existing arrow-right.gif of Modern skin.

It would seem less confusing and more maintainable, if both skins would use
files with the same names where possible. Renaming arrow-rit.gif and changing
the references in the first two files would achieve this.

The other option is to keep the disparity and just to fix the references in the
wizardoverlay.css.

This applies to left and down arrows, too.

Steps to repeat:
1. Select Modern theme
2. Select "Mail & Newsgroups Account settings..." in Edit menu of Main News
3. Press "New Account" button
Actual results:
No arrows in Back and Next buttons.
Error messages on console (debug build)

Comment 1

[Joe Hewitt (gone)]

Mass reassigning my theme bugs to Shuehan.

Comment 2

[shliang]

Attachment: patch

Comment 3

[Joe Hewitt (gone)]

Comment on attachment 52102 [details] [diff] [review]
patch

sr=hewitt

Comment 4

[andreww]

r=andreww

Comment 5

[shliang]

Attachment: doing this over

Comment 6

[shliang]

checked in 

Comment 7

[Patty Mac]

Verified on all platforms (2001-11-14-08-trunk)