geoadmin / ol3

OpenLayers 3
http://ol3js.org/
Other
11 stars 6 forks source link

WORK IN PROGRESS: Tooltip #8

Closed cedricmoullet closed 10 years ago

cedricmoullet commented 11 years ago

Initial version of tooltip

cedricmoullet commented 10 years ago

Single click is done

elemoine commented 10 years ago

My main comment is that the tooltip should not be a control. I think it should just be an object using/wrapping an overlay and a vector layer. (With the current code the container element is added to the map and then stolen, and removed from the map, by the overlay.) Tell me if you want me to write some code.

cedricmoullet commented 10 years ago

Yes, if you could write some code, it would be great !

elemoine commented 10 years ago

@cedricmoullet I wrote a skeleton: https://github.com/elemoine/ol3/compare/geoadmin:master...elemoine:dev_tooltip_eric

cedricmoullet commented 10 years ago

Thanks a lot. Very clear. I'll try to finish it.

cedricmoullet commented 10 years ago

@elemoine here is a new version https://github.com/geoadmin/ol3/pull/12 based on your code. Review welcome ;-)

procrastinatio commented 10 years ago

So this branch is dead ?

cedricmoullet commented 10 years ago

yes.

On Wed, Nov 27, 2013 at 3:35 PM, procrastinatio notifications@github.comwrote:

So this branch is dead ?

— Reply to this email directly or view it on GitHubhttps://github.com/geoadmin/ol3/pull/8#issuecomment-29388357 .

Twitter: http://twitter.com/cedricmoullet Linked In: http://www.linkedin.com/in/cedricmoullet

http://twitter.com/cedricmoullet