| Summary: | Sections of Kate settings are barely readable | ||
|---|---|---|---|
| Product: | [Applications] kate | Reporter: | Patrick Silva <bugseforuns> |
| Component: | general | Assignee: | KWrite Developers <kwrite-bugs-null> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | christoph, mattioli.fernando, nate, nicolas.fella |
| Priority: | NOR | Keywords: | usability |
| Version First Reported In: | 20.07.80 | ||
| Target Milestone: | --- | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/utilities/kate/-/commit/895160f40b9f55c50a17f7a9d968b952bcac4f50 | Version Fixed/Implemented In: | 20.12 |
| Sentry Crash Report: | |||
| Attachments: |
screenshot
Enabled items Selected item |
||
Created attachment 130302 [details]
Enabled items
Created attachment 130303 [details]
Selected item
In kateconfigdialog.cpp, both top level items ("Application" and "Editor Component") are disabled:
KPageWidgetItem *applicationItem = addPage(new QWidget, i18n("Application"));
applicationItem->setIcon(QIcon::fromTheme(QStringLiteral("preferences-other")));
applicationItem->setHeader(i18n("Application Options"));
applicationItem->setCheckable(false);
applicationItem->setEnabled(false);
m_applicationPage = applicationItem;`
and
// editor widgets from kwrite/kwdialog
m_editorPage = addPage(new QWidget, i18n("Editor Component"));
m_editorPage->setIcon(QIcon::fromTheme(QStringLiteral("accessories-text-editor")));
m_editorPage->setHeader(i18n("Editor Component Options"));
m_editorPage->setCheckable(false);
m_editorPage->setEnabled(false);
By enabling them, they're presented with the same color group as their children, but now their clickable, with no content currently available (see attached screenshot for example).
What do you think: should a default content be added to this top level entries or should we work in distinguishing enabled and clickable behavior (maybe there is already something similar in the frameworks)?
I agree that non-disabled is better readable, but I would like to avoid this empty top level pages. Not sure if there is already something that allows that in the API. We are working on redoing the config dialog: https://invent.kde.org/utilities/kate/-/merge_requests/105 This was fixed by https://invent.kde.org/utilities/kate/-/commit/895160f40b9f55c50a17f7a9d968b952bcac4f50 in Kate 20.12 |
Created attachment 130286 [details] screenshot SUMMARY See the atacched screenshot please. EXPECTED RESULT better readbility SOFTWARE/OS VERSIONS Operating System: Arch Linux KDE Plasma Version: 5.19.3 KDE Frameworks Version: 5.72.0 Qt Version: 5.15.0