Skip to content

Fe/feature/ri 7233 key details on web and full screen do not expand to the width of the container and do not have space between each other #4762

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

Conversation

KIvanow
Copy link
Collaborator

@KIvanow KIvanow commented Jul 28, 2025

Moved sass styling to a styled component.
Added spacing between the inputs on the various add keys screens.
Increased space before the remove icon on the various key screens.
Added space beneath the first row of the various keys screens.
Expanded the keys screen to fit the available space.

KIvanow added 3 commits July 28, 2025 09:35
…th of the container and do not have space between each other - fixed spacing between name and ttl, and below the same row
…th of the container and do not have space between each other - fixed spacing between different input components
…th of the container and do not have space between each other - fixed styles to stretch from one end to the other
Copy link
Contributor

Code Coverage - Frontend unit tests

St.
Category Percentage Covered / Total
🟢 Statements 81.46% 18830/23115
🟡 Branches 66.93% 8229/12295
🟡 Functions 75.18% 4916/6539
🟢 Lines 81.87% 18431/22512

Test suite run success

4829 tests passing in 632 suites.

Report generated by 🧪jest coverage report action from 90639ba

@KIvanow KIvanow merged commit 7d34422 into fe/feature/RI-7039-replace-eui Jul 29, 2025
17 checks passed
@KIvanow KIvanow deleted the fe/feature/RI-7233---key-details-on-web-and-full-screen-do-not-expand-to-the-width-of-the-container-and-do-not-have-space-between-each-other branch July 29, 2025 11:05
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