*** Bug 377149 has been marked as a duplicate of this bug. ***

This bug still happens on Plasma 5.10 with Frameworks 5.35.

The bug still occurs on Plasma 5.10.5 and KDE Frameworks 5.37.

There is no need to post every 2 months that this is still not implemented. Once we implement it the bug will be updated :-)

Sorry, I thought it was already implemented and that I was encountering a bug.

this bug affects plasma 5.12 beta on Arch Linux.

still not working in plasma 5.13 beta, Arch Linux.

Still valid for plasma 5.14 beta on Arch Linux.

There is no need to post every 2 months that this is still not implemented. Once we implement it the bug will be updated :-)

Created attachment 115640 [details]
Possible patch

This is a possible patch for the feature. Unfortunately I only have systems without num block, so I cannot test the patch.

Please try and report back whether the patch implements it correctly.

(In reply to Martin Flöser from comment #10)
> Created attachment 115640 [details]
> Possible patch
> 
> This is a possible patch for the feature. Unfortunately I only have systems
> without num block, so I cannot test the patch.
> 
> Please try and report back whether the patch implements it correctly.

I tried it with Plasma 5.13.5 and it doesn't work.

Is there anything you want me to try to try to debug this patch out?

(In reply to Andrius Štikonas from comment #11)
> (In reply to Martin Flöser from comment #10)
> > Created attachment 115640 [details]
> > Possible patch
> > 
> > This is a possible patch for the feature. Unfortunately I only have systems
> > without num block, so I cannot test the patch.
> > 
> > Please try and report back whether the patch implements it correctly.
> 
> I tried it with Plasma 5.13.5 and it doesn't work.
> 
> Is there anything you want me to try to try to debug this patch out?

no, not really. I probably have to check into xkbcommon code to understand better what I need to do, to set the modifier.

I think I found the reason: I'm reading the config value from the wrong file.

Created attachment 115644 [details]
Try 2

(In reply to Martin Flöser from comment #14)
> Created attachment 115644 [details]
> Try 2

Still nothing :(

Just to make sure: you have the config option to enable numlock set, right?

(In reply to Martin Flöser from comment #16)
> Just to make sure: you have the config option to enable numlock set, right?

Yes, of course. In kcminputrc, I have [Keyboard] section which contains NumLock=0.

And your patch is checking for 0 there.

Created attachment 115900 [details]
Try 3

I added a unit test to investigate why my patch doesn't work and found a bug. I'm quite confident that it works now.

(In reply to Martin Flöser from comment #18)
> Created attachment 115900 [details]
> Try 3
> 
> I added a unit test to investigate why my patch doesn't work and found a
> bug. I'm quite confident that it works now.

Third time's a charm.

Thanks a lot!

Git commit 7c3a851b040e3b7d33186ce787c43683ec076525 by Martin Flöser.
Committed on 01/11/2018 at 16:49.
Pushed by graesslin into branch 'master'.

Support enabling numlock on startup

Summary:
This change brings improved num lock support to KWin. The modifier state
is read and also mapped to Qt::KeyboardModifiers. Furthermore the input
config is read and the NumLock key is evaluated. If the requested state
does not match the current num lock state the state is swapped.
FIXED-IN: 5.15

Test Plan: New unit test added, no manual test due to lack of hardware

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D16428

M  +43   -0    autotests/integration/keyboard_layout_test.cpp
M  +1    -0    keyboard_input.cpp
M  +28   -0    xkb.cpp
M  +11   -0    xkb.h

https://commits.kde.org/kwin/7c3a851b040e3b7d33186ce787c43683ec076525

I can confirm that it works great now on Plasma 5.15 beta !