Created attachment 155077 [details] Big font example SUMMARY Using html tags, you can make font size enormous. For example: <p style="font-size:50px;">Test</p> This should be checked and there's need to be upper limit to font sizes. Or remove font-size styling. STEPS TO REPRODUCE 1. Open Neochat 2. Write <p style="font-size:50px;">Test</p> and send it 3. Observe huge text OBSERVED RESULT Big text appears EXPECTED RESULT Text would be clamped to some specific size, probably the default size of the system. SOFTWARE/OS VERSIONS Operating System: Fedora Linux 37 KDE Plasma Version: 5.26.4 KDE Frameworks Version: 5.101.0 Qt Version: 5.15.7 Kernel Version: 6.0.16-300.fc37.x86_64 (64-bit) Graphics Platform: Wayland Processors: 12 × AMD Ryzen 5 3600 6-Core Processor Memory: 15,5 GiB of RAM Graphics Processor: AMD Radeon RX 6600
I think the move here is to strip the style attributes. The matrix spec has a list of recommendations https://spec.matrix.org/v1.5/client-server-api/#mroommessage-msgtypes and it doesn't include style tags on paragraphs and highly limits them on others.
A possibly relevant merge request was started @ https://invent.kde.org/network/neochat/-/merge_requests/808
A possibly relevant merge request was started @ https://invent.kde.org/network/neochat/-/merge_requests/832
Resolved by https://invent.kde.org/network/neochat/-/merge_requests/808
Git commit f6ba4f2ecd470487f1133a570459086059750775 by James Graham. Committed on 13/03/2023 at 17:28. Pushed by nvrwhere into branch 'master'. Improve Text Handling Improve the handling of text both when sending and receiving. The main feature is to fix the linked bug (and a host of others that are unreported but similar) which is caused by the fact that we don't properly clean html. This mr does that as per the matrix spec https://spec.matrix.org/v1.5/client-server-api/#mroommessage-msgtypes. So any disallowed tags or attributes are removed and it does the special handling for certain attributes. Additionally the functions are also designed to cover any other text formatting required, particularly fro received strings. The receive side is covered by 2 functions `handleRecieveRichText` and `handleRecievePlainText`. The rich/plain in the function name refers to the output type not the input type (both can take plain and rich input), so `handleRecieveRichText` is called to get a string suitable to go in a rich text control and `handleRecievePlainText` for a plain control. The functions also handle the following some of which was previously handled by `eventToString` in `NeoChatRoom`: - Strip and reply from the string - Format any user mentions - Linkify links in plain strings - Handle mxc urls in rich text (uses the new `room->makeMediaUrl` functionality from libQuotient) - `handleRecievePlainText` also deals with markup making `NeoChatRoom->subtitle` redundant There is also an extensive test suite which defines the behaviour and the best way to review this is probably to look at the tests and decide whether you agree with the expected output given the inputs and/or if there is any missing behaviour. The final aim especially with the test suite is to give us a framework to make further updates in the future easier and hopefully prevent a new feature breaking old behaviour with the tests. Related: bug 466330, bug 466930 M +6 -0 autotests/CMakeLists.txt M +1 -1 autotests/neochatroomtest.cpp A +482 -0 autotests/texthandlertest.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] M +1 -1 src/CMakeLists.txt M +5 -17 src/actionshandler.cpp M +0 -2 src/actionshandler.h M +0 -1 src/models/messageeventmodel.cpp M +1 -1 src/models/roomlistmodel.cpp M +28 -76 src/neochatroom.cpp M +2 -8 src/neochatroom.h M +4 -2 src/notificationsmanager.cpp M +2 -19 src/qml/Component/Timeline/RichLabel.qml A +378 -0 src/texthandler.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +131 -0 src/texthandler.h [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] D +0 -4 src/utils.cpp D +0 -16 src/utils.h https://invent.kde.org/network/neochat/commit/f6ba4f2ecd470487f1133a570459086059750775
Git commit da1c664f941b96d650e64240253ba09f46ed291a by James Graham. Committed on 13/03/2023 at 18:18. Pushed by nvrwhere into branch 'release/23.04'. Cherrypick Improve Text Handling to 23.04 Improve the handling of text both when sending and receiving. The main feature is to fix the linked bug (and a host of others that are unreported but similar) which is caused by the fact that we don't properly clean html. This mr does that as per the matrix spec https://spec.matrix.org/v1.5/client-server-api/#mroommessage-msgtypes. So any disallowed tags or attributes are removed and it does the special handling for certain attributes. Additionally the functions are also designed to cover any other text formatting required, particularly fro received strings. The receive side is covered by 2 functions `handleRecieveRichText` and `handleRecievePlainText`. The rich/plain in the function name refers to the output type not the input type (both can take plain and rich input), so `handleRecieveRichText` is called to get a string suitable to go in a rich text control and `handleRecievePlainText` for a plain control. The functions also handle the following some of which was previously handled by `eventToString` in `NeoChatRoom`: - Strip and reply from the string - Format any user mentions - Linkify links in plain strings - Handle mxc urls in rich text (uses the new `room->makeMediaUrl` functionality from libQuotient) - `handleRecievePlainText` also deals with markup making `NeoChatRoom->subtitle` redundant There is also an extensive test suite which defines the behaviour and the best way to review this is probably to look at the tests and decide whether you agree with the expected output given the inputs and/or if there is any missing behaviour. The final aim especially with the test suite is to give us a framework to make further updates in the future easier and hopefully prevent a new feature breaking old behaviour with the tests. Related: bug 466330, bug 466930 (cherry picked from commit f6ba4f2ecd470487f1133a570459086059750775) M +6 -0 autotests/CMakeLists.txt M +1 -1 autotests/neochatroomtest.cpp A +482 -0 autotests/texthandlertest.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] M +1 -1 src/CMakeLists.txt M +5 -17 src/actionshandler.cpp M +0 -2 src/actionshandler.h M +0 -1 src/models/messageeventmodel.cpp M +1 -1 src/models/roomlistmodel.cpp M +28 -76 src/neochatroom.cpp M +2 -8 src/neochatroom.h M +4 -2 src/notificationsmanager.cpp M +2 -19 src/qml/Component/Timeline/RichLabel.qml A +378 -0 src/texthandler.cpp [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] A +131 -0 src/texthandler.h [License: GPL(3+eV) GPL(v3.0) GPL(v2.0)] D +0 -4 src/utils.cpp D +0 -16 src/utils.h https://invent.kde.org/network/neochat/commit/da1c664f941b96d650e64240253ba09f46ed291a