Skip to content
Commit 8dd8fe55 authored by Alexander Lohnau's avatar Alexander Lohnau Committed by Harald Sitter
Browse files

characters: Refactoring, improve validation in kcm, optimize

Summary:
The config keys are now in a separate file, instead of having them at two places.
Runner
The duplicate call to reloadConfiguration has been removed and aliases/codes get validated.
Additionally the match method has been simplified and the character is now copied to the clipboard when selected.
KCM
Validation of the add/delete buttons, error message now in GUI and deprecated methods removed.

Test Plan:
Compile, test runner and test editing entries.
Manually add alias(and no hex value) to ~/.config/krunnerrc and reload the kcm, error message should be shown.

Before:
{F8075415}
{F8075416}

After:
{F8075419}
{F8075420}
{F8075422}

Reviewers: davidedmundson, ngraham, sitter, broulik, #plasma

Reviewed By: sitter

Subscribers: davidre, dvratil, plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D27096
parent f07e3a54
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment