- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
May 27 2020
Mar 2 2020
Feb 28 2020
Feb 20 2020
Nov 9 2019
Nov 4 2019
Oct 10 2019
Aug 22 2019
Aug 14 2019
Aug 13 2019
Aug 12 2019
Thanks again, I have already pushed you original suggestions, see https://github.com/KDE/ktouch/commit/799d2d1a1619caef5787b91a5cbb92a1dd8d6b7a
In D23048#510825, @manuelal wrote:This (and the changes here) should go into the Applications/19.08 branch
I'm new to contribute to KDE. Is this done automatically or I need to do something else?
With Phabricator, an automatic merge is not possible, unfortunately. It can only do code review.
Thanks for the update!
Aug 8 2019
Aug 7 2019
Sorry for the long wait. I discovered a crash in the keyboard layout editor which I suspected to be related to you work, but it is not. Still, I wanted to figure this out first.
Aug 6 2019
I found two more spots which leak connections over time. These should be fixed.
Aug 5 2019
Looks mostly good and works as intended. But its seems you have missed the member declarations and include directives:
Aug 3 2019
Jul 16 2019
Jul 14 2019
Jul 13 2019
Jul 11 2019
Thanks for the patch. If you change the dependencies in the CMake files, you also need to update the README so they stay in sync.
Jul 10 2019
In D22313#493883, @asturmlechner wrote:Okay then... I do have commit access, fyi.
I took your patch and fixed the remaining issues. I think, this review can be closed now.
Jul 9 2019
In D22313#492604, @aacid wrote:@gottfried any reason you removed kde edu from the list of subscribers?
Jul 8 2019
Jul 4 2019
Jun 27 2019
If no one objects within a week, I am going the merge this.
Jun 21 2019
Jun 20 2019
Jun 19 2019
Jun 17 2019
In D20953#477062, @aacid wrote:There's a few relative scary warnings though
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:367:25: QML LearningProgressChart: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:286:21: QML Label: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:293:21: QML ComboBox: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:306:21: QML Label: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:316:21: QML ComboBox: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:337:21: QML LegendItem: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:345:21: QML LegendItem: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/scorescreen/ScoreScreen.qml:162:17: QML IconToolButton: Detected anchors on an item that is managed by a layout. This is undefined behavior; use Layout.alignment instead.
qrc:/ktouch/qml/homescreen/ProfileSelector.qml:90: TypeError: Cannot read property 'isNewButton' of nullhave you seen this?
I don't see the layout warnings. Maybe you Qt is more recent than mine. Nonetheless, they should all be fixed now.
Fix QML runtime warnings.
May 4 2019
In D20953#460428, @ognarb wrote:In D20953#459771, @gottfried wrote:In D20953#459468, @ognarb wrote:Hi, not a KTouch dev, but I found a typo. ;)
Yay, I'm basically the only one developing features for KTouch. I still value review for non-trivial changes, though.
:) So I tried it on my machine on it worked very well. :) Can't really comment about the code but found the sidebar shadow a bit strange. The shadow gave me the wrong impression that the sidebar is collapsible. Maybe you could ask the VDG for feedback. Otherwise, the UI looks great.
Thank you. I can see your point. I have added the shadow as visual separator between the sidebar and the lesson grid. I agree its a bit misleading. Maybe the VDG has a better idea.
May 2 2019
In D20953#459468, @ognarb wrote:Hi, not a KTouch dev, but I found a typo. ;)
Yay, I'm basically the only one developing features for KTouch. I still value review for non-trivial changes, though.
Fix Typo in checkqmlimports.py
Apr 28 2019
Works as advertised.
Apr 11 2019
Mar 9 2019
Feb 24 2019
I have pushed the changes to the KTouch Git repository.
Feb 16 2019
Jan 2 2019
I have added your course to KTouch, see https://cgit.kde.org/ktouch.git/commit/?id=8598e4af105418a92d54f5dbfbbaf04be9a492d7. It will be part of the upcoming 18.04 release.
Alright, I will wait.
Hi Simone,