diff options
| author | Mavridis Philippe <[email protected]> | 2023-02-10 14:57:28 +0200 |
|---|---|---|
| committer | Mavridis Philippe <[email protected]> | 2023-03-24 15:28:11 +0200 |
| commit | e8208c1dfb4dcd17b1168ac2614aa8d5b761f3fd (patch) | |
| tree | 26a597b2a9a11e9cab352940be0d687ea6593668 /kxkb/layoutmap.cpp | |
| parent | a67db2d4847d798c01d4fd7584c5bb9297e109e3 (diff) | |
| download | tdebase-e8208c1dfb4dcd17b1168ac2614aa8d5b761f3fd.tar.gz tdebase-e8208c1dfb4dcd17b1168ac2614aa8d5b761f3fd.zip | |
Kxkb: improve TDE Control Centre module.
1) Add conflicts check for hotkey checkboxes
Due to the ability to set multiple keyboard layout switching hotkeys,
it is important to inform the user of conflicting options that are
not handled properly by the X.org server when set at the same time
(e.g. Win+Space and Alt+Space).
This change adds a warning that informs the user about the problem and
the conflicting options. This warning is shown only when setting
multiple hotkeys via the Xkb options tab, which is for the advanced
user. Most users will ever need only one hotkey, and the combobox on
the first tab should be more than enough.
2) Add "none" item to layout switching options
3) Replace Reset old options checkbox with radio buttons
As per discussion, this makes the function of the option more
apparent. WhatIs hints have been added for additional clarity.
4) Update hotkey combobox per server options
5) Avoid duplication of options by querying Xkb for already set options.
This was a problem in Append Mode in which `setxkbmap` strings would
get too long due to setting already set options. This code checks for
already set options and omits them from the new `setxkbmap` call.
This does not apply to Overwrite Mode.
6) Overwrite previous grp: options when using the combobox
See previous commit message about the addition of hotkeys combobox.
Signed-off-by: Mavridis Philippe <[email protected]>
Diffstat (limited to 'kxkb/layoutmap.cpp')
| -rw-r--r-- | kxkb/layoutmap.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/kxkb/layoutmap.cpp b/kxkb/layoutmap.cpp index 7b5136e2a..636911199 100644 --- a/kxkb/layoutmap.cpp +++ b/kxkb/layoutmap.cpp @@ -88,7 +88,7 @@ LayoutState& LayoutMap::getNextLayout() { layoutQueue.enqueue(layoutState); kdDebug() << "map: Next layout: " << layoutQueue.head()->layoutUnit.toPair() - << " for " << m_currentWinId << endl; + << " for " << m_currentWinId << endl; return *layoutQueue.head(); } @@ -96,7 +96,7 @@ LayoutState& LayoutMap::getNextLayout() { void LayoutMap::setCurrentLayout(const LayoutUnit& layoutUnit) { LayoutQueue& layoutQueue = getCurrentLayoutQueue(m_currentWinId); kdDebug() << "map: Storing layout: " << layoutUnit.toPair() - << " for " << m_currentWinId << endl; + << " for " << m_currentWinId << endl; int queueSize = (int)layoutQueue.count(); for(int ii=0; ii<queueSize; ii++) { |