Don't impose arbitrary maximum length on picker controls text (PR #26350)

8 views
Skip to first unread message

VZ

unread,
Apr 5, 2026, 9:42:50 AM (5 days ago) Apr 5
to wx-...@googlegroups.com, Subscribed

This code was there ever since ec376c8 (added
wx{Colour|File|Dir|Font}PickerCtrl (patch 1472329 by Francesco), 2006-05-31), but didn't make any sense because it should be possible to enter arbitrarily long strings in these controls.

Additionally, the maximum length was increased for wxFilePickerCtrl after creation which meant that that its initial contents was not shown in it, but the same string could be shown later, which was even more confusing.

Just stop imposing any length constraints.

See #7863.

Closes #26314.


You can view, comment on, or merge this pull request online at:

  https://github.com/wxWidgets/wxWidgets/pull/26350

Commit Summary

  • 4d60dba Don't impose arbitrary maximum length on picker controls text

File Changes

(2 files)

Patch Links:


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26350@github.com>

VZ

unread,
Apr 5, 2026, 12:40:48 PM (5 days ago) Apr 5
to wx-...@googlegroups.com, Subscribed
vadz left a comment (wxWidgets/wxWidgets#26350)

Applied as 932004d


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26350/c4189165320@github.com>

VZ

unread,
Apr 5, 2026, 12:40:48 PM (5 days ago) Apr 5
to wx-...@googlegroups.com, Subscribed

Closed #26350.


Reply to this email directly, view it on GitHub, or unsubscribe.

You are receiving this because you are subscribed to this thread.Message ID: <wxWidgets/wxWidgets/pull/26350/issue_event/24208004699@github.com>

Reply all
Reply to author
Forward
0 new messages