Last modified: 2014-10-16 11:52:42 UTC
Finally comparing our skins/contrib/Wikimedia/global.css in https://git.wikimedia.org/tree/wikimedia%2Fbugzilla%2Fmodifications.git to skins/standard/global.css in upstream http://bzr.mozilla.org/bugzilla/4.2/files , we have quite some delta. The Bugzilla 4.2 upstream code defines various classes which are not even listed in our custom CSS, such as .field_value .text_input .field_help_link div.bz_query_buttons .arrow_button input.requestee .bz_comment_text span.quote .bz_collapse_expand_comments .bz_collapse_expand_comments li tbody.file pre etc., etc. My guts tell me this might be the reason for numerous issues, like bug 36064 comment 14 ("input.requestee", anyone?) or maybe even bug 42850. Yeah. :-( Have I mentioned that I dislike creating custom skins and then not maintaining them anymore?
Change 86877 had a related patch set uploaded by Aklapper: Remove ugly background images of buttons when changing search result columns. Can be seen on https://bugzilla.wikimedia.org/colchange.cgi https://gerrit.wikimedia.org/r/86877
After looking at this a bit more this might be false alarm to some extend, as we load CSS in this order: <link href="skins/standard/global.css" rel="alternate stylesheet" title="Classic"> <link href="skins/standard/global.css" rel="stylesheet" type="text/css" > <link href="skins/contrib/Wikimedia/global.css" rel="stylesheet" type="text/css" title="Wikimedia"> Still some cleanup should be done, hence expect some more patches to come...
Change 87053 had a related patch set uploaded by Aklapper: Italicize bug numbers of unconfirmed tickets in "Blocks/Depends on" field. https://gerrit.wikimedia.org/r/87053
Change 87057 had a related patch set uploaded by Aklapper: Update CSS for duplicates.cgi from upstream https://gerrit.wikimedia.org/r/87057
Change 87067 had a related patch set uploaded by Aklapper: Remove custom voting.css https://gerrit.wikimedia.org/r/87067
Change 87068 had a related patch set uploaded by Aklapper: Remove unused panel.css https://gerrit.wikimedia.org/r/87068
Change 87071 had a related patch set uploaded by Aklapper: Delete unused create_attachment.css https://gerrit.wikimedia.org/r/87071
Change 87067 merged by Dzahn: Remove custom voting.css https://gerrit.wikimedia.org/r/87067
Change 87068 merged by Dzahn: Remove unused panel.css https://gerrit.wikimedia.org/r/87068
Change 87071 merged by Dzahn: Delete unused create_attachment.css https://gerrit.wikimedia.org/r/87071
Change 86877 merged by Dzahn: Remove ugly background images of buttons when changing search result columns. Can be seen on https://bugzilla.wikimedia.org/colchange.cgi https://gerrit.wikimedia.org/r/86877
the ugly buttons are gone: https://bugzilla.wikimedia.org/colchange.cgi and the unused files have been deleted in prod
Change 87057 merged by Dzahn: Remove custom CSS for duplicates.cgi https://gerrit.wikimedia.org/r/87057
No delta at all (delete our custom copy or sync): * dependency-tree.css * editusers.css * params.css * show_multiple.css * summarize-time.css Smaller irrelevant changes, either sync or delete our custom copy: * admin.css Files do not exist in upstream, need to investigate: * bug_activity.css * help.css * release-notes.css * skins.tgz * vector.css * yui/calendar.css Quite some diff: * buglist.css * global.css * IE-fixes.css * index.css * search_form.css * show_bug.css
Change 89795 had a related patch set uploaded by Aklapper: Remove custom CSS files identical to upstream standard CSS files. https://gerrit.wikimedia.org/r/89795
(In reply to comment #15) > Remove custom CSS files identical to upstream standard CSS files. > https://gerrit.wikimedia.org/r/89795 This kills the following files: > * dependency-tree.css and related images > * editusers.css > * params.css > * show_multiple.css > * summarize-time.css > * admin.css
Change 89800 had a related patch set uploaded by Aklapper: Remove unused release-notes.css https://gerrit.wikimedia.org/r/89800
Change 87053 merged by Dzahn: Italicize bug numbers of unconfirmed tickets in "Blocks/Depends on" field. https://gerrit.wikimedia.org/r/87053
Change 89800 merged by Dzahn: Remove unused release-notes.css https://gerrit.wikimedia.org/r/89800
Change 89795 merged by Dzahn: Remove custom CSS files identical to upstream standard CSS files. https://gerrit.wikimedia.org/r/89795
Change 91776 had a related patch set uploaded by Aklapper: Delete unused custom bug_activity.css file https://gerrit.wikimedia.org/r/91776
Change 91777 had a related patch set uploaded by Aklapper: Remove unused custom help.css file https://gerrit.wikimedia.org/r/91777
Change 91780 had a related patch set uploaded by Aklapper: Remove custom skins.tgz file https://gerrit.wikimedia.org/r/91780
Change 91781 had a related patch set uploaded by Aklapper: Remove unused custom yui/calendar.css and yui/sprite.png https://gerrit.wikimedia.org/r/91781
Quick update after today's cleanup patches above so I don't lose track. What's left to sort out: Files which do not exist in upstream: * vector.css Quite some diff: * buglist.css * global.css * IE-fixes.css * index.css * search_form.css * show_bug.css
* buglist.css Reviewed and tested: Can stay as is, no needless definitions in there. Yay.
* search_form.css Can stay as is. only one definition for summary_field which makes sense.
Change 91792 had a related patch set uploaded by Aklapper: Clean up custom index.css https://gerrit.wikimedia.org/r/91792
Change 91794 had a related patch set uploaded by Aklapper: Delete custom IE-fixes.css https://gerrit.wikimedia.org/r/91794
Change 91801 had a related patch set uploaded by Aklapper: Clean up custom show_bug.css https://gerrit.wikimedia.org/r/91801
Stopping for today. Left to sort out: * vector.css * global.css
Change 91839 had a related patch set uploaded by Aklapper: Remove obsolete CSS definitions. https://gerrit.wikimedia.org/r/91839
Change 91776 merged by Dzahn: Delete unused custom bug_activity.css file https://gerrit.wikimedia.org/r/91776
Change 91777 merged by Dzahn: Remove unused custom help.css file https://gerrit.wikimedia.org/r/91777
Change 91780 merged by Dzahn: Remove custom skins.tgz file https://gerrit.wikimedia.org/r/91780
Change 91781 merged by Dzahn: Remove unused custom yui/calendar.css and yui/sprite.png https://gerrit.wikimedia.org/r/91781
Change 91794 merged by Dzahn: Delete custom IE-fixes.css https://gerrit.wikimedia.org/r/91794
Change 91839 merged by Dzahn: Remove obsolete CSS definitions. https://gerrit.wikimedia.org/r/91839
Assuming that we'll get https://gerrit.wikimedia.org/r/#/c/91792/ and https://gerrit.wikimedia.org/r/#/c/91801/ merged soonish, I'll unassign myself from this task for the time being as there's other stuff on the plate. Two files are left to investigate for some other day: * https://git.wikimedia.org/blob/wikimedia%2Fbugzilla%2Fmodifications.git/HEAD/skins%2Fcontrib%2FWikimedia%2Fvector.css * https://git.wikimedia.org/blob/wikimedia%2Fbugzilla%2Fmodifications.git/HEAD/skins%2Fcontrib%2FWikimedia%global.css
Change 91792 merged by Dzahn: Sync our custom index.css with upstream https://gerrit.wikimedia.org/r/91792
Change 91801 merged by Dzahn: Clean up custom show_bug.css https://gerrit.wikimedia.org/r/91801