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.
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?
Created attachment 16275 [details] [diff] [review] add getNewThemesURL to commercial brand.properties
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 ...
btw, http://home.netscape.com/bookmark/6_0/themes.html works today
Whiteboard: [rtm need info]Fix in hand → [rtm+]Fix in hand, reviewed and approved
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
Last Resolved: 18 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).
You need to log in before you can comment on or make changes to this bug.