Skip to content

Update forms for consistency and improved experience - #1167

Merged
AndyRae merged 23 commits into
masterfrom
feat/1075/forms/dataset
Aug 1, 2025
Merged

Update forms for consistency and improved experience#1167
AndyRae merged 23 commits into
masterfrom
feat/1075/forms/dataset

Conversation

@AndyRae

@AndyRae AndyRae commented Jul 31, 2025

Copy link
Copy Markdown
Member
♻️ Refactor
✨ Feature

PR Description

Updates the apps forms for a consistent UI and improved user experience.

The forms have been rebuilt using the shadcn components that have been reduced back to their original styles, with a focus on providing consistent forms experience across the entire application.

This is a continuation made possible by #1097.

I tried to keep this limited to just the forms, but it does creep into other components slightly.. so this work continued in #1166

Related Issues or other material

Related #1075

Screenshots, example outputs/behaviour etc.

Screenshot 2025-07-31 at 20 36 33 Screenshot 2025-07-31 at 20 28 43 Screenshot 2025-07-31 at 20 28 50 Screenshot 2025-07-31 at 20 29 04 Screenshot 2025-07-31 at 20 29 30

@AndyRae
AndyRae merged commit f215a47 into master Aug 1, 2025
16 checks passed
@AndyRae
AndyRae deleted the feat/1075/forms/dataset branch August 1, 2025 10:49

@AndrewThien AndrewThien left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should have given comments on this earlier. But I just wondered if it would be nicer when the upload SR form is in the middle of the page? Cause this page only has this form and the menu bar.

@AndyRae

AndyRae commented Aug 1, 2025

Copy link
Copy Markdown
Member Author

Should have given comments on this earlier. But I just wondered if it would be nicer when the upload SR form is in the middle of the page? Cause this page only has this form and the menu bar.

Really happy for more changes - on this one specifically, interestingly I had the same thought. The reason I didn't is the Scan Reports breadcrumb at the top, which felt suddenly in the middle of nowhere. I then figured, kind of odd to have one piece of centre aligned when nothing else is.

But yeah - I think it could work with a bit of thought

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants