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

DH8 icon #15

Closed
wants to merge 1 commit into from
Closed

DH8 icon #15

wants to merge 1 commit into from

Conversation

daviessm
Copy link

Update DH8 family icons to large twin turboprop to match FA webview (and reality)

@daviessm
Copy link
Author

I noticed that aircraft type icons come from icao_aircraft_types and are overridden in markers.js. Since the former isn't a dump from an upstream source any more, does it make sense to have this maintained in two files? I'd be happy to work on merging them into one if you think it would work better.

@mutability
Copy link

I think the dash-8 case is just one case of a more general problem which is that the L2T and L2P icon mappings look like they have been reversed. Asking the skyview developers what the story there is now.

I don't understand your question about icao_aircraft_types. It was mechanically generated and should be kept up to date (I will remind the skyview devs). It maps aircraft type designators (DH8D) to descriptions (L2T). Then markers.js either has a specific rule for the model, or a fallback rule based on the generic description.

@daviessm
Copy link
Author

Re: the icao_aircraft_types I could have sworn I saw in the readme file that it wasn't being maintained any more but it looks like I was wrong and that was the hex code DB readme instead.

If the modification is made upstream then the override I put in can probably be removed, along with the original override. Let me know if you want me to make any changes but if not you can close this.

@mutability
Copy link

I fixed the L2P/L2T mapping and removed the Dash 8 override.
Will revisit this later, I need to make sure that the mappings stay in sync with the FA maps somehow.

@mutability mutability closed this Jul 31, 2017
@daviessm daviessm deleted the dh8_icon branch August 2, 2017 10:33
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.

None yet

2 participants