Skip to content

fix(plugin-id/ui): harmonize actions column layout across Identity ListViews#30

Merged
fdaugan merged 1 commit into
feature/vuejsfrom
norman/fix-list-actions-column-layout
May 22, 2026
Merged

fix(plugin-id/ui): harmonize actions column layout across Identity ListViews#30
fdaugan merged 1 commit into
feature/vuejsfrom
norman/fix-list-actions-column-layout

Conversation

@Terracosmos
Copy link
Copy Markdown
Collaborator

Harmonize the actions column on the three Identity ListViews
(User / Group / Company).

Finding

The review item mentioned the user list, but investigation showed
UserListView and CompanyListView were already aligned at width 100px —
GroupListView was the outlier at 120px.

However, 100px combined with align: 'end' pushed the two action
icons (edit + delete) against the right edge where they wrapped onto
two lines on any viewport narrower than very wide — inconsistent AND
not usable.

Fix

Apply a single shared config to the actions column on all three
ListViews:

  • width: '100px' → '120px' (room for the two icons side by side)
  • align: 'end' → 'center' (header and buttons centered in the column)

The three ListViews now share an identical actions column config.

Tested

Visual check with the three pages side by side:

  • Identical column width
  • edit + delete icons on a single horizontal line, centered
  • Consistent row height, no vertical drift
  • No regression on other columns or on edit/delete clicks

Files

3 files, +3/-3 — User/Group/CompanyListView.vue.

…s ListViews

The actions column on the three Identity ListViews (User / Group /
Company) showed two inconsistencies:
- GroupListView was 120px, the other two were 100px
- align="end" + a tight width pushed the two icon buttons (edit +
  delete) against the right edge, where they wrapped onto two lines
  on narrower viewports

Aligning all three on width="120px" + align="center" so:
- the two icon buttons sit horizontally on a single line at any
  reasonable viewport width
- the column header and the buttons are centered in the cell

Branch name is historical (originated as a single-file fix on
GroupListView); the scope was extended after Norman's review to
cover the three lists together for visual consistency.
@Terracosmos Terracosmos requested a review from fdaugan May 20, 2026 12:06
@sonarqubecloud
Copy link
Copy Markdown

@sonarqubecloud
Copy link
Copy Markdown

@fdaugan fdaugan merged commit 008e7f5 into feature/vuejs May 22, 2026
4 of 5 checks passed
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.

2 participants