Skip to content

courier prime non-APL font; language-[not APL] as nonAPL font#56

Merged
rikedyp merged 5 commits into
mainfrom
55-non-APL-font
Jun 2, 2026
Merged

courier prime non-APL font; language-[not APL] as nonAPL font#56
rikedyp merged 5 commits into
mainfrom
55-non-APL-font

Conversation

@rikedyp
Copy link
Copy Markdown
Collaborator

@rikedyp rikedyp commented May 21, 2026

  • Unspecified code blocks and inline code use the APL font
  • Class language-APL and language-apl use the APL font
  • Code blocks and inline code with class language-* for anything other than APL use the nonAPL code font
  • The nonAPL code font is now Courier Prime, included with license and copyright notice

fixes #55

@rikedyp rikedyp requested a review from abrudz June 1, 2026 10:15
@rikedyp rikedyp changed the title courier prime non-APL font; lanuguage-[not APL] as nonAPL font courier prime non-APL font; language-[not APL] as nonAPL font Jun 1, 2026
Copy link
Copy Markdown
Member

@abrudz abrudz left a comment

Choose a reason for hiding this comment

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

Wrong font. Courier Prime is designed specifically for screenplays. There are lots of good free coding fonts, but if we really want to go with a Courier, use Courier Prime Code.

@FionaDyalog
Copy link
Copy Markdown
Collaborator

Courier Prime Code is not serif-y enough – it needs to be really obvious that it is not APL385 with even short extracts (Courier Prime is OK).

It does not have to be a courier-based font, but does need to be monospace with serif. Let's discuss offline.

@rikedyp
Copy link
Copy Markdown
Collaborator Author

rikedyp commented Jun 2, 2026

Go font was proposed.

image

Looks a bit large so I'm also decreasing the font size by .1rem.

image

@rikedyp rikedyp requested a review from abrudz June 2, 2026 13:17
Comment thread css/main.css
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Sizes should be adjust relative to the current size, not the root size. Otherwise, code in headings, footnotes, etc. will be wrong.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

so just em, not rem

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I've adjusted for this non-APL code issue only, but might be worth a separate issue to audit and clean up the CSS in general. Adjusting font sizes relative to the current size, if best practise, sounds like it will need care to take account of cascading effects.

@rikedyp rikedyp requested a review from abrudz June 2, 2026 13:33
@rikedyp rikedyp merged commit 954d461 into main Jun 2, 2026
@rikedyp rikedyp deleted the 55-non-APL-font branch June 2, 2026 13:52
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.

Non-APL code font and inclusion

3 participants