Skip to content
This repository has been archived by the owner on Aug 26, 2022. It is now read-only.

Bug 1070001: Organizing custom CSS #3670

Merged
merged 1 commit into from Nov 24, 2015
Merged

Bug 1070001: Organizing custom CSS #3670

merged 1 commit into from Nov 24, 2015

Conversation

stephaniehobson
Copy link
Contributor

  • combine duplicate bug and warning declarations
  • add dividers in comments
  • remove unused fa-trash class
  • better :last-child control in indicators
  • remove RTL only styling for page-content (we should fix MDN styles if there's a specific problem rather than over-riding. I could not see a compelling reason for lists in the one place this was used to have their own special list styling)
  • remove unused coloured icon classes experimentalIcon, nonstandardIcon, obsoleteIcon, deprecatedIcon (coloured icons not generated this way anymore)
  • remove duplicate definitions for #wikiArticle i.icon-beaker and other coloured icons
  • remove references to unused #nav-sub-docs

Testing:

- remove duplicate bug and warning declarations
- move icons to be together in source
- add dividers in comments
- remove unused fa-trash class
- better :last-child control in indicators
- remove unused .page-content dd declaration
- remove RTL only styling for .page-content (we should fix MDN styles if there's a specific problem rather than over-riding. I could not see a compelling reason for lists on the 1 RTL page with the .page-content class to have its own special list styling)
- remove unused coloured icon classes (coloured icons not generated this way anymore)
- remove references to unused #nav-sub-docs
openjck added a commit that referenced this pull request Nov 24, 2015
Bug 1070001: Organizing custom CSS
@openjck openjck merged commit 5731b46 into mdn:master Nov 24, 2015
@stephaniehobson stephaniehobson deleted the stylus-reorg branch November 25, 2015 00:40
@stephaniehobson stephaniehobson restored the stylus-reorg branch December 3, 2015 00:03
@stephaniehobson stephaniehobson deleted the stylus-reorg branch December 3, 2015 00:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants