Skip to content

fix(web): prevent table overflow issue causing hidden actions column#2058

Merged
dgdavid merged 3 commits intomasterfrom
fix-gh-2044
Feb 24, 2025
Merged

fix(web): prevent table overflow issue causing hidden actions column#2058
dgdavid merged 3 commits intomasterfrom
fix-gh-2044

Conversation

@dgdavid
Copy link
Contributor

@dgdavid dgdavid commented Feb 24, 2025

Problem

In some tables content is overflowing, resulting in "hidden" actions column. It's the case of Wired network Wired and storage iSCSI Initiator sections

Solution

Adjust the table layout to prevent column overflow by displaying connection IPs in a stack and applying column modifiers in the iSCSI Initiator section to ensure proper rendering.

Screenshots

Before After
Screen Shot 2025-02-24 at 12 55 06 Screen Shot 2025-02-24 at 13 01 27
Screen Shot 2025-02-24 at 12 55 23 Screen Shot 2025-02-24 at 12 57 16

Fixes #2044

@dgdavid dgdavid changed the title Fix gh 2044 fix(web): prevent table overflow issue causing hidden actions column Feb 24, 2025
Previously, connection IP addresses were displayed as a single long,
comma-separated list. Apart from adding a column modifiter, this update
renders them in a stack without gutters to prevent uncontrolled column
width, ensuring that other columns within the table are not "hidden" due
to the table overflowing beyond the section holding it.
Updated column modifiers to prevent content from overflowing the section
containing it.
@dgdavid dgdavid merged commit ea85584 into master Feb 24, 2025
6 checks passed
@dgdavid dgdavid deleted the fix-gh-2044 branch February 24, 2025 13:57
dgdavid added a commit that referenced this pull request Feb 24, 2025
## Problem

The network form wasn't showing errors when the request failed, due to
the incorrect order of the `.then()` and `.catch()` promise methods.

## Solution

Fixed the order of these methods and render request error message at the
top of the form to inform users when something goes wrong during the
request.

## Notes

This is, of course, far for being perfect. It's a workaround to avoid
users both, navigating back without changes and still in the form
without noticing something was wrong.

Detected while fixing #2058
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 2025
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.

Network Settings: The option to edit an already configured network is hidden (smaller screen)

2 participants