12.0d Tahoe FormItem hover issues (broken styling, missing styling and problem on enter slow/fast)

[Blama]

Hi Isomorphic,

please see this sample (SNAPSHOT_v12.0d_2018-01-07):

1. All the last three items (Date Range, Mini Date Range, Relative Date) move on hover if you hover either the item or the hint text for the selected date.
2. Stacked Spinner and Unstacked spinner do not have a hover-style, yet. I assume they should have after your post here, but I don't know if that's already known or in progress, as it is a "d" build of course.
3. Same for Silder and Radio Group, although I don't know how a good hover would look here, so this might be on purpose.
4. Select Multiple (Grid) does not have a hover-styling
5. Select Multiple (Grid) moves on hover

Actually, #5 might be a bigger problem, as it seems that hovering the border does stop the item from being hover-styled, while it is hover-styled just before the border. This is also true for normal SelectItems.
Additionally, entering the "Select Multiple (Grid)" slowly (only slowly, not when fast), also stops the styling from being applied. This is similar to this issue I reported a long time ago, which also only happens for slow mouse movements.

SelectItem on border-hover:



SelectMultiple (Grid) on hover and on entering slow/fast:



Best regards
Blama

[Blama]

Addition:

The "SelectItem on border-hover"-issue where the styling is removed for a short time also exists in v11.1p_2018-01-07 and also exists if you hover the arrow that opens the pickList:

Best regards
Blama

[Isomorphic]

Aside from the the quite tricky issue of the 1px gap around selects and their icons showing the wrong border, all of these others should be addressed for tomorrow's builds (in 12.0 only - they should all be 12.0-only problems)

[Blama]

Hi Isomorphic,

I can see that

• 1/4/5 are fixed
• 2 is still open
• 3 is still open, but might be on purpose
• The 1px gap around selects is very minor IMHO

Best regards
Blama

[Isomorphic]

Please retest #2 in tomorrow's build (it works as expected for us in local builds, so possibly the changes didn't make it into the nightlies).

On #3, we're not sure what styles might be applied for these, but we'll take a look

[Blama]

Hi Isomorphic,

#2: Using online showcase SNAPSHOT_v12.0d_2018-01-11 with cache refresh, Stacked Spinner/Unstacked Spinner still do not have a hover-effect.

Best regards
Blama

[Blama]

Hi Isomorphic,

the spinner issue might not be styling related, but a JS error. on hover, this is the error in the developer console:

14:23:02.207:MMV6:WARN:Log:TypeError: this._stackedMode is not a function
Stack from error.stack:
    _3.getUpdateControlOnOver(<no args: exited>) on [SpinnerItem ID:isc_SpinnerItem_0 name:stackedSpinner] @ [no file]:1:21141
    FormItem._getCellStyle(<no args: exited>) on [SpinnerItem ID:isc_SpinnerItem_0 name:stackedSpinner] @ ISC_Forms.js:859:113
    SpinnerItem.getControlStyle(<no args: exited>) on [SpinnerItem ID:isc_SpinnerItem_0 name:stackedSpinner] @ ISC_Forms.js:2807:224
    FormItem._updatePickerCellAndControlTableState(<no args: exited>) on [SpinnerItem ID:isc_SpinnerItem_0 name:stackedSpinner] @ ISC_Forms.js:814:645
    FormItem.updateStateForRollover(<no args: exited>) on [SpinnerItem ID:isc_SpinnerItem_0 name:stackedSpinner] @ ISC_Forms.js:811:855
    FormItem.handleMouseMove(<no args: exited>) on [SpinnerItem ID:isc_SpinnerItem_0 name:stackedSpinner] @ ISC_Forms.js:1298:155
    DynamicForm._itemMouseEvent(<no args: exited>) on [DynamicForm ID:form1] @ ISC_Forms.js:564:262
    DynamicForm.handleMouseMove(<no args: exited>) on [DynamicForm ID:form1] @ ISC_Forms.js:564:595
    [c]EventHandler.bubbleEvent(<no args: exited>) on [Class EventHandler] @ ISC_Core.js:2079:89
    EventHandler.__handleMouseMove(<no args: exited>) on [Class EventHandler] @ ISC_Core.js:1904:126
    EventHandler._handleMouseMove(<no args: exited>) on [Class EventHandler] @ ISC_Core.js:1886:244
    [c]EventHandler.handleMouseMove(<no args: exited>) on [Class EventHandler] @ ISC_Core.js:1886:11
    [c]EventHandler.dispatch(_1=>[c]EventHandler.handleMouseMove(), _2=>[object MouseEvent]) on [Class EventHandler] @ ISC_Core.js:2165:108
    HTMLDocument.eval(event=>[object MouseEvent]) @ [no file]:3:123

Best regards
Blama

[Isomorphic]

Ah, we see the problem and you should find it working as expected in tomorrow's builds.

[Blama]

Hi Isomorphic,

using SNAPSHOT_v12.0d_2018-01-14, I can see all the FormItems now behave the same in the showcase sample.

Best regards
Blama