Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#13029 closed enhancement (fixed)

[Patch] Replace hookUpMapView by attachToMapView

Reported by: michael2402 Owned by: team
Priority: normal Milestone: 16.06
Component: Core Version:
Keywords: gsoc-core Cc: Don-vip, bastiK, stoecker


When supporting multiple (or none) map views, hookUpMapView() is not what most layers need.

I add a new method attachToMapView and detachFromMapView to allow for handling multiple MapViews.

The basic idea is that all you can make your layer extend LayerPainter and add the detachFromMapView and paint method to it the way it was before.

Or you can create a new LayerPainter instance for every map view if you need to cache data that is specific for each map view.

The layer painters are then used to paint the current layer. For now, this is just a call to the old Layer.paint method.

This is currently only added for the Layer class. I want to add it to temporary layers later.

Attachments (1)

patch-layer-manager-mapview-attach.patch (33.4 KB) - added by michael2402 7 years ago.

Download all attachments as: .zip

Change History (4)

comment:1 Changed 7 years ago by michael2402

Milestone: 16.06

Changed 7 years ago by michael2402

comment:2 Changed 7 years ago by Don-vip

Resolution: fixed
Status: newclosed

In 10458/josm:

fix #13029 - Replace hookUpMapView by attachToMapView (patch by michael2402, modified) - gsoc-core

comment:3 Changed 7 years ago by Don-vip

In 10461/josm:

see #13029 - fix javadoc warnings

Modify Ticket

Change Properties
Set your email in Preferences
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted.

Add Comment

E-mail address and name can be saved in the Preferences.

Note: See TracTickets for help on using tickets.