256207 - move CSS files to new location

Status: RESOLVED FIXED

(Bugzilla :: User Interface, defect)

Description

[Myk Melez [:myk] [@mykmelez]]

Per the Bugzilla CSS plan, move CSS files to skins/standard/:

css/buglist.css => skins/standard/buglist.css
css/duplicates.css => skins/standard/duplicates.css
css/edit_bug.css => skins/standard/edit_bug.css
css/global.css => skins/standard/global.css
css/panel.css => skins/standard/panel.css
css/show_multiple.css => skins/standard/show_multiple.css

We should physically move these on the CVS server to preserve history and then
cvs remove them from their old locations.

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v1: moves files to new location

In addition to applying this patch we also need to create the new directories,
move the actual files on the CVS repository, and cvs remove the old ones.

Comment 2

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 156557 [details] [diff] [review]
patch v1: moves files to new location

Gerv, can you take a look at this one as well?	It's part 2 of implementing the
CSS plan.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v2: removes no-longer-necessary patch for show.html.tmpl

show.html.tmpl no longer has its own CSS file, so removing the patch that
changes its location.

Comment 4

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 156557 [details] [diff] [review]
patch v1: moves files to new location

Moving this review to the newer patch.

Comment 5

[Gervase Markham [:gerv]]

Comment on attachment 160290 [details] [diff] [review]
patch v2: removes no-longer-necessary patch for show.html.tmpl

Are we supposed to be hard-coding "standard" everywhere, or is this not
supposed to be some sort of parameter?

Gerv

Comment 6

[Myk Melez [:myk] [@mykmelez]]

> Are we supposed to be hard-coding "standard" everywhere, or is this not
> supposed to be some sort of parameter?

It's going to be a parameter, but we're doing the work in stages.  Right now
we're just moving files to the right places.  Later we're going to make it
possible for installations to install multiple skins and switch between them.

Comment 7

[Gervase Markham [:gerv]]

Comment on attachment 160290 [details] [diff] [review]
patch v2: removes no-longer-necessary patch for show.html.tmpl

r=gerv.

Gerv

Comment 8

[Myk Melez [:myk] [@mykmelez]]

Thanks Gerv.  Dave says this can go into 2.20.  Files moved, fix checked in.

Checking in duplicates.xul;
/cvsroot/mozilla/webtools/bugzilla/duplicates.xul,v  <--  duplicates.xul
new revision: 1.2; previous revision: 1.1
done
Removing css/buglist.css;
/cvsroot/mozilla/webtools/bugzilla/css/buglist.css,v  <--  buglist.css
new revision: delete; previous revision: 1.4
done
Removing css/duplicates.css;
/cvsroot/mozilla/webtools/bugzilla/css/duplicates.css,v  <--  duplicates.css
new revision: delete; previous revision: 1.2
done
Removing css/global.css;
/cvsroot/mozilla/webtools/bugzilla/css/global.css,v  <--  global.css
new revision: delete; previous revision: 1.6
done
Removing css/panel.css;
/cvsroot/mozilla/webtools/bugzilla/css/panel.css,v  <--  panel.css
new revision: delete; previous revision: 1.1
done
Removing css/show_multiple.css;
/cvsroot/mozilla/webtools/bugzilla/css/show_multiple.css,v  <--  show_multiple.css
new revision: delete; previous revision: 1.1
done
Checking in template/en/default/sidebar.xul.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/sidebar.xul.tmpl,v  <-- 
sidebar.xul.tmpl
new revision: 1.15; previous revision: 1.14
done
Checking in template/en/default/bug/show-multiple.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/show-multiple.html.tmpl,v
 <--  show-multiple.html.tmpl
new revision: 1.18; previous revision: 1.17
done
Checking in template/en/default/global/header.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/global/header.html.tmpl,v
 <--  header.html.tmpl
new revision: 1.29; previous revision: 1.28
done
Checking in template/en/default/list/list-simple.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/list-simple.html.tmpl,v
 <--  list-simple.html.tmpl
new revision: 1.8; previous revision: 1.7
done
Checking in template/en/default/list/list.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/list/list.html.tmpl,v 
<--  list.html.tmpl
new revision: 1.27; previous revision: 1.26
done