Skip to content

TextMetrics: introduced a sample for advanced text usages#28

Merged
hermet merged 1 commit intomainfrom
hermet/textmetrics
Feb 26, 2026
Merged

TextMetrics: introduced a sample for advanced text usages#28
hermet merged 1 commit intomainfrom
hermet/textmetrics

Conversation

@hermet
Copy link
Member

@hermet hermet commented Feb 26, 2026

depending on: thorvg/thorvg#4177

2026-02-26-16-30-56.mp4

Copilot AI review requested due to automatic review settings February 26, 2026 07:34
@hermet hermet self-assigned this Feb 26, 2026
Copy link

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 introduces a new TextMetrics example demonstrating advanced text manipulation capabilities including a simple text editor with cursor positioning, line wrapping, and basic text input handling. It extends the example framework to support keyboard events.

Changes:

  • Added keyboard event handling to the example framework (keydown callback, shift key tracking)
  • Created TextMetrics.cpp demonstrating text metrics API with an interactive text editor
  • Added TextMetrics.cpp to the build configuration

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 10 comments.

File Description
src/meson.build Added TextMetrics.cpp to the build list
src/TextMetrics.cpp New interactive text editor example demonstrating text metrics, cursor positioning, and line tracking
src/Example.h Added keydown virtual method and lshift tracking to support keyboard input in examples

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

@hermet hermet force-pushed the hermet/textmetrics branch from 4501f12 to 7f88309 Compare February 26, 2026 09:07
@hermet hermet merged commit 66df422 into main Feb 26, 2026
1 check passed
@hermet hermet deleted the hermet/textmetrics branch February 26, 2026 09:10
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