VATSIM-UK / UK-Sector-File

UK Sector File
GNU General Public License v3.0
64 stars 123 forks source link

Move 'Coastline' Region to top of [REGIONS] #546

Closed trevorhannant closed 7 years ago

trevorhannant commented 7 years ago

Summary of issue/change

The 'Coastline' Region is overwriting some airport regions due to it's placement in the REGIONS section of the file. As it covers a large area as a background, it should be loaded first before airfield specific regions so that these are layered on top of this.

Reference (amendment doc/official source/forum) incl. page number(s)

None

Affected areas of the sector file (if known)

Misc/Regions_Park Air MAN.txt Unknown others

cpawley commented 7 years ago

I think this cannot be completed directly here, we need to change the compiling tools to enable this

hazzas-99 commented 7 years ago

When would you want both the coastline region and SMR regions on at the same time @trevorhannant ?

hazzas-99 commented 7 years ago

@cpawley - I think this could be achieved by changing the sctgen_master like so...

[REGIONS] Airports*\SMR\Regions.txt;sort=[A-Z]{4}=0 Misc\RegionsPark Air MAN.txt Misc\Regions*.txt;remove=Regions_Park Air MAN.txt

trevorhannant commented 7 years ago

Haven't checked to see if every airport is/isn't affected by this but there potentially could be coastal airports that are (now in future) so having Coastline above the airport regions would make more sense.

Additionally, from a standards/best practice perspective, wouldn't you want to go Wide Area -> Medium Area -> Local Area anyway?

cpawley commented 7 years ago

@pierr3 actually made a decluttering plugin which could be made to turn off the region of coastline at a certain zoom setting?

trevorhannant commented 7 years ago

What was the Coastline region originally used for, just to shade the land area when it's enabled?

If so, would the de-cluttering then make this and Issue #542 something that could be worked through?

pierr3 commented 7 years ago

@cpawley Yes, it doesn't work.

cpawley commented 7 years ago

Test Build 19655ef with the change to regions for testing.

cpawley commented 7 years ago

Fixed in from 2017/03 onwards