@splatio has done a truely amazing job at overriding core CSS and markup in the sandbox in the past weeks: http://drupal.org/sandbox/sumsi/1708250. No exaggeration: It's brilliant and will give us a major boost forward. So... What we need to do now is:

a) Write a proper change notification that reflects all the markup and class changes so people who would like to update their sites can do so with less headache.
b) Merge.
c) Tag next alpha (last alpha before beta)

Comments

fubhy’s picture

fubhy commented
Title: Prepare for merging sandbox version back into latest 7.x-4.x » Prepare for merging sandbox version back into HEAD

Clarifying title...

msmithcti’s picture

msmithcti commented

Thanks fubhy! I thought I'd give a quick summary here for anyone that doesn't want to root through the sandbox (but you should, and give feedback ;) ). So, all core modules (except a few low priority ones such as color, dashboard and dblog) have had overrides added in Omega, we've tried to focus on the following:

  • Adhering to the Drupal 8 CSS coding standards, this includes following SMACSS, using BEM class naming where possible and reducing selector specificity.
  • CSS has been split up into base, admin and theme files which allows you to easily override or remove only parts of the CSS (for instance by using the "Exclude CSS files" textarea in Omega's settings). Base only contains the styles that affect functionality, which means you can safely strip out the theme CSS for an instant performance gain without worrying about breaking things.
  • In some places, core CSS tries to guess too much about how you want something to look, this has been removed as much as possible so you don't have to override it in your subtheme's stylesheets.

There's still plenty left to do! The cleanup of system module is well underway, there's also the CSS for vertical tabs to tackle. This all then needs a second pass to make sure anything that should have been removed, kept or moved has not been missed along with more testing.

msmithcti’s picture

msmithcti commented
Status: Active » Closed (fixed)

This was merged last week! Hurray! Setting this to fixed :)