feat: add expandable snapshot diff details #360
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…stment - Add expandable/collapsible diff details for snapshots showing state changes - Implement dynamic slider height adjustment that responds to snapshot expansion/collapse - Fix slider height reset when clear button is clicked - Fix expand icon direction (always shows ▶, CSS handles rotation to ▼ when expanded) - Add snapshot diff utility to calculate and format state changes between snapshots - Improve UI/UX with smooth height transitions and proper visual feedback Fixes: - Slider height now correctly increases when snapshots expand - Slider height now correctly decreases when snapshots collapse - Slider height resets properly when clear button is clicked - Expand icon now points correctly in both expanded and collapsed states
- Aligns with goal of showing only state changes in diff
|
@Agboolafeyikemi is attempting to deploy a commit to the Reactime Team on Vercel. A member of the Team first needs to authorize it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds expandable/collapsible diff details for snapshots and fixes slider height adjustment issues. Users can now expand snapshots to see detailed state changes, and the slider height dynamically adjusts to accommodate expanded content.
Key Features
Fixes
Technical Changes
snapshotDiff.tsutility for calculating state differencesAction.tsxto support expandable diff detailsRouteDescription.tsxwith dynamic height calculation using ResizeObserverstatelessCleaningto only track state changes (removed props preservation)_actionComponent.scssChecklist
[x ] I've followed the Contributing guidelines
[ x] I've titled my PR according to the Conventional Commits spec
I've linked an open issue
I've added tests that fail without this PR but pass with it
I've linted and tested my code
I've updated documentation (if appropriate)