Closed
Bug 917736
Opened 11 years ago
Closed 11 years ago
Add NumberPicker input type to PromptInput.java
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: capella, Assigned: capella)
Details
Attachments
(1 file)
84.90 KB,
image/png
|
Details |
In bug 770101 we added a |number| input type (for about:config) and briefly considered adding native NumberPicker input for Android devices that support it (android.os.Build.VERSION.SDK_INT > 10, or versions starting with Honeycomb) to PromptInput.java , but decided to tackle that here in a dedicated bug. I think the last point of contention is that due to a limitation of the NumberPicker object, we have to provide user input for a +/- toggle to enable entry of the entire range of Integer values. The widget allows scrolling through the number range 0-MAXINT, as well as tapping the edit box and manually keying values to avoid scrolling to ridiculously large numbers. wesj: I think the sticking point here is the UI presentation?
Attachment #806566 -
Flags: feedback?(wjohnston)
Comment 1•11 years ago
|
||
Yeah. I don't really like the button. I tried playing with the theme for it a bit, removing the background makes it look unclickable. I'm not sure of a great fix.
Comment 2•11 years ago
|
||
Should this just be a simple text entry, then? Seems like the number picker isn't the best way to solve this, given the +/- limitations. So maybe it's just something like ----------------------------------------------------- Enter a value for newnumber, between -3 and 300 (or whatever the limits are) [input field] Cancel | OK
Assignee | ||
Comment 3•11 years ago
|
||
Since that's pretty much what's in place now, I guess I'll just give up on the NumberPicker ... sighs ... I liked it though ;-P
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → WONTFIX
Assignee | ||
Updated•11 years ago
|
Attachment #806566 -
Flags: feedback?(wjohnston)
Updated•3 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•