Let the editor fill the window, with a configurable measure - #24
Open
x3m wants to merge 1 commit into
Open
Conversation
Greptile SummaryThis PR makes the editor fill the available window width by default while preserving a configurable fixed-column measure.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| src/Main.qml | Adds persistent editor-column configuration and switches the default measure to the bounded available window width. |
| omawrite.pro | Enforces Qt 6.5 or newer at qmake configuration time for the new QtCore Settings dependency. |
| pkgbuild/PKGBUILD | Aligns Arch build and runtime Qt package version constraints with the new minimum. |
| README.md | Documents the Qt 6.5 minimum required by the updated QML interface. |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart TD
S[Read layout/editorColumns] --> C{Value greater than 0?}
C -- No --> A[Use available window width]
C -- Yes --> F[Clamp to at least 20 columns]
F --> M[Cap at available window width]
A --> E[Set editor width]
M --> E
Reviews (4): Last reviewed commit: "Let the editor fill the window, with a c..." | Re-trigger Greptile
x3m
force-pushed
the
configurable-line-length
branch
from
August 25, 2026 09:07
63e2b48 to
3bed051
Compare
x3m
force-pushed
the
configurable-line-length
branch
from
August 25, 2026 09:30
3bed051 to
84dff04
Compare
The editor measure is hardcoded at 65 average character widths, so on a large display most of the window is empty margin, and long list items (meeting minutes, task lists) wrap early. It also caps information density: once text-size shortcuts exist, shrinking the font just shrinks the column instead of fitting more text per line. Let the text use the window instead: the editor now fills the available width, keeping the existing ten characters of margin on either side. The classic fixed measure remains available through the QSettings store — [layout] editorColumns in omawrite.conf, where 0 (the default) means fill the window and a positive value (clamped to at least 20, still capped by the window width) gives a fixed measure; 65 restores the previous look exactly. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01D5cK4VRT6nuLGuwEJrZvRD
x3m
force-pushed
the
configurable-line-length
branch
from
August 25, 2026 10:26
84dff04 to
a7c79a0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The editor measure is hardcoded at 65 average character widths. That is a classic default for prose, but on a large display most of the window is empty margin, and notes with long list items (meeting minutes, task lists) wrap early. It also caps information density: combined with text-size shortcuts (#25), shrinking the font just shrinks the column — you never fit more text per line.
This lets the text use the window instead:
width - 20chterm that already existed).[layout] editorColumnsinomawrite.conf—0(default) fills the window; a positive value gives a fixed measure (clamped ≥20, still capped by window width);65restores today's look exactly. No new UI.To be upfront: this changes the default look — the column is wider on large windows. If you'd rather keep 65 as the default, the same patch works with the default flipped; happy to adjust.
Tested on Omarchy (Qt 6.11) in both modes, including the interplay with #25 (smaller font → more characters per line when filling the window).
🤖 Generated with Claude Code
https://claude.ai/code/session_01D5cK4VRT6nuLGuwEJrZvRD