Bug 486308 - Render (plasmoid) licenses in the default text editor.
Summary: Render (plasmoid) licenses in the default text editor.
Status: RESOLVED DUPLICATE of bug 486305
Alias: None
Product: kde
Classification: I don't know
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR wishlist
Target Milestone: ---
Assignee: Unassigned bugs mailing-list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-04-29 16:04 UTC by Roke Julian Lockhart Beedell
Modified: 2024-09-18 11:37 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:
4wy78uwh: performance-


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Roke Julian Lockhart Beedell 2024-04-29 16:04:12 UTC
SUMMARY
(Plasmoid) licenses should be rendered in the default text editor. This would allow the line breaks to be handled properly without modifications, provide line numbers, and allow myriad alternative improvements. Considering that KParts aren't supported in Kirigami, this is the best way to not reinvent the KWrite KPart for the current Kirigami overlay used to display licenses.

STEPS TO REPRODUCE
1.  Open any plasmoid with an About page (which contains a license) like Application Launcher and Kate Sessions.
2.  Invoke the link to the fake dialog overlay which displays the license.

OBSERVED RESULT
https://bugs.kde.org/show_bug.cgi?id=486305#c0 and https://bugs.kde.org/show_bug.cgi?id=486304#c0 - the experience simply isn't as good as if it were invoked in the default text editor.

EXPECTED RESULT
It should be invoked in the native text editor.

SOFTWARE/OS VERSIONS
Operating System: Fedora Linux 40
KDE Plasma Version: 6.0.4
KDE Frameworks Version: 6.1.0
Qt Version: 6.7.0
Kernel Version: 6.8.8-300.fc40.x86_64 (64-bit)
Graphics Platform: Wayland
Processors: 12 × AMD Ryzen 5 7600X 6-Core Processor
Memory: 30.5 GiB of RAM
Graphics Processor: AMD Radeon RX 5700
Manufacturer: ASRock
Product Name: X670E Taichi

ADDITIONAL INFORMATION
I'm not requesting this specifically for Kate Sessions (or Application Launcher). I want this to apply to all relevant plasmoids.
Comment 1 cwo 2024-09-18 11:20:31 UTC
Thank you for the bug report! I'm marking this as a duplicate of the related bug you filed, as that bug is more focused on the actual problem (unsatisfactory display of licenses due to line breaks) rather than a particular proposed solution.

*** This bug has been marked as a duplicate of bug 486305 ***
Comment 2 Roke Julian Lockhart Beedell 2024-09-18 11:36:42 UTC
(In reply to cwo from comment #1)
Please reconsider that, because this FR stands on its own merits. I really don't want to read licenses within a QML window that I can't even resize, much less move about independently of its parent. Simply rendering the text file in a standard text editor makes it a lot easier to read, in ways that would be completely out of scope of the existent pseudo-window, especially with QtQuick's inability to use the KWrite KPart. I care more about this than any bugs in the current implementation.
Comment 3 Roke Julian Lockhart Beedell 2024-09-18 11:37:25 UTC
(In reply to Roke Julian Lockhart Beedell from comment #2)
Irrespective, it's certainly not a duplicate. It could be closed as WONTFIX, but not as a duplicate of a bug. That bug should be in the See Also, I think.