inconsistent theming for arrow buttons

[ben hockey]

when number spinner and select are focused, the arrow buttons are a shade of blue but filtering select, combo box, date text box, time text box (are there more?) are a shade of gray.

fwiw, i like the blue.

[bill]

Blue sounds good to me, do you have a patch?

[bill]

I like the blue too, but it turns out that's the hover color. In other words, when NumberSpinner or Select is focused, it's as though those arrow buttons are being hovered even though they aren't. For NumberSpinner, from this rule:

.claro .dijitSpinnerFocused .dijitArrowButton {
	background-color: @arrowbutton-hovered-background-color;
}

You could make a stretch argument that it makes sense semantically, since the user is poised to effectively click those arrow buttons by pressing the keyboard's up/down arrow keys.

I'll assume that was the intended behavior for all widgets, and update ComboBox etc. to match. It looks nicer aesthetically, except for a ComboBox in an error state, where the blue button clashes with the red border.

Note though that I'm just talking about the arrow button's background color. The color of the arrow itself still changes based on actual hover effect.

[bill]

In [29915]:

Make ComboBox etc. more consistent with NumberSpinner and Select, by making the arrow background turn to it's hover color whenever the ComboBox is focused. Fixes #14923 !strict.