Skip to content

Comments

fix: webapp list page UI (HEXA-1455)#1627

Merged
yolanfery merged 3 commits intomainfrom
HEXA-1455-improve-the-web-apps-list-page
Feb 20, 2026
Merged

fix: webapp list page UI (HEXA-1455)#1627
yolanfery merged 3 commits intomainfrom
HEXA-1455-improve-the-web-apps-list-page

Conversation

@yolanfery
Copy link
Contributor

Improve the Webapp list UI. Changes are actually coming from #1516 but PR is too big I want to take a more incremental approach

Screenshots / screencast

Screenshot 2026-02-19 at 09 54 55

@yolanfery yolanfery marked this pull request as ready for review February 19, 2026 09:00
@yolanfery yolanfery requested a review from bramj February 19, 2026 09:01
{getWebappTypeLabel(item.type)}
</div>
)}
</BaseColumn>
Copy link
Contributor

Choose a reason for hiding this comment

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

Obviously we're nitpicking, but this is the sort of PR where we should 😛

I see there's a very slight vertical misalignment here:

Image

On the root@openhexa I added a margin: auto, and that seems to align it nicely.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I agree its good to nitpick here, even though my poor eyes can't catch this 😂

Anyway I found an issue with the main Table component, h-full on rows doesn't work as intended, I removed it, this should fix the issue globally

@yolanfery yolanfery merged commit 16983d9 into main Feb 20, 2026
7 checks passed
@yolanfery yolanfery deleted the HEXA-1455-improve-the-web-apps-list-page branch February 20, 2026 15:00
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