Announcement

Collapse
No announcement yet.
X
  • Filter
  • Time
Clear All
new posts

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

    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:
    Click image for larger version

Name:	SelectItem on border-hover.gif
Views:	126
Size:	25.0 KB
ID:	251050


    SelectMultiple (Grid) on hover and on entering slow/fast:
    Click image for larger version

Name:	SelectMultiple (Grid) on hover.gif
Views:	72
Size:	129.7 KB
ID:	251051


    Best regards
    Blama

    #2
    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:
    Click image for larger version

Name:	SelectItem on arrow-hover.gif
Views:	72
Size:	66.3 KB
ID:	251053
    Best regards
    Blama

    Comment


      #3
      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)

      Comment


        #4
        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

        Comment


          #5
          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

          Comment


            #6
            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

            Comment


              #7
              Hi Isomorphic,

              the spinner issue might not be styling related, but a JS error. on hover, this is the error in the developer console:
              Code:
              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

              Comment


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

                Comment


                  #9
                  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

                  Comment

                  Working...
                  X