[Git][wxwidgets/wxwidgets][3.2] Add support for HTML 5 charset attribute in wxHtmlParser

1 view
Skip to first unread message

Vadim Zeitlin (@_VZ_)

unread,
Nov 25, 2025, 12:15:50 PM (2 days ago) Nov 25
to wx-commi...@googlegroups.com

Vadim Zeitlin pushed to branch 3.2 at wxWidgets / wxWidgets

Commits:

  • bfdd0df6
    by Kees-van-der-Oord-Nikon at 2025-11-25T18:14:29+01:00
    Add support for HTML 5 charset attribute in wxHtmlParser
    
    See #25546.
    
    (cherry picked from commit 59ed646ae35e8c5d7db3c57f5456f23d9f01eeb4)
    

2 changed files:

Changes:

  • docs/changes.txt
    ... ... @@ -268,6 +268,7 @@ All (GUI):
    268 268
     - Fix selecting columns in wxGrid with mouse (Dietmar Schwertberger, #25884).
    
    269 269
     - Improve wxActivityIndicator dots colour contrast with background (#25405).
    
    270 270
     - Update wxGenericTreeCtrl item height after changing its font (#16428).
    
    271
    +- Respect HTML5 charset attribute in wxHTML (Kees van der Oord, #25546).
    
    271 272
     
    
    272 273
     wxGTK:
    
    273 274
     
    

  • src/html/htmlpars.cpp
    ... ... @@ -906,6 +906,14 @@ bool wxMetaTagHandler::HandleTag(const wxHtmlTag& tag)
    906 906
             return false;
    
    907 907
         }
    
    908 908
     
    
    909
    +    // HTML 5 charset
    
    910
    +    if (tag.GetParamAsString(wxT("charset"), m_retval))
    
    911
    +    {
    
    912
    +        m_Parser->StopParsing();
    
    913
    +        return false;
    
    914
    +    }
    
    915
    +
    
    916
    +    // HTML 4 charset
    
    909 917
         wxString httpEquiv,
    
    910 918
                  content;
    
    911 919
         if (tag.GetParamAsString(wxT("HTTP-EQUIV"), &httpEquiv) &&
    


View it on GitLab.
You're receiving this email because of your account on gitlab.com. Manage all notifications · Help Notification message regarding https://gitlab.com/wxwidgets/wxwidgets/-/commit/bfdd0df6da91eda26b7b20d2c3555e745bf744d8 at 1764090946

Reply all
Reply to author
Forward
0 new messages