Closed Bug 54938 Opened 24 years ago Closed 24 years ago

Need to change Get New Themes URL in comm builds

Categories

(SeaMonkey :: Themes, defect, P1)

Product:
SeaMonkey
Component:
Themes
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: bugzilla, Assigned: paulkchen)

References

(
URL
)

Details

(Whiteboard: [rtm++]Fix in hand, reviewed and approved)

Attachments

(1 file)

add getNewThemesURL to commercial brand.properties
340 bytes, patch
Details | Diff | Splinter Review 
The Get New Themes menu item View > Apply Theme needs to go to Netscape Theme 
Park (http://home.netscape.com/bookmark/6_0/themes.html) in commercial builds.  
It currently goes to http://x.themes.org/viewresources.phtml?type=chrome.

  Just have to change the URL after getNewThemesURL= in brand.properties.
URL: http://home.netscape.com/bookmark/6_0...
Keywords: nsonly, rtm
OS: Windows 98 → All
Hardware: PC → All
This needs to be rtm+ in my opinion:

- Only link within browser to help users discover the theme park
- Will drive traffic to the theme park, generating page views/revenue
- Easy, low risk change

Ugh.  It's really messed up in beta 3.  It doesn't seem to work at all.
Priority: P3 → P1
Whiteboard: [rtm need info]
yes, because there's no getNewThemesURL=(url) in the brand.properties in the 
branch.  I can't checkin to the comm tree, so I couldn't add it...
taking this for Ben
Assignee: ben → pchen
Ummm, http://home.netscape.com/bookmark/6_0/themes.html doesn't work, but
http://home.netscape.com/themes/ does get you to Theme Park. Which one should it be?
Status: NEW → ASSIGNED
Raphael, this is the link that you specified - is it the right one?

adding fix in hand to status whiteboard, still need marketing to supply correct URL!
Whiteboard: [rtm need info] → [rtm need info][fix in hand]
Rafael Ebron from Netscape.com says that this page is going into production
today and should be live tomorrow, hence the reason for the link not working
right now.

This is, however the right URL:

http://home.netscape.com/bookmark/6_0/themes.html

Thanks Todd!  OK Paul, get this reviewed ...
Whiteboard: [rtm need info][fix in hand] → [rtm need info]Fix in hand
Adding to cc to alert L10n folks ...

r=blake
cc-ing scc

btw, http://home.netscape.com/bookmark/6_0/themes.html works today

rtm+ ...
Whiteboard: [rtm need info]Fix in hand → [rtm+]Fix in hand, reviewed and approved
a=scc

I've confirmed that we need to localize this URL in the localized builds, it
will be a different one per language

PDT marking [rtm++]
Whiteboard: [rtm+]Fix in hand, reviewed and approved → [rtm++]Fix in hand, reviewed and approved
Fix checked in
Status: ASSIGNED → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
verified ok on all platforms (2000-10-08-08-MN6).
Status: RESOLVED → VERIFIED
Verified ok on all platforms (2000-11-28-08-Mtrunk).
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: