Opened 13 years ago
Closed 10 years ago
#22965 closed enhancement (fixed)
Change Recent Comments Number Input to type="number"
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.4 | Priority: | normal |
| Severity: | normal | Version: | 3.5 |
| Component: | Comments | Keywords: | has-patch |
| Focuses: | ui, administration | Cc: |
Description
In order to be consistent with the rest of the dashboard, the "Number of comments to show:" input field in the Recent Comments widget should be changed to type="number" in order to add the up/down arrows.
Attachments (5)
Change History (18)
#1
@
13 years ago
- Component changed from General to Administration
- Milestone changed from Awaiting Review to 3.6
#4
@
13 years ago
Also to be consistent, these should probably have min and step attributes specified - 1 for both, seems like. 0 on the Dashboard tells you there are no comments yet, and 0 in the widget just gets you the default of 5.
@
13 years ago
Updates type to "number" and adds step and min attributes. Changes class to small-text
#5
@
13 years ago
.small-text on the widgets one makes the input just wide enough to push to a second line, at least on OSX in English. Not the end of the world, but wondering if maybe we should create a .tiny-text class? Would be something like 22965.diff.
#6
@
13 years ago
I'm all for a new .tiny-text class. The small-text class has always seemed just a little too large for input fields that will never (in all sane applications) go over two digits.
#7
@
13 years ago
2 quick comments on 22965.diff:
1) Having the input push to a new line doesn't seem worth fixing to me, as the title field just above wraps to a new line as well. It looks almost more uniform to me on a new line. Also, even with a new .tiny-text class, won't it still likely wrap for some translations?
2) The padding for the number selector looks off (not sure we can do anything about that):

Patch for both dashboard recent comments and main widget