#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 |
Description
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)
Change History (4)
comment:1 by , 8 years ago
Milestone: | → 16.06 |
---|
by , 8 years ago
Attachment: | patch-layer-manager-mapview-attach.patch added |
---|
comment:2 by , 8 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
In 10458/josm: