Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add pseudo-elements to universal selector in print media query #1585

Closed
wants to merge 1 commit into from
Closed

Add pseudo-elements to universal selector in print media query #1585

wants to merge 1 commit into from

Conversation

grantgeard
Copy link
Contributor

The pseudo-elements :before and :after are not targeted by the universal * selector. When the page is printed the CSS properties of any pseudo-elements will only be overridden if they are being inherited from their selector. Any styles that have been defined directly on a pseudo-element are unaffected.

@drublic
Copy link
Member

drublic commented Aug 27, 2014

Thanks for the improvement. I think this is worth including within our print stylesheet.
Here is a short test case: http://codepen.io/drublic/full/vHzpF
In general Chrome does not print backgrounds by default and there is an option for that in Firefox.

@arthurvr
Copy link
Member

👍

@alrra alrra added this to the 5.0.0 milestone Aug 27, 2014
@alrra
Copy link
Member

alrra commented Aug 27, 2014

@grantgeard thanks for the PR!

Can you also:

If not, don't worry, we'll take care of that! :)

@drublic
Copy link
Member

drublic commented Aug 27, 2014

Thanks @grantgeard, great contribution! I've updated the changes and merged this in.

JimBobSquarePants added a commit to ResponsiveBP/Responsive that referenced this pull request Feb 5, 2015
- Fixes issues #64 #66
- Add pseudo-elements to universal selector in print media query
(h5bp/html5-boilerplate#1585)
- Remove gap below iframes
(h5bp/html5-boilerplate#1613)
- Add vertical centering for svg
(h5bp/html5-boilerplate#1453)
- simplify `:focus` selector
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants