Not working on next.js project about vscode-css-modules

[clinyong]

Could you supply a reproduced demo?

[owlmafia]

@clinyong actually, I just noticed that the problem is specific to ids (when using the id attribute instead of class). Is that a known limitation?

Also, when using classes, is there a way to prevent this selection?

These issues can be reproduced with a nextjs starter project:

npx create-next-app@latest nextjs-blog --use-npm --example "https://github.com/vercel/next-learn/tree/master/basics/learn-starter"

[clinyong]

Yes, the id attribute is not supported. We can't prevent the selection because there isn't an api for extension to do this.

[owlmafia]

Would a feature request for the id make sense? And about the api, would it make sense to open an issue on the vscode repo maybe (I could do it)?

Edit: actually, id is not so important to me. I can easily replace them with classes.

[karlhorky]

I think the "duplicate definitions in Go to Definition" issue is documented over here:

• #63

As I mentioned in my last comments, someone could open an issue in the microsoft/vscode or microsoft/TypeScript repos to present the use case clearly and ask for some way for an extension (or user configuration) to be able to disable useless definition entries in Go to Definition.

[karlhorky]

For Next.js itself, you can also consider modifying the node_modules/next/types/global.d.ts file containing these types, as I mentioned in #63 (comment)

You could modify it permanently in your project using something like patch-package.

[iChenLei]

@clinyong actually, I just noticed that the problem is specific to ids (when using the id attribute instead of class). Is that a known limitation?

Also, when using classes, is there a way to prevent this selection?

These issues can be reproduced with a nextjs starter project:

npx create-next-app@latest nextjs-blog --use-npm --example "https://github.com/vercel/next-learn/tree/master/basics/learn-starter"

Duplicate #63