Mark Banner (:standard8) Assignee
	Description • 18 years ago

After, or at the time of, the xpfe to toolkit switch over on trunk we need to remove the themes pane from the preferences window as it'll no longer be required.

Note: this does *not* mean we're near to being ready to switch over.

The xpfe builds still require the themes pane, and to save having to ifdef xul and js files etc, we've decided to leave xpfe builds along in this respect and apply the patch (that I'll attach later) at the time of the change over.

I'm putting it here mainly for reference (so its ready when we need it) and if anyone wants to test it (not that it affects much).

	lenochod
	Comment 1 • 18 years ago

It is view in right parts vindow:

XML Parsing Error: undefined entity
Location: chrome://communicator/content/pref/pref-themes.xul
Line Number 109, Column 31:  <label class="small-margin"><html:a href="&getNewThemesURL;" id="themesLink"

	Mark Banner (:standard8) Assignee
	Comment 2 • 18 years ago

Like I said in comment 0, this patch removes the themes pref pane from all SeaMonkey builds. We won't be implementing this until the time we switch to suiterunner on trunk (bug 328887) but if anyone wishes to test/run with this, feel welcome. I'm putting it here so its ready for when its needed.

	Mark Banner (:standard8) Assignee
	Comment 3 • 18 years ago

Comment on attachment 258964 [details] [diff] [review]
Removes the themes pane.

We're not going to land this until the switch, but I'd like to get reviews on it so once we're ready to switch we can land this at the same time.

	neil@parkwaycc.co.uk
	Comment 4 • 18 years ago

Comment on attachment 258964 [details] [diff] [review]
Removes the themes pane.

Doesn't the Languages panel need to go too?

	Mark Banner (:standard8) Assignee
	Comment 5 • 18 years ago

(In reply to comment #4)
> (From update of attachment 258964 [details] [diff] [review])
> Doesn't the Languages panel need to go too?
> 
If I remember correctly Robert said it needed reworking into something else.

	Robert Kaiser
	Comment 6 • 18 years ago

Right. We need our languages panel as long as bug 377881 is open - I'd love to get that one fixed and remove our language switching panel as well.

	Mark Banner (:standard8) Assignee
	Comment 7 • 18 years ago

Patch checked in. Fixed.

	Ian Neal
	Comment 8 • 18 years ago

Sorry, should have mentioned we need amend help references to themes. Do you want that done as a separate bug?

	Mark Banner (:standard8) Assignee
	Comment 9 • 18 years ago

(In reply to comment #8)
> Sorry, should have mentioned we need amend help references to themes. Do you
> want that done as a separate bug?
> 
Yeah, probably should block 373359 on that as well.