fix: table app count in app group listing css#3075
fix: table app count in app group listing css#3075Elessar1802 wants to merge 1 commit intodevelopfrom
Conversation
|
Some linked issues are invalid. Please update the issue links:\nIssue # in is not found or invalid (HTTP }404).\n |
|
There was a problem hiding this comment.
Pull request overview
This PR adjusts the layout styling for the “application count” cell in the application group environment listing table to correct its alignment/appearance.
Changes:
- Add
flex leftclasses to the app count cell wrapper to align the count text consistently with other table cells.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| <div> | ||
| <div className="flex left"> | ||
| {count} | ||
| {count === 0 || count === 1 ? GROUP_LIST_HEADER.APPLICATION : GROUP_LIST_HEADER.APPLICATIONS} |
There was a problem hiding this comment.
The singular/plural logic will render "0 Application" for a zero count due to count === 0 || count === 1. Typically 0 should use the plural label, so this should only be singular when count === 1.
| {count === 0 || count === 1 ? GROUP_LIST_HEADER.APPLICATION : GROUP_LIST_HEADER.APPLICATIONS} | |
| {count === 1 ? GROUP_LIST_HEADER.APPLICATION : GROUP_LIST_HEADER.APPLICATIONS} |



Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Checklist: