Skip to content

Close the deferred minors from the initial review - #5

Open
kattni wants to merge 22 commits into
beeware:mainfrom
kattni:main
Open

kattni wants to merge 22 commits into
beeware:mainfrom
kattni:main

Conversation

@kattni

@kattni kattni commented Oct 10, 2026

Copy link
Copy Markdown
Collaborator

Closes the minor findings parked during the initial review of CodeEditor, across the interface, the highlighting module, the three native backends, the example app, and the docs.

The user-visible changes: construction paints once, with the theme in place first; a failure in the debounced re-highlight reaches the event loop's exception handler at once; DEFAULT_THEME is read-only; zero-length spans never reach a backend; macOS reads only plain text from the pasteboard, and Android strips pasted formatting when it re-highlights; toga-core is bounded below 0.6; and the Android keyboard note now matches what Android 12 and 15 emulators show.

Verified by the test suite at 100% coverage and by running the example app on macOS, the iOS 26.3 simulator, and Android 12 and 15 emulators.

PR Checklist:

  • I will abide by the BeeWare Code of Conduct
  • I have read and have followed the CONTRIBUTING.md file
  • This PR was generated or assisted using an AI tool
    Assisted-by: Claude Code (Claude Fable 5.1)

kattni added 22 commits October 10, 2026 00:09
# Conflicts:
#	docs/superpowers/specs/2026-10-06-code-editor-design.md
#	tests/test_highlighting.py
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.

1 participant