Skip to content

Conversation

@Pallavikumarimdb
Copy link
Contributor

@Pallavikumarimdb Pallavikumarimdb commented Oct 4, 2025

Community Contribution License Agreement

By creating this pull request, I grant the project maintainers an unlimited,
perpetual license to use, modify, and redistribute these contributions under any terms they
choose, including both the AGPLv3 and the Fossorial Commercial license terms. I
represent that I have the right to grant this license for all contributed content.

Description

  1. Add refresh button to all the pages.
  2. Improve target configuration table design.
image image image

How to test?

@Pallavikumarimdb Pallavikumarimdb force-pushed the fix/UI-adjustment branch 2 times, most recently from 3354328 to 64d6739 Compare October 5, 2025 19:38
@Pallavikumarimdb Pallavikumarimdb marked this pull request as ready for review October 5, 2025 19:46
@oschwartz10612 oschwartz10612 force-pushed the fix/UI-adjustment branch 2 times, most recently from be7f915 to 7f36a53 Compare October 6, 2025 00:20
@miloschwartz
Copy link
Member

image

@miloschwartz
Copy link
Member

@Pallavikumarimdb Thanks! This is looking better. I got kind of carried away and implemented an advanced toggle to hide some of the columns by default, so it's easier to parse for new users. A couple of other things I noticed which would be good to worked out before merging:

  • Instead of Dialog components we have a Credenza component which is essentially just a wrapper that is also mobile responsive. Can you check anywhere where there is a Dialog component and replace it with Credenza? The migration should be simple and you can take a look at other instances of the component to see how it's done.
  • When creating a resource via the create wizard, there is a target table at the bottom so you can quick add targets. Can you look at standardizing this component so it matches the targets input table in the resource edit settings?
  • Check for any untranslated strings in the these components and add them to the en-us.json file.

@Pallavikumarimdb Pallavikumarimdb force-pushed the fix/UI-adjustment branch 2 times, most recently from f4fb9dc to 20d0ead Compare October 13, 2025 16:20
@Pallavikumarimdb Pallavikumarimdb marked this pull request as ready for review October 13, 2025 18:56
@oschwartz10612
Copy link
Member

LGTM! Thanks so much! :}

@oschwartz10612 oschwartz10612 merged commit 65eb3e4 into fosrl:dev Oct 13, 2025
2 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.

3 participants