Skip to content

Conversation

KrumTy
Copy link
Collaborator

@KrumTy KrumTy commented Sep 30, 2025

Old UI Before After
Screenshot 2025-09-30 at 13 08 02 Screenshot 2025-09-30 at 13 08 45 Screenshot 2025-09-30 at 13 06 47

@KrumTy KrumTy marked this pull request as ready for review September 30, 2025 10:14
Copy link
Contributor

github-actions bot commented Sep 30, 2025

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 82.11% 20018/24379
🟡 Branches 67.44% 8674/12861
🟡 Functions 76.26% 5329/6988
🟢 Lines 82.54% 19600/23746

Test suite run success

5162 tests passing in 677 suites.

Report generated by 🧪jest coverage report action from 4251d82

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should try to get rid of scss - move to single css solution - rather than adding to it

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I brought the class name and style that was removed with the previous file commit

now removed in favor of grow prop

@KrumTy KrumTy requested a review from pd-redis September 30, 2025 10:49
@KrumTy KrumTy merged commit 9c5e513 into main Oct 1, 2025
18 checks passed
@KrumTy KrumTy deleted the fe/bugfix/RI-7540/key-name-editor branch October 1, 2025 06:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants