Skip to content

Conversation

Copy link

Copilot AI commented Dec 12, 2025

The colon_colon.liga glyph had inconsistent widths across master layers: m019 used 618 while m020 used 600. In monospace fonts, all glyphs must maintain identical widths across all masters.

Changes

  • Standardized m020 layer width to 618 in sources/MonaSansMono.glyphspackage/glyphs/colon_colon.liga.glyph

The width value 618 matches all other monospace ligatures (bar_bar.liga, exclam_equal.liga, hyphen_hyphen.liga, etc.).


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Update metrics and numeral improvements in composite glyphs Fix width inconsistency in colon_colon ligature for monospace font Dec 12, 2025
@kylewaynebenson kylewaynebenson marked this pull request as ready for review December 12, 2025 17:12
Copilot AI review requested due to automatic review settings December 12, 2025 17:12
@kylewaynebenson kylewaynebenson merged commit fee4f4c into mono-sans-mono-ligatures Dec 12, 2025
1 check passed
@kylewaynebenson kylewaynebenson deleted the copilot/sub-pr-98 branch December 12, 2025 17:13
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a width inconsistency in the colon_colon.liga glyph for the MonaSansMono font. The m020 master layer incorrectly had width 600 while m019 had width 618, violating the monospace requirement that all glyphs must maintain identical widths across all masters. The fix standardizes the m020 width to 618, matching the established convention used by all other MonaSansMono ligatures.

  • Corrected m020 layer width from 600 to 618 in the colon_colon ligature glyph
  • Ensured consistency across both master layers (m019 and m020)
  • Aligned with the standard width used by all other monospace ligatures

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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