mirror of
https://github.com/louislam/uptime-kuma.git
synced 2026-03-02 22:57:00 -05:00
Disable scroll wheel value change on panel #1181
Labels
No labels
A:accessibility
A:api
A:cert-expiry
A:core
A:dashboard
A:deployment
A:documentation
A:domain expiry
A:incidents
A:maintenance
A:metrics
A:monitor
A:notifications
A:reports
A:settings
A:status-page
A:ui/ux
A:user-management
Stale
ai-slop
blocked
blocked-upstream
bug
cannot-reproduce
dependencies
discussion
duplicate
feature-request
feature-request
good first issue
hacktoberfest
help
help wanted
house keeping
invalid
invalid-format
invalid-format
question
releaseblocker 🚨
security
spam
type:enhance-existing
type:new
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/uptime-kuma#1181
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Balls0fSteel on GitHub (Jun 20, 2022).
⚠️ Please verify that this feature request has NOT been suggested before.
🏷️ Feature Request Type
UI Feature
🔖 Feature description
This should be super simple to implement, test, deploy.
When you ADD:
When you EDIT:
✔️ Solution
Disable this behavior.
While moving around with the mouse, it's extremely simple to change these values.
According to this, you can use "tel" instead of "number" input type:
https://stackoverflow.com/questions/9712295/disable-scrolling-on-input-type-number
I checked it, worked for me (disabled scroll) but of course, more testing is needed I believe.
❓ Alternatives
No response
📝 Additional Context
No response
@CommanderStorm commented on GitHub (Dec 7, 2023):
Why is not allowing users to use the scroll wheel on these inputs important/better?
@OpenSpaghettiSauce commented on GitHub (Dec 7, 2023):
Because when you scroll through the settings, you can accidentally change the port number or other number resulting in a headache. That's about it.
@CommanderStorm commented on GitHub (Dec 7, 2023):
There appears to be some literature out there supporting the use of
<input type="text" inputmode="numeric" pattern="[0-9]*">for integersIdk if this is a fix worth implementing.
I am going to leave this open for post-
v2.0discussion@chakflying commented on GitHub (Dec 7, 2023):
On the other hand, the ability to quickly increment/decrement the value does provide convenience for e.g. increasing the number of retries by 1 easily.
We don't really have the user research to say which method is generally better for people.
@OpenSpaghettiSauce commented on GitHub (Dec 7, 2023):
Yeah up to you guys. I just often go through monitors to double-check and it's very easy to mess something up due to this. I remember Qt also had fields like this and qBittorrent would change values as I scrolled settings.
I don't know, you guys know best. Thanks for your hard work btw!