[Git][wxwidgets/wxwidgets][master] 2 commits: Don't consume wxEVT_STC_UPDATEUI in wxStyledTextCtrlMiniMap

0 views
Skip to first unread message

Vadim Zeitlin (@_VZ_)

unread,
Oct 31, 2025, 11:10:41 AMOct 31
to wx-commi...@googlegroups.com

Vadim Zeitlin pushed to branch master at wxWidgets / wxWidgets

Commits:

  • d58a97f8
    by Eran Ifrah at 2025-10-31T15:20:49+01:00 
    Don't consume wxEVT_STC_UPDATEUI in wxStyledTextCtrlMiniMap

Insert a call to `event.Skip()` at the start of
`wxStyledTextCtrlMiniMap::OnEditUpdate` so the event continues its
propagation.

Closes #25940.

Signed-off-by: Eran Ifrah <er...@codelite.org>
  • 4bf58e23
    by Vadim Zeitlin at 2025-10-31T15:22:31+01:00 
    Fix a small typo in wxStyledTextCtrlMiniMap comment

No real changes.

1 changed file:

Changes:

  • src/stc/minimap.cpp
    ... ... @@ -544,6 +544,9 @@ void wxStyledTextCtrlMiniMap::OnEditPaint(wxPaintEvent& event)
    544 544
    545 545 
     void wxStyledTextCtrlMiniMap::OnEditUpdate(wxStyledTextEvent& event)
    546 546 
     {
    547 
    +    // Let other handlers process this event in any case.
    548 
    +    event.Skip();
    549 
    +
    547 550 
         if ( event.GetUpdated() & wxSTC_UPDATE_V_SCROLL )
    548 551 
         {
    549 552 
             auto const editFirst = m_edit->GetFirstVisibleLine();
    ... ... @@ -563,7 +566,7 @@ void wxStyledTextCtrlMiniMap::OnEditUpdate(wxStyledTextEvent& event)
    563 566 
             SyncSelection();
    564 567
    565 568 
         // Note that we don't need to refresh the map for wxSTC_UPDATE_CONTENT
    566 
    -    // changes, as Scintilla already refreshs it on its own when the shared
    569 
    +    // changes, as Scintilla already refreshes it on its own when the shared
    567 570 
         // document contents change.
    568 571 
     }
    569 572

Reply all
Reply to author
Forward
0 new messages