PB-589 : limit WMS request to LV95 extent#886
Conversation
Passing run #2509 ↗︎
Details:
Review all test suite changes for PR #886 ↗︎ |
|||||||||||||||
ltkum
left a comment
There was a problem hiding this comment.
In the viewer when zooming, the box is a bit under the slightly transparent footer
If there is an easy way to move the box a few pixels up, that would be great, but I can live with the box being where it is
In 3d, those labels appear in the middle of every tile.
We can fix this issue in a second PR (as the cause is most certainly not the same),but this would be better if the behavior between 2d and 3d was the same
OK, that opened up another huge can of worms 🙈 That will requires a bigger change and I will do that separately
|
1474d72 to
f84bee8
Compare
f84bee8 to
d5daa51
Compare
d5daa51 to
18ff2cd
Compare
Layer that are telling, with a label at bottom, when was their last update weren't displayed in a manner that permit this label to be shown (it was out of scope of the map viewport with how the request was built) This fixes two things : - Limits the request to our own WMS to LV95 extent - Tells OpenLayers to not request extra information around (gutter) but keep a ratio of 1:1 when requesting a WMS (this way the update label can be inside the viewport)
18ff2cd to
885fdd7
Compare
Layer that are telling, with a label at bottom, when was their last update weren't displayed in a manner that permit this label to be shown (it was out of scope of the map viewport with how the request was built)
This fixes two things :
Good candidate to test :
ch.bazl.luftfahrthindernisTest link with layer
Test link