-
Notifications
You must be signed in to change notification settings - Fork 42
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
[Darkside] Loader CSS update #3355
Conversation
|
Storybook demo6acbd0937 | 91 komponenter | 135 stories |
Still WIP? |
No, should be good-to-go |
stroke-dashoffset: 0; | ||
stroke: var(--ac-loader-stroke, var(--a-border-default)); | ||
} | ||
&.navds-loader--3xlarge { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why nest this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Personally like to have changes in local tokens before it is used --__axc-loader-stroke-width
. Feels disorienting to have "definition -> use -> definition -> definition" in some cases. But for a short file like this it doesn't really matter
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you misunderstand. I'm wondering why you put &.navds-loader--3xlarge
etc. inside .navds-loader {
?
Do you need the selector to be .navds-loader.navds-loader--3xlarge
?
I get some weird shaking in Chrome 😅 shaky.loaders.mp4Only seems to happen when there are multiple loaders, so I don't think we need to worry about it. |
@HalvorHaugan Is this new in this PR, or the case with the old loader to? |
Only with the new darkside css in this PR |
@HalvorHaugan So strange, no changes were made to how this were handled 🤔 Does it change if you remove
|
@KenAJoh Yes, it doesn't happen if I remove |
No description provided.