120985 - Get New Themes should point to xulplanet.com, not themes.org

Status: VERIFIED FIXED

(SeaMonkey :: Themes, defect, P3)

Description

[Tom Jac]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (X11; U; Linux i586; en-US; rv:0.9.7) Gecko/20011221
BuildID:    all

view themes shows just the old www.themes.org and not www.xulplanet.com

Reproducible: Always
Steps to Reproduce:
1. just click
2.
3.

Comment 1

[Christopher Aillon (sabbatical, not receiving bugmail)]

*** Bug 120374 has been marked as a duplicate of this bug. ***

Comment 2

[Christopher Aillon (sabbatical, not receiving bugmail)]

Cool, I didn't know xulplanet.com had themes.  This seems very reasonable. 
They've got 5 themes already.

Confirming bug.

Comment 3

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Patch: Switch the links

I decided to link to their download page because that's what we currently do
with themes.org.

URL points here:
http://xulplanet.com/downloads/view.cgi?category=skins&view=all

Comment 4

[Matthias Versen [:Matti]]

AFAIK this menu entry will removed but we schould point to XULplanet in the meantime

Comment 5

[Jeremy M. Dolan]

Matti: I filed bug 120374 four days ago to remove it, but Netscape is
complaining it'll be too hard for them to re-add it in their builds... boo-hoo.
I'm eventually going to be talking to drivers about all of the third-party crap
we ship with Mozilla (What's Related probably being the worst offense).

Comment 6

[André Dahlqvist]

The link in Preferences is the same. Is that one going to be removed too?

Comment 7

[Blake Ross]

Oh, grow up. I was referring to distributors in general and I was right. I'd be 
happy to discuss better solutions if you've got any.

Comment 8

[Hixie (not reading bugmail)]

caillon: don't the "&" signs in the DTD file have to be escaped with "&" ?

Comment 9

[Christopher Aillon (sabbatical, not receiving bugmail)]

Attachment: Patch v2

Well, of course they do.  Duh.	;-)

Comment 10

[Hixie (not reading bugmail)]

r=hixie

Comment 11

[David Hyatt]

sr=hyatt

Comment 12

[Aaron Andersen]

Is this going into the 0.9.8 branch?

Comment 13

[Hixie (not reading bugmail)]

To get into 0.9.8, you need an a=, and to get an a=, you need to send a request
by e-mail to [e-mail].

Comment 14

[Aaron Andersen]

I know all about a= and such.  I was actually asking if the person who was going
to check this in was planning on getting one.  I could of course send such a
request myself, but this isn't really my bug, and I'm not entirely sure someone
else hasn't already done so.

Comment 15

[Christopher Aillon (sabbatical, not receiving bugmail)]

Aaron: I hadn't intended on requesting approval for 0.9.8 but it appears there
is some want for this to make 0.9.8 so I will mail drivers.  It is a cool little
link and I bet users will be surprised that there really are themes out there (I
certainly was).

Taking bug to request approval.

Comment 16

[Asa Dotzler [:asa]]

a=asa (on behalf of drivers)

Comment 17

[Hasbullah Pit]

Wait...

The url :-
http://xulplanet.com/downloads/view.cgi?category=skins&view=all
returned error 404 , 1 minutes before i post this.

Comment 18

[Christopher Aillon (sabbatical, not receiving bugmail)]

Yes, XUL Planet is currently upgrading their server software.  We noticed this
and I am holding check-in pending its return.  Sorry, I thought I posted
something in this bug already.

Comment 19

[Aaron Andersen]

The server upgrade has been completed.  We'ar just coming back online now.

Comment 20

[Boris Zbarsky [:bzbarsky]]

Checked into trunk.  Will land on branch once that reopens for checkins.

Comment 21

[ajbu]

Wouldn’t it be better to have the ‘Get New Themes’ URL link to a page that can 
be maintained by Mozilla (for example something like www.mozilla.org/themes), 
and put the actual theme site URLs on that page, or let that page jump forward 
to xulplanet? 

Comment 22

[Tom Jac]

To "Additional Comment #21"
Yes, best idea!

Comment 23

[Boris Zbarsky [:bzbarsky]]

Checked into branch.

Comment 24

[Patty Mac]

Verified (mozilla build: 2002-02-04-08-trunk)