Skip to content

Commit 0b8c23e

Browse files
authored
feat: Change default transition to all (#5431)
## Description Originally we chose due to perf considerations to use opacity by default as a safe default, but most of the time these days we don't find All to be dangerous. ## Steps for reproduction 1. click button 2. expect xyz ## Code Review - [ ] hi @kof, I need you to do - conceptual review (architecture, feature-correctness) - detailed review (read every line) - test it on preview ## Before requesting a review - [ ] made a self-review - [ ] added inline comments where things may be not obvious (the "why", not "what") ## Before merging - [ ] tested locally and on preview environment (preview dev login: 0000) - [ ] updated [test cases](https://github.com/webstudio-is/webstudio/blob/main/apps/builder/docs/test-cases.md) document - [ ] added tests - [ ] if any new env variables are added, added them to `.env` file
1 parent 19cfd12 commit 0b8c23e

File tree

1 file changed

+1
-1
lines changed
  • apps/builder/app/builder/features/style-panel/sections/transitions

1 file changed

+1
-1
lines changed

apps/builder/app/builder/features/style-panel/sections/transitions/transitions.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,7 +123,7 @@ export const Section = () => {
123123
setIsOpen(true);
124124
addRepeatedStyleItem(
125125
styles,
126-
parseCssFragment("opacity 200ms ease 0ms normal", [
126+
parseCssFragment("all 200ms ease 0ms normal", [
127127
"transition",
128128
])
129129
);

0 commit comments

Comments
 (0)