-
Notifications
You must be signed in to change notification settings - Fork 4.5k
chore: Transitions for IDE #35714
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
chore: Transitions for IDE #35714
Changes from all commits
5d6c310
84662bd
be2b59c
e05c222
7c12d18
1866d2f
5a60120
cc95473
1ebac75
9664239
6dc8f62
226160b
55c40b4
2c82b26
b0019a9
acfae93
590ffad
156296e
8fc4c7c
bff4536
9080f5c
44244a0
75866ef
a912f5c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| export const IDE_HEADER_HEIGHT = 40; |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,9 +1,11 @@ | ||
| import { easings } from "@react-spring/web"; | ||
| import type { AnimatedGridUnit } from "./types"; | ||
|
|
||
| /** Default rows config. */ | ||
| export const DEFAULT_ROWS: AnimatedGridUnit[] = ["1fr"]; | ||
|
|
||
| export const SPRING_ANIMATION_CONFIG = { | ||
| easing: easings.easeInCirc, | ||
| duration: 375, | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I've a few points to make here:
|
||
| friction: 32, | ||
| mass: 1, | ||
| tension: 205, | ||
| }; | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| export const BOTTOM_BAR_HEIGHT = 37; | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 37 seems like an odd number to use (some pun intended). Are we not following some sort of multiple of 2 or 4 rule? |
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Enhanced Clarity in Assertions
The change from
AssertElementVisibilitytoAssertElementAbsenceis a significant improvement. It makes the test assertions more precise by explicitly checking for the non-existence of the sidebar, which aligns better with the test's intent to verify the absence of certain UI elements under specific conditions.However, consider addressing the use of
cy.wait(1000). This method of waiting is generally discouraged because it can lead to flaky tests. It's better to wait for a specific condition or element to appear or disappear, which would make the tests more reliable and faster.Overall, the modifications enhance the test suite's clarity and robustness. Good job on making these changes.
Consider replacing
cy.wait(1000)with a more deterministic wait condition, such as waiting for a specific API call to complete or a UI element to become visible. This approach reduces the potential for flaky tests and improves the reliability of the test execution.