Hi Andreas, On Fri, Dec 12, 2008 at 01:01:42PM +0100, Andreas Demmer wrote:
I changed all stylesheet/javascript /image links of the openSUSE frontpage. They now point hard-coded to frontpage SVN checkout on static.opensuse.org
This reduces the server load since static.opensuse.org is highly optimized to deliver static content.
The code is now in SVN but not deployed on the live server yet. Reviews of my changes are highly welcome.
I reviewed your change: 1) There is an invalid link in js/fancyzoom_html.js to http://static.opensuse.org/hosts/www.o.o/images/closebox.png although it is commented, so not in use. The URLs constructed by Javascript rely on a common path which is set here: js/fancyzoom.js:var zoomImagesURI = '/images/fancyzoom/'; // Location of the zoom and shadow images I guess this needs to be changed too. (Old URLs will still be valid of course. So, no breakage is to be feared.) 2) These two won't work: http://static.opensuse.org/hosts/www.o.o/images/jcarousel_skins/opensuse/nex... http://static.opensuse.org/hosts/www.o.o/images/jcarousel_skins/opensuse/pre... The jcarousel_skins directory contains only pictures named horizontal-something, not vertical-something. 3) http://static.opensuse.org/hosts/www.o.o/images/spacer.gif should be http://static.opensuse.org/hosts/www.o.o/images/fancyzoom/spacer.gif And the following http://static.opensuse.org/hosts/www.o.o/images/zoom-shadow1.png http://static.opensuse.org/hosts/www.o.o/images/zoom-shadow3.png http://static.opensuse.org/hosts/www.o.o/images/zoom-shadow6.png http://static.opensuse.org/hosts/www.o.o/images/zoom-shadow8.png also need to have /images/fancyzoom/ instead of /images/ if they should work. (I didn't check if they are used at all though, or if they might just occur in a comment) Peter -- Contact: admin@opensuse.org (a.k.a. ftpadmin@suse.com) #opensuse-mirrors on freenode.net Info: http://en.opensuse.org/Mirror_Infrastructure SUSE LINUX Products GmbH Research & Development