OpenLayers OpenLayers

Ticket #688 (closed feature: wontfix)

Opened 1 year ago

Last modified 2 months ago

Mouse cursor = "wait" on WFS loading

Reported by: fvanderbiest Assigned to:
Priority: minor Milestone: 2.7 Release
Component: general Version: SVN
Keywords: Cc:
State: Needs More Work

Description

This patch allows for the change of mouse cursor when it is over the map div. Here, it is used to show a wait cursor on WFS loading.

Attachments

wfs_wait_cursor.diff (2.3 kB) - added by fvanderbiest on 04/28/07 11:46:34.
patch to get a waiting mouse cursor on wfs loading

Change History

04/28/07 11:46:34 changed by fvanderbiest

  • attachment wfs_wait_cursor.diff added.

patch to get a waiting mouse cursor on wfs loading

04/28/07 11:51:17 changed by fvanderbiest

Oups, I noticed a typo in the patch ... You have to replace cursor = 'default' with cursor = 'auto' in the "default" section for the switch in setCursor.

06/17/07 09:28:33 changed by crschmidt

  • keywords changed from review wfs map cursor mouse to needstests.

This patch needs tests: I want to see that if we call 'setCursor', it actually sets it. (The usage of it in the WFS layer is harder to test -- I'm willing to let that one go.)

06/18/07 15:10:23 changed by tschaub

CSS class names please.

Instead of handling this type of behavior differently in different places - and instead of burying style declarations in the code, I'd like to see this handled with CSS class names.

If we had Util functions for addStyleClass and removeStlyeClass (or some similarly named) - then we could let users (or application developers) decide what they wanted those classes to look like (maybe they want their own cute icons for the cursor). By using multiple class names, we can add a class name that corresponds to just the cursor style - remove that class name and the style returns to the default.

08/03/07 12:02:11 changed by crschmidt

  • keywords deleted.
  • milestone changed from 2.5 Release to 2.6 Release.

Bumping to 2.6 pending change to use css

01/20/08 15:50:30 changed by crschmidt

  • state set to Needs More Work.

01/22/08 17:34:15 changed by crschmidt

  • milestone changed from 2.6 Release to 2.7 Release.

Mass ticket move to 2.7 post dev meeting.

07/02/08 04:34:04 changed by tschaub

As Uz kindly reminded me, this deserves the same treatment as #1484 (I believe). Both could use something like http://trac.openlayers.org/browser/sandbox/topp/almanac/lib/OpenLayers/BaseTypes/Element.js#L126

Will check if there is a ticket for that now. Didn't see one - #1607 now.

07/28/08 14:28:45 changed by euzuro

  • status changed from new to closed.
  • resolution set to wontfix.

once #1607 is in, then the correct way to do this will be to listen to the events on the layer (startloading, endloading, or whatever they are called) and then change the map div's class name to be loading (or whatever).

marking as won't fix