Skip to content

[Bug]: Unable to let Dart Sass compile "dist/css/tom-select.css" #897

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

Open
sato11 opened this issue Mar 6, 2025 · 1 comment · May be fixed by #898
Open

[Bug]: Unable to let Dart Sass compile "dist/css/tom-select.css" #897

sato11 opened this issue Mar 6, 2025 · 1 comment · May be fixed by #898
Labels
bug Something isn't working

Comments

@sato11
Copy link

sato11 commented Mar 6, 2025

Bug description

I found it unexpected that sass panics when referencing dist/css/tom-select.css:

% sass --version
1.83.4
% sass dist/css/tom-select.css > /dev/null    
Error: 6px and 0 are incompatible.
   ╷
29 │   padding: calc(8px - 2px - 0) 8px calc(8px - 2px - 3px - 0);
   │                 ^^^^^^^^^^^^^
   ╵
  dist/css/tom-select.css 29:17  root stylesheet

On the other hand, using dist/css/tom-select.min.css seems alright:

% sass dist/css/tom-select.min.css > /dev/null 
# OK!

In other words, I can simply ignore it and work around by sticking to dist/css/tom-select.min.css. Just thought it'd be nicer if a set of minor trouble and its fix becomes visible in case someone comes across the same trouble.

Expected behavior

Using dist/css/tom-select.css causes no trouble, hopefully.

sass dist/css/tom-select.css > /dev/null    

I am not quite familiar with today's front-end technologies and the subtlety was hard to conceive when the behaviors differed between "*.min.css" and "*.css".

Or if there is something like a well known reason behind it, let me know and I'm happy to learn from it.

Steps to reproduce

This should reproduce it. Thanks!

git clone git@github.com:orchidjs/tom-select.git
npm install
npm run build
sass dist/css/tom-select.css > /dev/null    

Additional context

  • OS: macOS Sonoma 14.6
  • Browser: N/A
  • Version: reproduces on the master branch
  • Device: N/A
@sato11 sato11 added the bug Something isn't working label Mar 6, 2025
@sato11
Copy link
Author

sato11 commented Mar 7, 2025

It seems like this had been fixed once at #398 in v2.1.0 and then regressed in #534 (24d3a1f) in v2.2.3

@sato11 sato11 linked a pull request Mar 7, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant