D12874: Piano_composition, add an indicator to highlight the currently selected note-type, minor refactoring and indentation corrections.

Aman Kumar Gupta noreply at phabricator.kde.org
Mon May 14 17:01:35 UTC 2018


amankumargupta marked 2 inline comments as done.
amankumargupta added inline comments.

INLINE COMMENTS

> jjazeix wrote in Piano_composition.qml:209
> why setting 0 and not just setting "visible: bar.level == 1 || bar.level == 2 ? false : true" here directly?

ahh, that's very silly of me :p Thanks! Corrected in the latest diff.

> jjazeix wrote in Piano_composition.qml:252
> I'm not sure why I used staff2, but it can probably be renamed :)

Renamed to multipleStaff

REPOSITORY
  R2 GCompris

REVISION DETAIL
  https://phabricator.kde.org/D12874

To: amankumargupta, #gcompris_improvements
Cc: jjazeix, kde-edu, #gcompris_improvements, himanshuvishwakarma, harrymecwan, ganeshredcobra, nityanandkumar, echarruau, rahulyadav, narvaez, scagarwal, apol, timotheegiet, hkaelberer, bcoudoin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20180514/0fe7c90a/attachment.html>


More information about the kde-edu mailing list