Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

continents #16

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

continents #16

wants to merge 4 commits into from

Conversation

xmlinc
Copy link

@xmlinc xmlinc commented Mar 20, 2023

No description provided.

@xmlinc xmlinc closed this Mar 20, 2023
@xmlinc xmlinc reopened this Mar 20, 2023
@xmlinc
Copy link
Author

xmlinc commented Mar 20, 2023

[continent]

fixed size ocean names
Copy link
Collaborator

@molind molind left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Есть замечания.

text-priority: -5;
details-enabled: 0;

icon-scale: 0.7;
[entrance=main] {
icon-scale: 0.85;
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что это за пустые строчки?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

мой редактор автоматически удаляет при сохранении файла спрятанные в коде пустые spaces & tabs, видимо там и присутсвовали такие ненужные символы, сорян

@@ -280,7 +296,7 @@ node,area {
font-size: 9;
details-enabled: 1;
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И тут

@@ -734,7 +750,7 @@ node,area {
text-color: @color_text_offices;
}
}

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

И дальше..

}
|z1-2 {
font-size:0;
details-enabled: 0;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Зачем font-size:0 ?
Если ты не хочешь чтоб что-то было видно - поправь зум маску в правиле, которое показывает объект, а не добавляй костыль.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@@ -10,6 +10,14 @@ node,area {
font-stroke-width: 1pt;
font-stroke-color: @color_label_stroke;

[place=continent] {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

если нас интересуют только точки, надо писать node[place=continent] потому что вверху правило node,area и запросы будут в базе искать еще place=country и среди полигонов. Если нам это не надо - сразу фильтруем.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

принял

fixed костыль)
@molind
Copy link
Collaborator

molind commented Apr 11, 2023

На всех устройствах больше iPhone 6s минимальный зум карты > 2.5, когда уже работают правила |z3. И даже на 6s они будут видны с 2.35 до 2.5. Получается имена континентов надо показывать на зумах 2-3. С 3-го зума уже видны названия стран и именам континентов надо дать приоритет выше и шрифт больше, чем у имен стран, чтоб их было видно в первую очередь. Так же в спец правилах для стран правило для z2-3 не будет работать потому что там только размер шрифта а свойство текст после изменений устанавливается с z3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants