D17377: keyboard binding support to "Money" based activities in gcompis

Smit Patil noreply at phabricator.kde.org
Thu Dec 6 09:13:37 GMT 2018


smitpatil added inline comments.

INLINE COMMENTS

> amankumargupta wrote in MoneyCore.qml:338
> Since you were already using if-else statements above, can you maintain the consistency by replacing the switch case statements with if-else?

I first tried if-else but that took a lot of lines. Thus, I find switch more consistent than if-else. still, if you want, I will implement it.

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

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


More information about the kde-edu mailing list