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

Update the NW CFS crest images #82

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

LukeTowers
Copy link
Contributor

Let me know if the references have to be updated anywhere else, didn't build or run the app to test locally.

@huguesfcadets
Copy link
Member

The new images size for the NW CFS locations does not match the old ones.

New:
image

Old:
image

Let me know if this is what you want. Otherwise, please update the new images with the desired sizes.

@LukeTowers
Copy link
Contributor Author

That's probably fine for now, it looks a bit better with the extra spacing added by the new images in my opinion (minus the two badges that weren't updated, but that's not the end of the world).

Another question for you though, should we change Gimli Cadet Flying Training Centre to Brandon Cadet Flying Training Centre since that's where the CFTC for Northwest is as of last year?

@huguesf
Copy link
Collaborator

huguesf commented Apr 19, 2023 via email

@LukeTowers
Copy link
Contributor Author

@huguesf we can proceed without making that change, if we even have a CFTC in NW this year it would be in Netook and a last minute thing.

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.

3 participants