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

New font branch #12701

Open
wants to merge 52 commits into
base: master
Choose a base branch
from
Open

Conversation

BrentEaston
Copy link
Collaborator

No description provided.

@BrentEaston BrentEaston added this to the 3.8.0 milestone Sep 15, 2023
@BrentEaston
Copy link
Collaborator Author

Comments from Discord:

  1. When I load some fonts, there's a clue to the vassal_plurals issue... intended text is displayed when there's only 1 font... [also notice all the fonts I loaded have "Registration failed", but as below, they seem to work in the text label!]
    image

  2. The module fonts appear in the text label dropdown. Maybe logically, they should appear above Vassal fonts... module...vassal...java.

3, Change 'Registration failed' to 'Already installed'

  1. Report Duplicate ModuleFont entries via validator.

  2. Make all base Vassal fonts 'Regular'

@BrentEaston BrentEaston linked an issue Oct 11, 2023 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant