Skip to content

bug due to conflict of auto-computation of shadcn/ui library #437

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

Closed
xarthurx opened this issue Mar 1, 2025 · 8 comments
Closed

bug due to conflict of auto-computation of shadcn/ui library #437

xarthurx opened this issue Mar 1, 2025 · 8 comments

Comments

@xarthurx
Copy link

xarthurx commented Mar 1, 2025

When using shadcn/ui libraries, usually you'll have components with computed values:

			variant: {
				default: "bg-primary text-primary-foreground hover:bg-primary/90",
				destructive:
					"bg-destructive text-destructive-foreground hover:bg-destructive/90",
				outline:
					"border border-input bg-background hover:bg-accent hover:text-accent-foreground",
				secondary:
					"bg-secondary text-secondary-foreground hover:bg-secondary/50",
				ghost: "hover:bg-accent hover:text-accent-foreground",
				link: "text-primary underline-offset-4 hover:underline",
			},

such as the bg-primary/90 above.

Somehow this will trigger some auto-computation module from nvchad (see img below) and create notification errors:

Image

Is there a way to disable it?

@siduck siduck transferred this issue from NvChad/NvChad Mar 2, 2025
@siduck
Copy link
Member

siduck commented Mar 2, 2025

for now , disable lsp field in colorify option, in chadrc

@xarthurx
Copy link
Author

Any update on this? Usually you're respectively fast on issues, but no pressure. @siduck

@siduck
Copy link
Member

siduck commented Mar 19, 2025

Any update on this? Usually you're respectively fast on issues, but no pressure. @siduck

this month i'll be slow, i will look into this in april sorry. fow now just disable the colorify's lsp method

M.colorify = { 
          highlight = {  lspvars = false },
}

@siduck
Copy link
Member

siduck commented Apr 11, 2025

unable to reproduce

@xarthurx
Copy link
Author

unable to reproduce

I'll setup up a repo around this weekend.

@siduck
Copy link
Member

siduck commented Apr 11, 2025

unable to reproduce

I'll setup up a repo around this weekend.

i have the repo already, i installed the button package

Image

@siduck
Copy link
Member

siduck commented Apr 11, 2025

but i dont see that error, show a video!

@xarthurx
Copy link
Author

OK, somehow the issue disappeared... Perhaps something got updated during the past weeks.
All good now.

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

No branches or pull requests

2 participants