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

Set modal focus to modal-name field #13677

Merged
merged 1 commit into from
Sep 28, 2023
Merged

Set modal focus to modal-name field #13677

merged 1 commit into from
Sep 28, 2023

Conversation

snipe
Copy link
Owner

@snipe snipe commented Sep 28, 2023

This sets the focus of the "new" modals to be on the modal-name field (and modal-first_name for users). Just a nice UI enhancement.

Signed-off-by: snipe <snipe@snipe.net>
@shortcut-integration
Copy link

This pull request has been linked to Shortcut Story #14880: Pop ups for text entry.

@snipe snipe merged commit 3b4004a into develop Sep 28, 2023
@snipe snipe deleted the feature/sc-14880 branch September 28, 2023 18:18
@what-the-diff
Copy link

what-the-diff bot commented Sep 28, 2023

PR Summary

  • Improved User Interaction in app.js, all.js, and snipeit_modals.js

    • We've added indications that focus should be automatically set to the 'name' field in various parts of the application. This helps the user by directing them to the next step in the process without needing to manually click on the field.
  • Enhanced Form Elements in app.js, all.js, and snipeit_modals.js

    • Instructions have been added to reapply the Select2 plugin to appropriate form elements after the page has loaded. This makes sure that these form items function as expected, providing a cleaner user experience.
  • Updated mix-manifest.json

    • The version IDs for app.js and all.js have received updated IDs. This ensures that all users get the most recent version of our scripts, enhancing reliability and consistency of the user interface.
  • User Interface Tweaks in user.blade.php

    • We've added a script to set the focus on the 'first_name' field specifically in the users' section. This addition will help save time for users entering new information and add to overall usability enhancements.

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.

1 participant