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

Replace calls to Form::checkbox pt1 #16213

Merged
merged 4 commits into from
Feb 10, 2025

Conversation

marcusmoore
Copy link
Collaborator

This PR replaces some calls to Form::checkbox with inline html on the following pages:

Copy link

what-the-diff bot commented Feb 7, 2025

PR Summary

  • Updated Checkbox Input Method
    The checkbox in various sections of the application has been switched from using a specific method Form::checkbox to using a more standard HTML component, <input>. This may enhance accessibility and compatibility with various browsers.

  • Added Control Directives for Checkboxes
    New directives @checked and @disabled were added to checkboxes in the user profile section for enhanced control according to the user's access rights and previous input values. This should improve user experience by providing immediate visual feedback.

  • Enhanced Checkbox Label Style and Structure
    Using blade directives, the visual layouts and styles of checkbox labels in the user profile section have been improved. This could increase readability and make it more user-friendly.

@snipe snipe merged commit 4ae2126 into snipe:develop Feb 10, 2025
9 checks passed
@marcusmoore marcusmoore deleted the chore/migrate-checkbox-helpers-p1 branch February 10, 2025 23:22
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.

2 participants