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

Deprecation Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0. #639

Closed
arturu opened this issue Feb 12, 2022 · 8 comments · Fixed by #662
Assignees
Labels
select Issue e PR relative al componente "select"

Comments

@arturu
Copy link
Contributor

arturu commented Feb 12, 2022

Comportamento attuale

Durante la compilazione viene restituito questo warning

Deprecation Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0.

Possibili soluzioni

  1. individuare dove viene usato l'operatore della divisione / fuori dalla funzione calc()
  2. usare calc() presumo, o qualcosa di simile.

Contesto

sass: ^1.49.7
webpack: ^5.68.0

@arturu
Copy link
Contributor Author

arturu commented Feb 12, 2022

Trovata la linea di codice, è nella dipendenza bootstrap-select

https://github.com/snapappointments/bootstrap-select/blob/6c4c75f61e69ca54d5ab557a2b90ffdccf1c63cd/sass/bootstrap-select.scss#L26

$amnt: $amnt / 100; // convert to percentage if int

nella nuova versione (che è in beta) hanno fixato il problema.

@astagi
Copy link
Member

astagi commented Feb 18, 2022

Grazie @arturu attendiamo un loro rilascio allora, vedo stanno lavorando sul branch dev anche a dei fix sulla ricerca, farò dei test, magari risolvono anche la #619

@astagi astagi added the select Issue e PR relative al componente "select" label Feb 25, 2022
@StefanoVollono
Copy link

Ho fixato da poco lo stesso problema su un mio progetto personale, migrando da node-sass a sass (dart-sass) e si, vi confermo che con la versione Dart Sass 2.0.0 verranno deprecate le divisioni al di fuori del calc. Qui per maggiori dettagli: https://sass-lang.com/documentation/breaking-changes/slash-div.

@astagi
Copy link
Member

astagi commented Mar 31, 2022

@StefanoVollono @arturu un possibile fix potrebbe essere portare il loro scss all'interno del progetto e manutenere il file all'interno.

@astagi astagi linked a pull request Mar 31, 2022 that will close this issue
5 tasks
@astagi
Copy link
Member

astagi commented Mar 31, 2022

ho fatto una PR al volo, sembra funzionare :) ditemi anche voi!

@StefanoVollono
Copy link

@astagi ho visto la PR, è un peccato dover includere un intero css all'interno della codebase per modificare una sola riga di una funzione, però al momento é l'unica soluzione se vogliamo eliminare quel warning in console. Almeno finché non pubblicheranno la versione che fixa quel problema e possiamo ripristinare il vecchio import dalla node_modules.

@astagi astagi self-assigned this Mar 31, 2022
@astagi
Copy link
Member

astagi commented Mar 31, 2022

Esatto @StefanoVollono unica strada al momento :(

@arturu
Copy link
Contributor Author

arturu commented Apr 1, 2022

Sì, mi sa che è l'unica soluzione, visto anche che ci vorrà un po' di tempo per la versione 2.0 di bootstrap-select (che porta dentro il fix) snapappointments/bootstrap-select#2228

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
select Issue e PR relative al componente "select"
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants