<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>316058</bug_id>
          
          <creation_ts>2013-03-03 03:57:11 +0000</creation_ts>
          <short_desc>Keyboard navigation (Tab and arrows) does not work in the profile form</short_desc>
          <delta_ts>2019-07-04 21:06:18 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>ktouch</product>
          <component>general</component>
          <version>2.0.0</version>
          <rep_platform>Other</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>major</bug_severity>
          <target_milestone>---</target_milestone>
          
          
          <everconfirmed>1</everconfirmed>
          <reporter name="Cristian Tibirna">tibirna</reporter>
          <assigned_to name="Sebastian Gottfried">sebastian.gottfried</assigned_to>
          <cc>cpigat242</cc>
          
          <cf_commitlink>https://commits.kde.org/ktouch/106d3bbe772568f82d758da7c74d5089fdb55120</cf_commitlink>
          <cf_versionfixedin>19.08.0</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1346952</commentid>
    <comment_count>0</comment_count>
    <who name="Cristian Tibirna">tibirna</who>
    <bug_when>2013-03-03 03:57:11 +0000</bug_when>
    <thetext>KTouch 2.0 starts, at first use, by displaying the profile form. The name textedit does not have focus and keyboard navigation doesn&apos;t work. No Tab or arrow keys can be used to change to the 4 inputs the user has to access before quitting this form. Mouse is mandatory.

Same goes for adding a new profile (action which displays the same profile form, in another window layout).

Reproducible: Always

Steps to Reproduce:
1. Open KTouch
2. If not opened for the first time, click on the username button at the left top corner of the window that appeared
3. try to select the name textedit or the radio buttons without the mouse
Actual Results:  
Nothing happens

Expected Results:  
Focus should be changeable with the keyboard.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1346955</commentid>
    <comment_count>1</comment_count>
      <attachid>77705</attachid>
    <who name="Cristian Tibirna">tibirna</who>
    <bug_when>2013-03-03 04:06:56 +0000</bug_when>
    <thetext>Created attachment 77705
And here is a patch.

It is only tentative, my local sources are still compiling.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1347540</commentid>
    <comment_count>2</comment_count>
    <who name="Sebastian Gottfried">sebastian.gottfried</who>
    <bug_when>2013-03-04 10:11:40 +0000</bug_when>
    <thetext>Thanks for the patch. I wasn&apos;t even aware that implementing tab navigation is that simple to implement in QML. Somehow I have managed to overlook the existence of the KeyNavigation element for all the time I&apos;m developing with Qt Quick ;)</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1355201</commentid>
    <comment_count>3</comment_count>
    <who name="Sebastian Gottfried">sebastian.gottfried</who>
    <bug_when>2013-03-28 17:15:37 +0000</bug_when>
    <thetext>Focus handling in Qt Quick is a buggy mess right now. I have come so far to implement keyboard navigation for the complete application provided the component in question has already the focus. Due to the dysfunctional state of the FocusScope [0] element in Qt Quick 1.1 under Qt 4.8 managing focus in non-trivial application is pretty much impossible for now. It seems I have to port to Qt 5 first in order to resolve this issue, so this ticket is going to stay open for a while.

[0]: https://bugreports.qt-project.org/browse/QTBUG-28288</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1867239</commentid>
    <comment_count>4</comment_count>
    <who name="Sebastian Gottfried">sebastian.gottfried</who>
    <bug_when>2019-07-04 21:06:18 +0000</bug_when>
    <thetext>Git commit 106d3bbe772568f82d758da7c74d5089fdb55120 by Sebastian Gottfried.
Committed on 04/07/2019 at 21:02.
Pushed by gottfried into branch &apos;master&apos;.

Merge Branch &apos;homescreen-ng&apos;

This replaces the home screen to solve KTouch&apos;s most egrigous problem:
content discovery. Previously, KTouch displayed only the courses for the
the currently active keyboard layout. If KTouch has no course for this
layout, the windows was basically empty showing only the option to
create a new custom lesson. Being forced to set up first the correct
keyboad layout for your computer has proven to be unintuitive. The
new home screen shows all available training content at a glance.

Improvements in more detail:

 * There is a new sidebar allowing the user to select from all available
   keyboard layouts with their associated courses. Users still have to
   set up the correct layout matching the selected course, though, and
   they are told how to do so with an passive notification. (BUG 395264,
   BUG 385425, BUG 331368, BUG 398773, BUG 391968, BUG 364011)

 * Training statistics are now available from the home screen for each
   lesson. Previously they have been only shown on the score screen
   after completing a lesson. (BUG 401359, BUG 317174)

 * The lesson selector has been redesign to show previews of each lesson
   as selectable items in a grid view.

  * Tab focus works now as expected and allows training without using the
    mouse at all. (BUG 316058)

  * Support for HiDPI rendering has been added.

  * KTouch now uses Qt Quick Controls 2 instead of version 1.
Related: bug 385425, bug 401359, bug 331368, bug 398773, bug 391968, bug 364011, bug 395264, bug 317174
FIXED-IN: 19.08.0
Differential Revision: https://phabricator.kde.org/D20953

M  +3    -4    CMakeLists.txt
M  +7    -2    src/CMakeLists.txt
M  +2    -2    src/declarativeitems/lessonpainter.h
M  +13   -12   src/qml/homescreen/ProfileDetailsItem.qml
M  +4    -4    src/qml/keyboard/KeyItem.qml
M  +1    -1    src/qml/keyboard/Keyboard.qml
M  +2    -2    src/qml/keyboard/KeyboardLayoutEditor.qml
M  +94   -208  src/qml/scorescreen/ScoreScreen.qml
M  +3    -3    src/qml/trainingscreen/TrainingWidget.qml

https://commits.kde.org/ktouch/106d3bbe772568f82d758da7c74d5089fdb55120</thetext>
  </long_desc>
      
          <attachment
              isobsolete="0"
              ispatch="1"
              isprivate="0"
          >
            <attachid>77705</attachid>
            <date>2013-03-03 04:06:56 +0000</date>
            <delta_ts>2013-03-03 04:06:56 +0000</delta_ts>
            <desc>And here is a patch.</desc>
            <filename>ProfileForm_keynav.patch</filename>
            <type>text/plain</type>
            <size>1035</size>
            <attacher name="Cristian Tibirna">tibirna</attacher>
            
              <data encoding="base64">ZGlmZiAtLWdpdCBhL3NyYy9xbWwvUHJvZmlsZUZvcm0ucW1sIGIvc3JjL3FtbC9Qcm9maWxlRm9y
bS5xbWwKaW5kZXggMmE4NDMyMS4uMDFmZDE4MyAxMDA2NDQKLS0tIGEvc3JjL3FtbC9Qcm9maWxl
Rm9ybS5xbWwKKysrIGIvc3JjL3FtbC9Qcm9maWxlRm9ybS5xbWwKQEAgLTQ0LDYgKzQ0LDkgQEAg
Q29sdW1uIHsKICAgICAgICAgaWQ6IG5hbWVUZXh0RmllbGQKICAgICAgICAgd2lkdGg6IHBhcmVu
dC53aWR0aAogICAgICAgICBwbGFjZWhvbGRlclRleHQ6IGkxOG4oIk5hbWUiKQorICAgICAgICBm
b2N1czogdHJ1ZQorICAgICAgICBLZXlOYXZpZ2F0aW9uLnRhYjogYmVnaW5uZXJSYWRpb0JveAor
ICAgICAgICBLZXlOYXZpZ2F0aW9uLmJhY2t0YWI6IGRvbmVCdG4KICAgICB9CiAKICAgICBQbGFz
bWFDb21wb25lbnRzLlJhZGlvQnV0dG9uIHsKQEAgLTYxLDYgKzY0LDggQEAgQ29sdW1uIHsKICAg
ICAgICAgY2hlY2tlZDogcm9vdC5za2lsbExldmVsID09PSBQcm9maWxlLkJlZ2lubmVyCiAgICAg
ICAgIHdpZHRoOiBwYXJlbnQud2lkdGgKICAgICAgICAgdGV4dDogaTE4bigiSSBoYXZlIG5vIG9y
IG9ubHkgdmVyeSBsaXR0bGUgZXhwZXJpZW5jZSBpbiBtYWNoaW5lIHR5cGluZyIpCisgICAgICAg
IEtleU5hdmlnYXRpb24udGFiOiBhZHZhbmNlZFJhZGlvQm94CisgICAgICAgIEtleU5hdmlnYXRp
b24uYmFja3RhYjogbmFtZVRleHRGaWVsZAogICAgIH0KIAogICAgIFJvdyB7CkBAIC05NSw2ICsx
MDAsOCBAQCBDb2x1bW4gewogICAgICAgICBjaGVja2VkOiByb290LnNraWxsTGV2ZWwgPT09IFBy
b2ZpbGUuQWR2YW5jZWQKICAgICAgICAgd2lkdGg6IHBhcmVudC53aWR0aAogICAgICAgICB0ZXh0
OiBpMThuKCJJIGFtIGFuIGV4cGVyaWVuY2VkIG1hY2hpbmUgdHlwaXN0IGFuZCB3YW50IHRvIGlt
cHJvdmUgbXkgc2tpbGxzIikKKyAgICAgICAgS2V5TmF2aWdhdGlvbi50YWI6IGJlZ2lubmVyUmFk
aW9Cb3gKKyAgICAgICAgS2V5TmF2aWdhdGlvbi5iYWNrdGFiOiBkb25lQnRuCiAgICAgfQogCiAg
ICAgUm93IHsK
</data>

          </attachment>
      

    </bug>

</bugzilla>