ICEfaces
  1. ICEfaces
  2. ICE-7691

Provide a CSS styleClass for customizing the ajax submit "Busy" indicator

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 3.0
    • Fix Version/s: 3.1
    • Component/s: Bridge
    • Labels:
      None
    • Environment:
      ICEfaces 3.0 RC2
    • Assignee Priority:
      P2
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.)

      Description

      When ICEfaces is waiting on a server response, it adds a style="cursor: wait;" to the body element. It would be better to add a class to the body instead. This would provide more flexibility.

        Activity

        Hide
        Ken Fyten added a comment -

        Keep in mind that for backwards compatibility, the default styling via the style-class should set the hourglass cursor.

        Show
        Ken Fyten added a comment - Keep in mind that for backwards compatibility, the default styling via the style-class should set the hourglass cursor.
        Hide
        Mircea Toma added a comment -

        Added 'busyIndicator' class that can be used to custom style the busy state of the page.

        Show
        Mircea Toma added a comment - Added 'busyIndicator' class that can be used to custom style the busy state of the page.
        Hide
        Ken Fyten added a comment -

        Should change the class name to "ice-busyIndicator" to namespace this.

        Show
        Ken Fyten added a comment - Should change the class name to "ice-busyIndicator" to namespace this.
        Hide
        Mircea Toma added a comment -

        Namespaced style class.

        Show
        Mircea Toma added a comment - Namespaced style class.
        Hide
        Ken Fyten added a comment -

        This changed seems to have broken the hourglass / busyIndicator functionality.

        Test results using the new icefaces/samples/tests/busyIndicator test:

        EE 3.0.0.GA:

        • IE 7 works
        • IE 8 works
        • IE 9 works
        • FF 13 works
        • Safari 5 no works
        • Chrome 18 no works

        3.1 (rvn #28834):
        IE 7 no works
        IE 8 no works
        IE 9 no works
        FF 13 works
        Safari 5 no works
        Chrome 18 no works

        Show
        Ken Fyten added a comment - This changed seems to have broken the hourglass / busyIndicator functionality. Test results using the new icefaces/samples/tests/busyIndicator test: EE 3.0.0.GA: IE 7 works IE 8 works IE 9 works FF 13 works Safari 5 no works Chrome 18 no works 3.1 (rvn #28834): IE 7 no works IE 8 no works IE 9 no works FF 13 works Safari 5 no works Chrome 18 no works
        Hide
        Mircea Toma added a comment -

        Reverted back to the strategy where only the cursor is changed programmatically (as opposed to using a style class).

        Show
        Mircea Toma added a comment - Reverted back to the strategy where only the cursor is changed programmatically (as opposed to using a style class).
        Hide
        Mircea Toma added a comment -

        Enabled 'wait' cursor for Webkit based browsers.

        Show
        Mircea Toma added a comment - Enabled 'wait' cursor for Webkit based browsers.
        Hide
        Ken Fyten added a comment -

        Won't be proceeding with this change as using a dynamic style class results in additional complexity and fragility on some browsers with little perceived gain.

        Show
        Ken Fyten added a comment - Won't be proceeding with this change as using a dynamic style class results in additional complexity and fragility on some browsers with little perceived gain.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Evgheni Sadovoi
          • Votes:
            1 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: