336899 2014-06-30 12:01:11 +0000 JJ "activate next/previous layer" shorcut 2017-02-06 13:43:41 +0000 1 1 2 Applications krita Layer Stack unspecified unspecified Linux RESOLVED FIXED junior-jobs NOR wishlist --- 1 kwadraatnope krita-bugs-null dimula73 halla nidhi98gupta 0 oldest_to_newest 1458654 0 kwadraatnope 2014-06-30 12:01:11 +0000 "activate next/previous layer" shorcut already exists,but I propose an alternate version that should also be present. The function as we have it works well,but skips groups entirely when it encounters them. I propose a seperate other function that works like the Qt layerbox shorcut. Select layerbox press up arrow/down arrow ->the layers get selected as expected,without skipping some layers. This does however not work on canvas. So when on canvas the shortcut should also be recognized. 13:27:21 - Bollebib: When you use arrows up and down in layer box it selects like I would expect 13:27:36 - Bollebib: 1 space up and down each time 13:27:36 - Bollebib: this does not seem to be the same as activate next/previous layer which jumps from group to group and ignores group contents when present 13:28:00 - Bollebib: and when on canvas you can't use the arrows up and down as it is then used for nudging the canvas,which is arguably expected,but in that case I would expect those 2 seperate features not share the same shortcuts 13:28:00 - Bollebib: is there a way for me to access those functions and change the shortcuts in the settings? 13:28:23 - boud: Bollebib: aren't they in the shortcuts dialog? 13:28:54 - Bollebib: boud: the only ones I found work as I described,they skip between groups,they don't work the same 13:29:05 - boud: ah, wait 13:29:23 - boud: in the layerbox, arrow up and down isn't a shortcut... it's just what the Qt layerbox provides 13:29:50 - boud: same, probably, for the canvas which is in a scrolled pane, and that scrolls wit hthe arrow keys by default 13:31:44 - Bollebib: boud: is there a way to provide this functionality by just adding a seperate function ,similar to activate next/previous layer,but that works like the QT layerbox shortcut? 13:32:21 - Bollebib: and so that it does work when on canvas? 13:33:25 - boud: Bollebib: yes 1458658 1 kwadraatnope 2014-06-30 12:15:05 +0000 this could then get another name or description obviously 1499845 2 halla 2015-02-14 11:13:38 +0000 Implemented by now. 1499879 3 kwadraatnope 2015-02-14 13:43:37 +0000 Reopened as something was not clear Activate next/previous only activates in the same hierarchy-level I need it to access ALL the levels (like how the QT arrow shortcuts in the layerbox work) 1531380 4 93566 g6793396 2015-07-11 18:32:29 +0000 Created attachment 93566 enter groups with activateNextNode/activatePreviousNode Since this is probably a shortcut used to speed workflow up, skipping the selection of the groups themselves could be desirable. I made a couple of tries at that but it is pretty hard to get right with the layers panel seemingly upside-down, it is easy one way with while (node && node->childCount() && node->lastChild()){ node = node->lastChild(); } but the other way need some recursive stuff. 1572104 5 halla 2016-01-24 14:28:14 +0000 WISHGROUP: Small Usability Fixes Ah, it doesn't actually enter groups yet, Dmitry says. 1638036 6 kwadraatnope 2016-11-06 11:11:42 +0000 right now ,this is actually solved. But it does not go into hidden groups i think even for hidden groups this should work,and go through EACH layer might perhaps be necessary to close this bug and open a new one? 1658184 7 dimula73 2017-02-02 08:04:40 +0000 Hi, Bollebib! Should this shortcut be able to enter *locked* groups as well? 1659065 8 dimula73 2017-02-06 13:43:41 +0000 Fixed in https://phabricator.kde.org/R37:6a909e99b81e 93566 2015-07-11 18:32:29 +0000 2015-07-11 18:32:29 +0000 enter groups with activateNextNode/activatePreviousNode krita-fix-336899 text/plain 898 g6793396 ZGlmZiAtLWdpdCBhL2tyaXRhL3VpL2tpc19ub2RlX21hbmFnZXIuY3BwIGIva3JpdGEvdWkva2lz X25vZGVfbWFuYWdlci5jcHAKaW5kZXggMWVkNjUzZi4uMWY2OGE3MyAxMDA2NDQKLS0tIGEva3Jp dGEvdWkva2lzX25vZGVfbWFuYWdlci5jcHAKKysrIGIva3JpdGEvdWkva2lzX25vZGVfbWFuYWdl ci5jcHAKQEAgLTg3Miw2ICs4NzIsMTEgQEAgdm9pZCBLaXNOb2RlTWFuYWdlcjo6YWN0aXZhdGVO ZXh0Tm9kZSgpCiAKICAgICBLaXNOb2RlU1Agbm9kZSA9IGFjdGl2ZU5vZGUtPm5leHRTaWJsaW5n KCk7CiAKKyAgICAvLyBlbnRlciBncm91cAorICAgIGlmIChub2RlICYmICFub2RlLT5jb2xsYXBz ZWQoKSAmJiBub2RlLT5maXJzdENoaWxkKCkpeworICAgICAgICBub2RlID0gbm9kZS0+Zmlyc3RD aGlsZCgpOworICAgIH0KKwogICAgIGlmICghbm9kZSAmJiBhY3RpdmVOb2RlLT5wYXJlbnQoKSAm JiBhY3RpdmVOb2RlLT5wYXJlbnQoKS0+cGFyZW50KCkpIHsKICAgICAgICAgbm9kZSA9IGFjdGl2 ZU5vZGUtPnBhcmVudCgpOwogICAgIH0KQEAgLTg5Miw2ICs4OTcsMTEgQEAgdm9pZCBLaXNOb2Rl TWFuYWdlcjo6YWN0aXZhdGVQcmV2aW91c05vZGUoKQogCiAgICAgS2lzTm9kZVNQIG5vZGUgPSBh Y3RpdmVOb2RlLT5wcmV2U2libGluZygpOwogCisgICAgLy8gZW50ZXIgZ3JvdXAKKyAgICBpZiAo bm9kZSAmJiAhYWN0aXZlTm9kZS0+Y29sbGFwc2VkKCkgJiYgYWN0aXZlTm9kZS0+bGFzdENoaWxk KCkpeworICAgICAgICBub2RlID0gYWN0aXZlTm9kZS0+bGFzdENoaWxkKCk7CisgICAgfQorCiAg ICAgaWYgKCFub2RlICYmIGFjdGl2ZU5vZGUtPnBhcmVudCgpKSB7CiAgICAgICAgIG5vZGUgPSBh Y3RpdmVOb2RlLT5wYXJlbnQoKS0+cHJldlNpYmxpbmcoKTsKICAgICB9Cg==