ModelObjectWidget changed keyboard and focus behaviour.#781
ModelObjectWidget changed keyboard and focus behaviour.#781RockFordgt wants to merge 1 commit intonullptrlabs:developfrom
Conversation
Switching focus form filter edit to tree/table by pressing arrows up/down Added event filter for tree/table, printed symbols + backspace update filter edit.
|
It makes me happy ti see people sending their pull requests! Thanks :) First, I liked the idea of switching between controls using arrows. But the tab switching isn't enough? Second, as I could understand (don't tested the code yet) the event filter also update the name filter as the user press any printable key, right? If this is it... I'm afraid that this behavior may cause user confusion or even make him annoyed in case of pressing a key by mistake and changing his original filtering. Finally, once I merge the pull request do you mind if I do some code formatting to make it in the standard I use? :) Anyway thank you so much for your work. |
|
Hi, about code formatting: I usually try to use KDE code formatting guidelines it looks like the one of the best "on the market", do you have any link to your code guidelines, I'll may update it and then prepare new patch set. Do you have any mail discussion group, or irc channel, or you are using only github facilities? |
|
Well, I'll test the code later and see how it works. Maybe we can make this improvement be toggled in settings. Honestly, I don't like to use arrow keys to change focus but it can be useful for other users the same is valid for the new behavior for filtering. About code formatting, I don't use a common convention I just format the code the way I find it easy to read. Anyway this is a small detail and can be discussed later.. :) Finally, for practical reasons I use only GitHub's facilities to communicate with users. If you prefer you can contact me via email. |
Switching focus form filter edit to tree/table by pressing arrows up/down key.
Added event filter for tree/table and widget, printed symbols + backspace update filter edit.