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

[BUG] some scss functions no longer work #808

Open
3 tasks done
Abaan404 opened this issue Jun 21, 2023 · 1 comment · May be fixed by #809
Open
3 tasks done

[BUG] some scss functions no longer work #808

Abaan404 opened this issue Jun 21, 2023 · 1 comment · May be fixed by #809
Labels
bug Something isn't working

Comments

@Abaan404
Copy link

Checklist before submitting an issue

  • I have searched through the existing closed and open issues for eww and made sure this is not a duplicate
  • I have specifically verified that this bug is not a common user error
  • I am providing as much relevant information as I am able to in this bug report (Minimal config to reproduce the issue for example, if applicable)

Description of the bug

After updating eww to the latest commit (coming from the release branch) I noticed it no longer recognized common scss functions (notably string.unique-id() and possibly others I haven't checked).
After some digging I found that removing the default-features flag from grass in Cargo.toml fixes this issue

Seems like this change was made for faster build times and should have no noticeable impact however I lack much knowledge of rust and this project so unsure if it has any unintended issues.

Reproducing the issue

use the following scss code

@use "sass:string";

$unique: string.unique-id();

and it will not execute the function

Expected behaviour

it should've ran the function as expected

Additional context

No response

@Abaan404 Abaan404 added the bug Something isn't working label Jun 21, 2023
@Abaan404 Abaan404 linked a pull request Jun 21, 2023 that will close this issue
4 tasks
@FieldofClay
Copy link

I have also come across this issue, except for the math.random() function. Confirming that removing default-features from grass resolves this.

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.

2 participants