Skip to content

Repaint the search box fully when it is resized - #489

Merged
lostindark merged 2 commits into
lostindark:masterfrom
indep-arg:pr/search-box-redraw
Oct 28, 2026
Merged

lostindark merged 2 commits into
lostindark:masterfrom
indep-arg:pr/search-box-redraw

Conversation

@indep-arg

@indep-arg indep-arg commented Oct 28, 2026 •

Copy link
Copy Markdown
Contributor

While resizing the window, short vertical lines pile up at the right end of the search box, above and below the text. The control draws its border along its right edge, but a resize only repaints the newly exposed area, so each old edge stays on screen.

image

This turns on ResizeRedraw (with double buffering) so the whole control is repainted on resize. One-line change in EnhancedSearchTextBox's constructor.

The search box draws its border along its right edge, but a resize only
repainted the newly exposed area, so every old right edge stayed behind
as short lines above and below the text box while the window was being
resized. Repaint the whole control on resize (double-buffered).
@indep-arg

Copy link
Copy Markdown
Contributor Author

Now with this fix:

image

UpdateColumnSize sizes the columns to fit header and content, but that
doesn't count the sort arrow ObjectListView draws in the header, nor the
priority number when sorting by two columns. With the default sort
(Driver Class, then Driver Date) the header read "Drive...". Add their
width to the sorted columns.
@indep-arg

Copy link
Copy Markdown
Contributor Author

"Driver Date" shown as "Drive..." (new since the ObjectListView 3.0.10 update). After 24320c5, the header is drawn by ObjectListView with a sort arrow and, when sorting by two columns, a priority number. UpdateColumnSize sizes columns to fit header and content, but that doesn't count the arrow and number, so with the default sort (Driver Class, then Driver Date) the Driver Date header gets cut. The current release still uses ObjectListView 2.9.1, so it isn't affected. The sorted columns now get that extra width.

@lostindark
lostindark merged commit 545271a into lostindark:master Oct 28, 2026
2 checks passed
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