@trmdi can you confirm me the following for your code ?

1. When a layout is loaded for first time then "global shortcuts based on position" are applied based on Latte set priority for which view to be forwarded
2. When the user changed the isPreferredForShortcuts for first time afterwards [1] is never used again EXCEPT if the user set the property

preferredForShortcutsTouched=false

on its own

Is this what you wanted to achieve?

Note: this is important, the preferredForShortcutsTouched property must be moved from universalsettings to layout

In D18717#404861, @mvourlakos wrote:

@trmdi can you confirm me the following for your code ?

1. When a layout is loaded for first time then "global shortcuts based on position" are applied based on Latte set priority for which view to be forwarded
2. When the user changed the isPreferredForShortcuts for first time afterwards [1] is never used again EXCEPT if the user set the property ` preferredForShortcutsTouched=false `
   
   on its own

Is this what you wanted to achieve?

Note: this is important, the preferredForShortcutsTouched property must be moved from universalsettings to layout

My code works like this:

1, if the user has never touched the Behavior > Activate... checkbox, Latte would automatically find the best view. (Not only the first time)
Yes, this should be moved to the layout class.

2, once he touched it, latte would remember that, and follow his setting.

What do you think?

In D18717#404866, @trmdi wrote:

In D18717#404861, @mvourlakos wrote:

What do you think?

it looks and behave nice, lets see with Layout how it will look but I believe there will be no issues,
we are ok to commit if the property is moved to Layout

In D18717#405084, @mvourlakos wrote:

it looks and behave nice, lets see with Layout how it will look but I believe there will be no issues,
we are ok to commit if the property is moved to Layout

I've moved it to Layout, and add a small improvement to the "Behavior> Activate based on position... " checkbox's checked value.
Please test it and improve it if needed.

1, please make a check that sortedViewsList returns count()>0 and if it doesnt it can return a nullptr
-> done

2, your turn

In D18717#406296, @mvourlakos wrote:

@trmdi would you like to commit it for you?
Can I use your fullname shown at: https://phabricator.kde.org/p/trmdi/ ?
and what is your email?

Yes, I don't have the commit permission.
Yes, that name, trmdi@yandex.com.

Thank you! :)