Skip to content

Commit

Permalink
Fix(web): Remove will-change from Modal to prevent high memory co…
Browse files Browse the repository at this point in the history
…nsumption

Firefox warns about potential high memory consumption when `will-change`
is used on big areas. Therefore it seems better to remove it in this case to
free up the memory for smaller tweaks in Spirit and also for other performance
optimisations our users might want to make.

https://dev.opera.com/articles/css-will-change-property/
  • Loading branch information
adamkudrna committed Jan 4, 2024
1 parent 8eefa26 commit a907e23
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion packages/web/src/scss/components/Modal/_Modal.scss
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,6 @@
@media (prefers-reduced-motion: no-preference) {
transition-property: visibility, opacity;
transition-duration: theme.$transition-duration;
will-change: visibility, opacity;

// 2.
&::before {
Expand Down

0 comments on commit a907e23

Please sign in to comment.