[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-release-team
Subject: Aggressively fixing Flatpak KCM
From: ivan tkachenko <me () ratijas ! tk>
Date: 2023-03-22 16:41:54
Message-ID: 48bbaf6e-9281-c7e5-724e-7e41c3bf4de9 () ratijas ! tk
[Download RAW message or body]
[Attachment #2 (multipart/mixed)]
[Attachment #4 (text/plain)]
Hi,
I'm working on Flatpak KCM.
This is all very broken and we can't do conservative fixes because they
won't
apply well between master and Plasma/5.27 branches if done little-by-little,
and because the thing is fundamentally broken on multiple levels. I managed
to refactor and rename members, methods and model roles, so they at least
make sense and don't duplicate each other too much. I also started writing
tests and expanding test suite on newly found broken cases and introduced
API. But this is not enough, and on my roadmap there are things like:
- Replacing all section headings from partially(!) translated strings
strings+enum heuristic into a single (and exposed to QML) enum;
- Rewriting value lists for ComboBox'es from pre-translated QStringList into
model with text and value roles;
- Rewriting all model setters (both toggling and changing string values)
from
storing override data in manually formatted ini/KConfig style string into
storing proper data structures and value identifiers inside individual
entries and serializing them in a centralized manner;
- Reading "defaults" not just from metadata, but from other three override
levels not writable by KCM (system-default, system per-app,
user-default);
- Fixing model signals, like using beginInsertRows/endInsertRows instead of
just dataChanged approriately;
- Changing UI styling for entries that are marked for removal, such as
unchecked non-default filesystems, D-Bus services & prefixes, and
environment variables
- Implementing default settings highlighter.
Can I please have permission to do aggressive cherry picking from master to
make it better?
--
Best,
ivan (@ratijas) tkachenko
["OpenPGP_signature.asc" (application/pgp-signature)]
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic