#12863 closed enhancement (fixed)
[Patch] LayerManager implementation
| Reported by: | michael2402 | Owned by: | michael2402 |
|---|---|---|---|
| Priority: | normal | Milestone: | 16.05 |
| Component: | Core | Version: | |
| Keywords: | gsoc-core, layers | Cc: | Don-vip, bastiK, stoecker |
Description
This is the implementation of a new manager class that takes over all layer management the MapView once did.
There are new interfaces for layer listeners. All listeners fire in the EDT to make implementing UI listeners easier.
See #11838 for more information about the new layer manager.
Attachments (1)
Change History (17)
by , 8 years ago
| Attachment: | patch-add-layer-manager.patch added |
|---|
comment:1 by , 8 years ago
comment:3 by , 8 years ago
Please for new functions, classes already add the "@since xxx". We'll fill in the number when doing the checkin. It's easier to search for "@since xxx" then to check the whole code :-)
BTW LayerManagerWithActive is not really a fancy name...
comment:4 by , 8 years ago
I agree that the name is not good. I could not really think of a better one. If you can come up with one, feel free to change it before it is used in any plugins or other places.
We could rename LayerManagerWithActive->LayerManager and LayerManager -> BasicLayerManager - since most people will be using the one that most people will use.
The LayerManager.java file seems to be missing in the commit.
follow-up: 8 comment:6 by , 8 years ago
LayerManagerWithActive --> MainLayerManager? One rename less. :-)
comment:8 by , 8 years ago
Replying to stoecker:
LayerManagerWithActive --> MainLayerManager? One rename less. :-)
Yes, we can do that. Can you do it or should I do a patch?
comment:12 by , 8 years ago
| Milestone: | 16.06 → 16.05 |
|---|
Looks good!