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

show also glider sectors on map #773

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

KaiWissel
Copy link
Member

No description provided.

@KaiWissel KaiWissel requested a review from JEK58 May 29, 2023 09:59
getAllRelevant: async () => {
const result = await Airspace.findAll({
where: {
class: {
[Op.notIn]: ["W"],
Copy link
Member

Choose a reason for hiding this comment

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

Aber dann sehen wir doch den W Luftraum wenn man da drauf flicht und nicht den eigentlichen. Das hatte ja irgendwie schon nen Sinn, das wir das so gemacht haben damals. Der W liegt ja immer komplett innerhalb eines anderen LR

Copy link
Member Author

@KaiWissel KaiWissel May 29, 2023

Choose a reason for hiding this comment

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

Wie in folgendem Screenshot, sieht man anhand der "dunkleren" Überlagerung das da eigentlich noch mehr ist.
Aber ich kann das Pop-Up noch etwas tunen, so dass beide Airspaces erwähnt werden.

image_2023-05-29_11-36-39

Copy link
Member Author

Choose a reason for hiding this comment

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

@JEK58

So besser?
grafik

Wäre es nur umständlich möglich die Daten des darunter liegenden Luftraums im selben Pop-Up anzuzeigen.

Copy link
Member

Choose a reason for hiding this comment

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

@JEK58

So besser? grafik

Wäre es nur umständlich möglich die Daten des darunter liegenden Luftraums im selben Pop-Up anzuzeigen.

Ne eigentlich nicht, man sieht ja den eigentlichen Airspace nicht und die meisten Leute raffen das mit den Segelflugsektoren ja offensichtlich eh nicht.
Ich würde die einfach garnicht anzeigen und ignorieren. Die liegen eh immer innerhalb eines LR auf den wir prüfen. Für den seltenen Fall, dass jemand einen Segelflugsektor nutzt wird er das plausibel erklären können. Alles andere verwirrt glaube ich nur oder verleitet zu dummen Aktionen wegen Halbwissen.

Copy link
Member Author

Choose a reason for hiding this comment

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

Und das ganze farblich anders darzustellen?

flyxc.app macht das z.B. so:
grafik

Copy link
Member

Choose a reason for hiding this comment

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

Ja könnte man machen, aber mit gelb signalisiert man irgendwie auch, dass es nicht so schlimm wäre da einzufliegen. Ich würde es tatsächlich lassen wie es ist. 99% der Piloten sollten und dürfen da eh nicht rein;)

@JEK58
Copy link
Member

JEK58 commented Jul 3, 2023

Und hier ebenso, wo waren wir stehengeblieben

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