(#16428) allow to set custom font for generic tree ctl (gtk) (PR #25851)

13 views
Skip to first unread message

Ivan Baidakou

unread,
Oct 2, 2025, 4:36:47 AM (4 days ago) Oct 2
to wx-...@googlegroups.com, Subscribed

Hi.

This allows to set custom font the tree component on linux.

Also should fix the issue #16428


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

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

Commit Summary

  • d9d7323 (#16428) allow to set custom font for generic tree ctl (gtk)

File Changes

(1 file)

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/25851@github.com>

VZ

unread,
Oct 2, 2025, 6:08:48 AM (4 days ago) Oct 2
to wx-...@googlegroups.com, Subscribed

@vadz commented on this pull request.

Thanks, this is probably correct, although the interaction of this check with the check for !m_hasExplicitFont just above is rather confusing.

I do have to wonder why do we need m_normalFont at all instead of just using m_font...


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/25851/review/3293762401@github.com>

Ivan Baidakou

unread,
Oct 2, 2025, 6:29:55 AM (4 days ago) Oct 2
to wx-...@googlegroups.com, Subscribed
basiliscos left a comment (wxWidgets/wxWidgets#25851)

I do have to wonder why do we need m_normalFont at all instead of just using m_font

I have no idea. If you are k, I can remove all usages of m_normalFont (+ probably m_hasExplicitFont)


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/25851/c3360384058@github.com>

Reply all
Reply to author
Forward
0 new messages