Skip to content

Refactor virtual keyboard cell height for layout consistency#10501

Merged
thebentern merged 4 commits into
meshtastic:developfrom
Bjk8kds:Vkey
May 21, 2026
Merged

Refactor virtual keyboard cell height for layout consistency#10501
thebentern merged 4 commits into
meshtastic:developfrom
Bjk8kds:Vkey

Conversation

@Bjk8kds

@Bjk8kds Bjk8kds commented May 19, 2026

Copy link
Copy Markdown
Contributor

Adjust virtual keyboard cell height calculation for better layout consistency across different screen sizes.

This is the best fix I can do. Maybe if someone can do better it would be very helpful.

Fix #10446

before Image
after Image

🤝 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)
      • NRF52 Pro-micro DIY

Bjk8kds added 2 commits May 10, 2026 18:31
Adjust keyboard cell height calculation for better layout consistency across different screen sizes.
@github-actions github-actions Bot added needs-review Needs human review enhancement New feature or request labels May 19, 2026
@Bjk8kds Bjk8kds marked this pull request as ready for review May 19, 2026 07:07
@Bjk8kds Bjk8kds changed the title Refactor keyboard cell height for layout consistency Refactor virtual keyboard cell height for layout consistency May 19, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 adjusts virtual keyboard rendering to improve layout consistency, particularly for SH1107 128x128 displays where the bottom row and key alignment were visibly off.

Changes:

  • Updates default non-wide keyboard row height calculation to use a font-derived height.
  • Applies consistent ceil-style horizontal text centering for non-action keys.

@HarukiToreda HarukiToreda added the baseui Issues directly related to BaseUI label May 19, 2026
@thebentern thebentern merged commit f3cb2bf into meshtastic:develop May 21, 2026
1 check passed
thebentern added a commit that referenced this pull request May 21, 2026
Adjust keyboard cell height calculation for better layout consistency across different screen sizes.

Co-authored-by: Ben Meadors <benmmeadors@gmail.com>
@Bjk8kds Bjk8kds deleted the Vkey branch May 23, 2026 16:36
Evil8it pushed a commit to Evil8it/ME4TACTNK that referenced this pull request Jun 10, 2026
Adjust keyboard cell height calculation for better layout consistency across different screen sizes.

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

Labels

baseui Issues directly related to BaseUI enhancement New feature or request needs-review Needs human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Misaligned Virtual keyboard on SH1107 128x128

5 participants