Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Spin-off from #1510532: [META] Implement the new create content page design - a more modern CSS reset for Seven theme. Please refer to original issue for reasoning.
Comment | File | Size | Author |
---|---|---|---|
#3 | 1747868-3-new-reset.patch | 10.83 KB | mrfelton |
#1 | 1747868-1-new-reset.patch | 10.23 KB | kika |
Comments
Comment #1
kika CreditAttribution: kika commentedAnd here's the patch.
Comment #2
mrfelton CreditAttribution: mrfelton commentedI totally agree with this. But, isn't a reset.css supposed to come first in order of loaded css files? What we are getting at the moment is like this:
So the reset stuff is happening after a bunch of module level css styles. Wonder if we can/should force it to load first?
Comment #3
mrfelton CreditAttribution: mrfelton commentedUpdated patch to ensure that reset.css is the first stylesheet loaded.
Comment #4
kika CreditAttribution: kika commentedIn order to get this in, it needs reviews from CSS team and good issue summary. Could someone go back to #1510532: [META] Implement the new create content page design and figure out at what point new reset was introduced and why that was important? There should be more reasoning than just "new reset.css is more modern"?
Comment #5
ry5n CreditAttribution: ry5n commentedI did some research in the issue queue and found #723392: Tame seven's reset.css: tame Seven's reset. I agree with the assessment that Seven's existing reset is overkill, but either way I wanted some resolution there before making theme changes for #1510532: [META] Implement the new create content page design . Consensus was that normalize.css would be considered a good replacement, so I posted that patch there and built the Creat Content patch on top. It was introduced around #238 in #1510532: [META] Implement the new create content page design.
Comment #6
kika CreditAttribution: kika commentedSo, @ry5n's patch at #723392: Tame seven's reset.css is duplicate of the patch in this issue? If so, one of them should be marked as dupe.
Note: I edited last comment to be more readable.
Comment #7
mrfelton CreditAttribution: mrfelton commentedOne of my problem with the old css reset from Seven is that it includes stuff that resets styles added by Drupal's system modules. A reset stylesheet should come first, and should only need to reset browser styles.
Comment #8
kika CreditAttribution: kika commentedSo what to do with this issue? It's not super-hard requirement for #1751754: Implement new form style for Seven, based on blueprint mockups. but it will remove some of the artifacts in font sizing http://drupal.org/node/1510532#comment-6381930
Any CSS cleanup people to weight in and give an opinion whenever we need a modern reset.css in core?
Comment #9
ry5n CreditAttribution: ry5n commentedThis is really a duplicate of http://drupal.org/node/723392. In my haste I included an updated reset as part of the create content page patch, which got spun off here. I think we should move back over to the original issue. Just posted an updated version of the latest patch there plus updates from normalize 2.0.1 and screenshots for testing.