Skip to content

Fixes #9850: Double space issue with Cyrillic OLED font#9971

Merged
thebentern merged 3 commits intomeshtastic:developfrom
dev-nightcore:fixes
Mar 23, 2026
Merged

Fixes #9850: Double space issue with Cyrillic OLED font#9971
thebentern merged 3 commits intomeshtastic:developfrom
dev-nightcore:fixes

Conversation

@dev-nightcore
Copy link
Contributor

@dev-nightcore dev-nightcore commented Mar 22, 2026

🤝 Attestations

  • I have tested that my proposed changes behave as described.
  • I have tested that my proposed changes do not cause any obvious regressions on the following devices:
    • Heltec (Lora32) V3
    • LilyGo T-Deck
    • LilyGo T-Beam
    • RAK WisBlock 4631
    • Seeed Studio T-1000E tracker card
    • Other (please specify below)
      • Heltec (Lora32) V4

@CLAassistant
Copy link

CLAassistant commented Mar 22, 2026 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added needs-review Needs human review bugfix Pull request that fixes bugs labels Mar 22, 2026
@fifieldt
Copy link
Member

fifieldt commented Mar 22, 2026

thanks for the patch. Can you please explain briefly what the issue is?

@dev-nightcore
Copy link
Contributor Author

dev-nightcore commented Mar 23, 2026

thanks for the patch. Can you please explain briefly what the issue is?

the issue is the excessive width of the space character in the font files (ua and ru). This causes text to overlap on the screen. Look at the pictures, i think it will be more informative.

Before:

IMG_1147

After:
IMG_1152

@thebentern thebentern merged commit 2ffe2d8 into meshtastic:develop Mar 23, 2026
66 of 67 checks passed
thebentern added a commit that referenced this pull request Mar 23, 2026
@dev-nightcore @thebentern
Fixes #9850: Double space issue with Cyrillic OLED font (#9971)
723209b 
* Fixes double space OLEDDisplayFontsRU.cpp

* Fixes double space OLEDDisplayFontsUA.cpp

---------

Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

bugfix Pull request that fixes bugs needs-review Needs human review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dev-nightcore @CLAassistant @fifieldt @thebentern