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

Adjustments to Font Loading to Support Other Platforms. #982

Closed

Conversation

Hendrin-Mckay
Copy link

Modified a few files dealing with how font loading is done on platforms besides Windows and tried to avoid too much repeating code.

I am still a beginner at both C# and GIT so please review this, I tried testing it on Windows 11/Linux/OSX to make sure I didn't break anything in the process.

Linux appears to work with one issue (you need to resize the Window before it starts rendering anything).
OSX (on Apple Silicon) has other issues as your using some stuff that requires OpenGL features beyond 4.1, which is the maximum supported on Apple Silicon (they expect you to support Metal or Metal via MoltenVK(Metal via Vulkan).

Plan to work on these issues but will do it on a separate request as they are completely different issues than what this one dealt with.

Adjusted Font System to add support for Linux and OSX in addition to Windows.
Updated to correct for OSX system font directory
@Hendrin-Mckay Hendrin-Mckay marked this pull request as ready for review July 3, 2024 01:34
@CalvinWilkinson
Copy link
Member

CalvinWilkinson commented Jul 8, 2024

@Hendrin-Mckay

Awesome!! Appreciate your time. 😉

One thing that I would do if you do mind is to check out the Contribution Guide.

This and the other markdown docs talk about how to contribute.

For example, I have some sophisticated CICD set up and it requires that the branch name be a certain name.
Also, every pull request needs to have an issue. I think the better way to go is to either submit a bug using the issue templates.

If you could please recreate the pull request with the correct branch following the feature branch syntax, that would be greatly appreciated. Once that is done, I will link up issue #692 the to pull request and we can move forward.

Example Branch: feature/123-my-branch.

Specifically, branching is here

If you have any questions or you see something that does not seem right, please don't hesitate to ask. 😊
I am also always available on the discord server.

Cheers!!

@Hendrin-Mckay Hendrin-Mckay deleted the hendrin-branch branch July 21, 2024 00:11
@Hendrin-Mckay Hendrin-Mckay restored the hendrin-branch branch July 21, 2024 00:17
@Hendrin-Mckay Hendrin-Mckay reopened this Jul 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants