Responses embedded below -
Brent Beardsley wrote:
replace line 117 with
var confthememenu = mb.AppendSubMenu("Load color
scheme", hmenu);
Ah very cool!.. I see what I was doing wrong..
Hmm I dont understand. The menu item copies the
localsearch_xxx.css
over localsearch.css and does a toolbar reload..
Do you mean there
should be a way to empty localsearch.css? or is
there a localsearch.css
delivered with a clean install that you want to
revert to?
Yes
Suggestion: We create a third css file for the
toolbar - theme.css, and
over-write this file with the theme menu selection
and have the
theme.css loaded by the toolbar *after* search.css
and *before*
localsearch.css. It seems like the simplest
solution to me, and the
toolbar could just clear out the theme.css file to
revert to a no-theme
setting..
Sounds good to me. Two other suggestions:
1) in the menu we should identify the current themes
as XP themes as they look terrible on others and don't
even work properly - hence the need to reset it back
to the default
2) if we go with the theme.css we should rename the
localsearch_blue, localsearch_olive, etc to
theme_blue, theme_olive
Agreed..
Attached is a zip archive cssmods.zip(.txt) It has all the above
mods. -
Submenu moved under Configure
Fourth selection "None" added to theme selection
Theme css files renamed to xxx_theme.css
Search.html mod to load theme.css file in between search.css and
localsearch.css
Don't forget to rename your localsearch.css file if yours is a copy of
one of the theme css files!
I like the changes. I think it is an improvement.. I definitely feel
more at ease with not over-writing the localsearch.css files..
Suggestions? Comments? Do I check the mods in? Does anyone know how
to rename a file in CVS or do we just create new entries with the new
css filenames?
Monty
|
PK �j�1l"��% &|
&