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

[WIP][COMPARISON] upgrade stylis #747

Draft
wants to merge 2 commits into
base: alpha
Choose a base branch
from
Draft

Conversation

huozhi
Copy link
Member

@huozhi huozhi commented Sep 6, 2021

This is a draft PR to check the effects of upgrading to latest stylist

  • Understand reason of duplicated class names generated
  • Disabling nesting

@huozhi huozhi changed the base branch from master to alpha September 6, 2021 13:38
@huozhi huozhi changed the title upgrade stylis [WIP][COMPARISON] upgrade stylis Sep 6, 2021
@ywroh
Copy link

ywroh commented Jan 9, 2022

The reason that the duplicate class name is created is that the element was added redundantly in the namespace, which is stylis middleware.

I modified this content and made a pull request to the stylis repository as follows.

thysultan/stylis#282

@huozhi
Copy link
Member Author

huozhi commented Jan 9, 2022

@ywroh looks like it's by designed (thysultan/stylis#101)

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.

None yet

2 participants