Use number input in SettingsForm

Created on 18 March 2019, over 5 years ago
Updated 20 May 2024, 4 months ago

The session_limit_max field currently has a '#maxlength' => 3 to limit input to 3 digits, but the field size is limited to '#size' => 2. This makes it impossible to read the actual value from the input field without selecting / copying the value when more than 2 digits are used.

Suggests to remove the size limit and even the max length, but it also makes more sense to use the number input type instead.

πŸ› Bug report
Status

Fixed

Version

2.0

Component

Code

Created by

πŸ‡§πŸ‡ͺBelgium baikho Antwerp, BE

Live updates comments and jobs are added and updated live.
  • Needs usability review

    Used to alert the usability topic maintainer(s) that an issue significantly affects (or has the potential to affect) the usability of Drupal, and their signoff is needed. When adding this tag, make it easy to review the issue. Make sure the issue summary describes the problem and the proposed solution. Screenshots usually help a lot! To get sign-off on issues with the "Needs usability review" tag, post about them in the #ux channel on Drupal Slack, and/or attend a UX meeting to demo the patch and get direct feedback from designers/UX folks/product management on next steps. If an issue represents a significant new feature, UI change, or change to the general "user experience" of Drupal, use Needs product manager review instead. See the scope of responsibilities for product managers.

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

Production build 0.71.5 2024