Skip to content

Glyph protocol renderer implementation#12612

Open
raphamorim wants to merge 18 commits intoghostty-org:push-rxukqqxmnmnwfrom
raphamorim:pr-12352-glyph-protocol
Open

Glyph protocol renderer implementation#12612
raphamorim wants to merge 18 commits intoghostty-org:push-rxukqqxmnmnwfrom
raphamorim:pr-12352-glyph-protocol

Conversation

@raphamorim
Copy link
Copy Markdown

  1. Initial support only for glyf
  2. Updated spec to 1.9 without positioning api only width parameter
image

@raphamorim raphamorim requested review from a team as code owners May 7, 2026 08:51
@ghostty-vouch
Copy link
Copy Markdown
Contributor

ghostty-vouch Bot commented May 7, 2026

Hi @raphamorim, thanks for your interest in contributing!

This project requires that pull request authors are vouched, and you are not in the list of vouched users.

This PR will be closed automatically. See https://github.com/ghostty-org/ghostty/blob/main/CONTRIBUTING.md for more details.

@ghostty-vouch ghostty-vouch Bot closed this May 7, 2026
@trag1c
Copy link
Copy Markdown
Member

trag1c commented May 7, 2026

!vouch

@ghostty-vouch ghostty-vouch Bot mentioned this pull request May 7, 2026
ghostty-vouch Bot added a commit that referenced this pull request May 7, 2026
Triggered by
[comment](#12612 (comment))
from @trag1c.

Vouch: @raphamorim

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@trag1c trag1c reopened this May 7, 2026
@raphamorim raphamorim force-pushed the pr-12352-glyph-protocol branch from 0a9c218 to 758f134 Compare May 7, 2026 09:01
/// PUA-A, supplementary PUA-B. Endpoints follow the spec (e.g.
/// 0xFFFFD, not 0xFFFFF) because codepoints above them are
/// noncharacters.
fn isPUA(cp: u21) bool {
Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

added this but if there's other place in the source code I can reuse lemme know

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.

2 participants