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

fix: stroke-opacity, stroke-width, stroke-linecap, fill-rule #20

Closed
wants to merge 0 commits into from

Conversation

khakimov
Copy link
Contributor

updated different naming convention for JSX to avoid errors in the console.

@UdaraJay
Copy link
Owner

this should be good if you can remove the changes to any .scss files - the properties in css are defined correctly.

@khakimov
Copy link
Contributor Author

this should be good if you can remove the changes to any .scss files - the properties in css are defined correctly.

ah, my bad or to be precise - its my prettier. could you share share or commit your config if you have, so I will be on the same page with you.

@UdaraJay
Copy link
Owner

I will commit it into the repo 👌

@UdaraJay
Copy link
Owner

@khakimov in the mean time I think the only pertinent one here is 2 spaces per tab vs 1. If you can set that ( "prettier.tabWidth": 2) manually for this one it should be good while I figure out a decent prettier config for the project.

@khakimov khakimov closed this Nov 13, 2023
@khakimov khakimov deleted the fix/link-previews branch November 13, 2023 08:10
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