Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced calls to Form::radio helper on bulk asset model edit page #16180

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

marcusmoore
Copy link
Collaborator

This PR replaces calls to Form::radio with plain html on the bulk asset model edit page.


Related: #16176, #16178, and #16179

Copy link

what-the-diff bot commented Feb 5, 2025

PR Summary

  • Changes to Form Usage for Radio Buttons
    The code that previously used the Form facade, a complex way of handling radio buttons, has now been replaced. Instead, we're using standard HTML input elements. This makes it simpler to understand the code and easier to maintain.

  • Retention of Functionality
    These changes do not affect how things actually work in our software. The attributes for each radio button like name, id, value, and aria-label have been appropriately set to ensure they function as before.

  • Default Radio Button Selection
    The default radio button selected when the page is loaded is now requestable_nochange. This alteration improves user interface consistency, creating a better user experience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant