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
Fixes #5305 - Add Burnham Box #5304
Conversation
@PLM1995 Slightly debatable as to where to put this. It's relevant to both Bristol and Cardiff, so it could go in misc. But it only applies to Bristol inbounds... I think by naming it with "(Bristol)" at the end will make it easy to locate. Thoughts? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PLM1995 Slightly debatable as to where to put this. It's relevant to both Bristol and Cardiff, so it could go in misc. But it only applies to Bristol inbounds... I think by naming it with "(Bristol)" at the end will make it easy to locate.
Thoughts?
Good question. We already have the Exmor Burnham Line
defined in the Radar Map (Stars)
folder (displays per below), and this feels very similar to that? Are you deliberately re-defining it, and if so I would probably update that file, maybe with a re-name, rather than adding it to the RMA one.
I think the line is the exact same from the look of it, so let's put them together. |
On a re-think. |
@PLM1995 check you're happy with the move |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
Fixes #5303
Summary of changes
Adds the Burnham box to the Bristol STAR/RMA section
Screenshots (if necessary)