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

content property block #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Heydon
Copy link

@Heydon Heydon commented Nov 15, 2020

Because you can't style pseudo-content without declaring content innit.

@Andy-set-studio
Copy link
Owner

Nice one bud! I'll add this version of the reset to a few bits I'm on over the next few weeks then get it merged if nothing terrifying happens. I'm p sure because yes, suddenly every element will have two pseudo-elements, they will be inline, so shouldn't cause issues.

@nhoizey
Copy link

nhoizey commented Nov 17, 2020

I'm wondering if this can have a rendering performance impact on documents with a lot of DOM elements, at least on low end mobile devices.

Probably very small, but might be worth testing.

@itsjavi
Copy link

itsjavi commented Mar 11, 2023

I am with @nhoizey I am not sure if this will have an impact on performance. If you inspect the DOM, all elements will have those pseudo elements when this is applied.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants