Page MenuHomePhabricator

Bugzilla: Decrease our custom CSS differences with upstream
Closed, DeclinedPublic

Description

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?


Version: wmf-deployment
Severity: normal

Details

Reference
bz54823

Event Timeline

bzimport raised the priority of this task from to Lowest.Nov 22 2014, 2:29 AM
bzimport set Reference to bz54823.
bzimport added a subscriber: Unknown Object (MLST).

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

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

Wikimedia has migrated from Bugzilla to Phabricator. Learn more about it here: https://www.mediawiki.org/wiki/Phabricator/versus_Bugzilla - This task does not make sense anymore in the concept of Phabricator, hence closing as declined.