OpenLayers OpenLayers

Ticket #822 (closed feature: fixed)

Opened 1 year ago

Last modified 10 months ago

Give Controls Tooltips

Reported by: euzuro Assigned to: penyaskito
Priority: minor Milestone: 2.6 Release
Component: Control Version: 2.4
Keywords: Cc:
State: Complete

Description

It's hardly a crucial feature at this point, but I think it would be nice to add tooltips to controls like the pan/zoom and the layerswitcher to give the users hints as to what the different buttons do.

Attachments

822.patch (4.8 kB) - added by penyaskito on 01/26/08 09:55:00.
Patch (including tests and example)

Change History

07/10/07 13:09:35 changed by pgiraud

It would also be great if user can define his own toolTip texts so that applications can support i18n. I don't know what solution you propose to add toolTips but labels must not be hard coded somewhere in the code, instead a default (english) value can be set and user can easily override it.

07/10/07 13:39:19 changed by euzuro

  • milestone set to 2.6 Release.

Oh absolutely. Since this ticket will mark the 3rd string (after "BaseLayer" and "Overlay") in the actual application, it is a good impetus to take internationalization seriously.

This ticket and #109 should be intimately linked.

09/15/07 10:42:46 changed by penyaskito

  • owner set to penyaskito.
  • status changed from new to assigned.

I started working on this, but doesn't work on Firefox :-(

http://dev.openlayers.org/sandbox/penyaskito/openlayers/examples/attribution.html

I don't know why this doesn't work on FF.

Maybe the solution is implementing our own tooltips. MooTips (from the MooTools framework) can be a good inspiration source, what do you think about this?

09/16/07 19:53:39 changed by penyaskito

The problem with Firefox is solved with setting fallThrough=true in the map. By default is false, crschmidt will ask euzuro why is this the default value.

09/16/07 20:40:34 changed by penyaskito

(In [4342]) A fix for controls' tooltip inside panels, and modified examples/panel.html for showing how it works (refs #822)

10/06/07 21:05:34 changed by crschmidt

  • keywords set to review.

12/15/07 10:04:13 changed by crschmidt

penyaskito:

Does this work in IE?

12/15/07 11:33:08 changed by crschmidt

  • state changed from Review to Needs More Work.

Bumping this from review until I hear a yes/no on this: if it's yes, please push it back to review.

01/22/08 17:36:52 changed by crschmidt

  • milestone changed from 2.6 Release to 2.7 Release.

Mass ticket move to 2.7 post dev meeting. If you are actively working on this task, please update this ticket with information and change the milestone to 2.6. At the time of the next IRC meeting (most likely 1-31-08), this will mean the ticket can *not* be brought back into 2.6 unless there is further feedback.

01/26/08 09:54:00 changed by penyaskito

  • state changed from Needs More Work to Review.
  • milestone changed from 2.7 Release to 2.6 Release.

Sorry, I forgot this :-( The control worked under IE7 and Firefox, and the tests run too. I update the patch.

01/26/08 09:55:00 changed by penyaskito

  • attachment 822.patch added.

Patch (including tests and example)

01/26/08 10:06:30 changed by penyaskito

Runs under Opera 9.24 too.

01/27/08 09:44:16 changed by crschmidt

  • status changed from assigned to closed.
  • state changed from Review to Complete.
  • resolution set to fixed.

(In [5910]) Add support to set the 'title' property of a control such that it will be displayed when hovering over the control in a control.panel via its panel_div. r=me (Closes #822)