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

🔴 LESS files should have .less extension #33

Open
igo opened this issue Dec 2, 2017 · 3 comments
Open

🔴 LESS files should have .less extension #33

igo opened this issue Dec 2, 2017 · 3 comments

Comments

@igo
Copy link

igo commented Dec 2, 2017

All LESS files should have .less file extension. It not only best practice to do so but also there are issues with webpack loaders which do not recognise and do not process correctly .variables and .overrides files. theme.config file should have .less extension as well.
Furthermore when including other less files, path should have .less extension (see for example semantic.less imports)

@davidpanzarella
Copy link

What's the latest on this? Anyone on the Semantic UI Team going to fix this so we can use it with LESS 3?

@sciyoshi
Copy link

sciyoshi commented Jun 7, 2018

I've opened a PR that fixes the issues with Less 3: #44

@thomasdarvik
Copy link

What is the reason they exclude the .less extension on .variables and .overrides? In my build pipeline I use a less-loader that looks for changes in .less-files. The recompile is not triggered, since the loader cannot see the changes, as the files are not .less, and I have to do this manually. Linting is also really weird, since tools like Visual Studio (2017/Code) will not view the file as .less.

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

4 participants