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

Fix tag starting position #36

Merged
merged 2 commits into from
Feb 12, 2025
Merged

Fix tag starting position #36

merged 2 commits into from
Feb 12, 2025

Conversation

Epse
Copy link
Owner

@Epse Epse commented Feb 10, 2025

Ever since the left-right alignment days, the tag has started out slightly to the left and either above or on top of the radar return.
This was a bug.

It also left aligned despite the leader line being sliiiightly to the left of the radar return and vice versa on the bottom. This has annyed me forever.

No more.

This MR does the following:

  1. Eliminate discrete tag angles. There's no reason, it's annoying and it introduces the LtR - RtL bug
  2. Start tag angle at 0. This makes the tag end up on the right of the PRS and never overlap. This is a workaround because at present we cannot compensate the position of the tag for the height of the tag.

Remaining at this point is the leader line length variable not actually defining the leader line length. Rather it defines the distance from the tag "start point" to the PRS. The start point is defined as top left when using LtR and top left when using RtL.
It may be intuitive now that when a tag has four lines, such as when there is an error, and is positioned above or to the left of the RPS, this makes a significant distance in the actual line length.

Epse added 2 commits February 10, 2025 18:59
Why was this ever even a thing???
Removes offset and sets default angle to 0 degrees,
directly right of the aircraft.

This is a somewhat messy workaround to ensure the tag never starts out
overlapping the radar return.
@Epse Epse merged commit ce6037c into master Feb 12, 2025
1 check passed
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.

1 participant