organicmaps/organicmaps-tmp

Archived

Fork 1

Make national park & military fills less obtrusive #1979

Merged

root merged 2 commits from pastk-styles into master

2022-02-01 13:51:50 +00:00

pastk commented

2022-01-31 11:09:38 +00:00

Owner

Closes #1222

pastk commented

2022-01-31 11:15:47 +00:00

Author

Owner

Before:

After:

Before: ![Screenshot_1643565352](https://user-images.githubusercontent.com/18434508/151783771-c83b7f10-7661-4c03-a371-f9a68b905eec.png) After: ![Screenshot_1643624590](https://user-images.githubusercontent.com/18434508/151784102-d406483d-f9e4-406a-9913-a1a47bd6b45d.png) ![Screenshot_1643565401](https://user-images.githubusercontent.com/18434508/151783917-38633b5a-2bfc-4e8b-af27-b470e275a8bc.png) ![Screenshot_1643624468](https://user-images.githubusercontent.com/18434508/151784185-1c00c10d-f4d9-44a1-b90e-ab5eaa6b7da3.png) ![Screenshot_1643565498](https://user-images.githubusercontent.com/18434508/151783929-f03cfe61-5afc-4d55-9ab1-e3be064d4245.png) ![Screenshot_1643624668](https://user-images.githubusercontent.com/18434508/151784204-c530697e-d6b2-4db1-9b89-32edfac1e06b.png)

pastk commented

2022-01-31 11:16:40 +00:00

Author

Owner

Before:

After:

Before: ![Screenshot_1643566738](https://user-images.githubusercontent.com/18434508/151784470-5142cf45-da62-4e5b-95cf-02f70f9db094.png) After: ![Screenshot_1643621009](https://user-images.githubusercontent.com/18434508/151784482-5938e5cd-a418-4e92-ae1c-187b36617020.png)

pastk commented

2022-01-31 11:20:29 +00:00

Author

Owner

Less obtrusive filling makes it possible to use at lower zooms (for better discoverability of the features from the distance - especially important for military areas):

This particular change will take full effect after maps regeneration only due to scale-index.

Less obtrusive filling makes it possible to use at lower zooms (for better discoverability of the features from the distance - especially important for military areas): ![Screenshot_1643625386](https://user-images.githubusercontent.com/18434508/151784780-2ad8f461-6ce1-46f0-abff-b1d7e4559c35.png) ![Screenshot_1643625429](https://user-images.githubusercontent.com/18434508/151784790-321f5d4e-0ced-4c71-bc05-f03a73e30119.png) ![Screenshot_1643625700](https://user-images.githubusercontent.com/18434508/151784799-330a8840-5ccd-457a-a56f-b39011c51223.png) This particular change will take full effect after maps regeneration only due to scale-index.

vng (Migrated from github.com) approved these changes 2022-01-31 13:02:56 +00:00

biodranik (Migrated from github.com) approved these changes 2022-01-31 13:21:42 +00:00

data/styles/clear/include/Basemap.mapcss

biodranik (Migrated from github.com) commented

2022-01-31 13:21:36 +00:00

А зачем поднимать зум до 12?

vng (Migrated from github.com) reviewed 2022-01-31 13:25:44 +00:00

data/styles/clear/include/Basemap.mapcss

vng (Migrated from github.com) commented

2022-01-31 13:25:44 +00:00

Тоже думал про это, но решил что не экспериментируя, стили мы не подтянем. Давайте пробовать, как-то так ..

pastk reviewed 2022-01-31 13:41:37 +00:00

data/styles/clear/include/Basemap.mapcss

pastk commented

2022-01-31 13:41:37 +00:00

Author

Owner

выше написал - чтобы издалека эти области видеть, особенно актуально для военных зон
а для нацпарков - чтобы проще можно было понять их границы

выше написал - чтобы издалека эти области видеть, особенно актуально для военных зон а для нацпарков - чтобы проще можно было понять их границы

pastk reviewed 2022-01-31 13:43:03 +00:00

data/styles/clear/include/Basemap.mapcss

pastk commented

2022-01-31 13:43:03 +00:00

Author

Owner

так-то бы ещё на меньших зумах хорошо бы видеть, но я решил быть консервативным :)

biodranik (Migrated from github.com) reviewed 2022-01-31 14:47:36 +00:00

data/styles/clear/include/Basemap.mapcss

biodranik (Migrated from github.com) commented

2022-01-31 14:47:36 +00:00

А как будет выглядеть на ещё меньших зумах? Можно генератором сделать одну страну/регион и посмотреть.

Но это на будущее.

А как будет выглядеть на ещё меньших зумах? Можно генератором сделать одну страну/регион и посмотреть. Но это на будущее.

pastk reviewed 2022-01-31 15:48:27 +00:00

data/styles/clear/include/Basemap.mapcss

pastk commented

2022-01-31 15:48:27 +00:00

Author

Owner

Military зоны не очень выглядят, т.к. они обычно небольшие. Если штриховку сделать почаще и/или обводить границы, то будет хорошо.

Заповедники на z10 выглядят неплохо и полезно (на z11 тем более) - они обычно большие.
Хотя штриховку более частую тоже надо бы в идеале (и границы тоже).

Military зоны не очень выглядят, т.к. они обычно небольшие. Если штриховку сделать почаще и/или обводить границы, то будет хорошо. ![Screenshot_1643643462](https://user-images.githubusercontent.com/18434508/151824586-8b58d010-1535-42f1-aeea-6f5078a61110.png) Заповедники на z10 выглядят неплохо и полезно (на z11 тем более) - они обычно большие. Хотя штриховку более частую тоже надо бы в идеале (и границы тоже). ![Screenshot_1643643503](https://user-images.githubusercontent.com/18434508/151824304-f04ee669-b02f-4a07-b92d-4ae6bfe60193.png) ![Screenshot_1643643605](https://user-images.githubusercontent.com/18434508/151824373-db30189d-3361-4481-9e35-1ad5eb5dda51.png)

pastk commented

2022-01-31 19:25:32 +00:00

Author

Owner

So I made national parks z10- and left military areas at z12-

biodranik commented

2022-01-31 19:56:43 +00:00

(Migrated from github.com)

Может милитари хотя бы с 11 показывать? Народ говорил, что критично.

pastk commented

2022-01-31 19:59:39 +00:00

Author

Owner

Может милитари хотя бы с 11 показывать? Народ говорил, что критично.

Ну я и так поднял его с z14- до z12-
На z11 зачастую уже не очень выглядит, надо там со штриховкой или границами ковыряться.

upd: для этого можно отдельный issue сделать

> Может милитари хотя бы с 11 показывать? Народ говорил, что критично. Ну я и так поднял его с z14- до z12- На z11 зачастую уже не очень выглядит, надо там со штриховкой или границами ковыряться. upd: для этого можно отдельный issue сделать

biodranik (Migrated from github.com) approved these changes 2022-01-31 20:34:30 +00:00

This repo is archived. You cannot comment on pull requests.

No reviewers

vng

biodranik