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

AP_OSD:Update HD fonts #27284

Merged
merged 1 commit into from
Jun 13, 2024
Merged

AP_OSD:Update HD fonts #27284

merged 1 commit into from
Jun 13, 2024

Conversation

Hwurzburg
Copy link
Collaborator

@Hwurzburg Hwurzburg commented Jun 11, 2024

Update HD fonts for use with ArduPilot for DJI and Walksnail HD Goggles. Non functional change. Once merged I can update the links in the wiki.

@Hwurzburg Hwurzburg requested review from peterbarker and IamPete1 and removed request for peterbarker June 11, 2024 13:42
@andyp1per
Copy link
Collaborator

@Hwurzburg does this include the latest SneakyFPV fonts? They are the best for WS. Think we already have those ...

@Hwurzburg
Copy link
Collaborator Author

Hwurzburg commented Jun 11, 2024

These are his latest, and no we don't already have these, only the old Europa...these include a unified ini file, and fonts for the optional CRSF extend link panels, if custom built

@rmaia3d
Copy link
Contributor

rmaia3d commented Jun 12, 2024

These are his latest, and no we don't already have these, only the old Europa...these include a unified ini file, and fonts for the optional CRSF extend link panels, if custom built

In the CRSF extended link panels PR, I only included one variation of the SneakyFPV fonts for Walksnail. Here you are including all the variations with support for the extended panels, which is great, more options for users! 👍

However, I'm just not sure if these WTFOS fonts have support for the panel extensions. Sneaky's latest fonts for WTFOS do have that support, but I don't know the if the fonts included in this directory are from him or the original WTFOS fonts without the extended icons. Just a heads up.

@Hwurzburg
Copy link
Collaborator Author

Rodrigo, you are right .....WTFOS just changed last month to png font....I had worked with Shannon on a unified set of all AP fonts for WS but now have included directories for both the old .bin WTFOS and new (which includes the new CRSF font additions) PNG style....only the Europa one though since you cant select fonts via the goggle menu like WS, but this will allow users reading the wiki to avoid the hunt thru Shannons zip files and get running

Copy link
Contributor

@rmackay9 rmackay9 left a comment

Choose a reason for hiding this comment

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

I'm not completely sure that I'm qualified to review this bug it looks ok to me.

@rmackay9 rmackay9 merged commit 247f5e5 into ArduPilot:master Jun 13, 2024
92 checks passed
@Hwurzburg
Copy link
Collaborator Author

thanks...will update wiki by tomorrow...

@Hwurzburg Hwurzburg deleted the HD_fonts_update branch June 13, 2024 00:14
@rmaia3d
Copy link
Contributor

rmaia3d commented Jun 13, 2024

but this will allow users reading the wiki to avoid the hunt thru Shannons zip files and get running

Yes, this is great! Hunting through those zip files can be a bit dauting at first, so having the files already neatly organized and sorted in the Ardu package will make life a lot easier, specially for new users, or users "new" to HD systems. :+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants