ubuntu-touch-coreapps-reviewers team mailing list archive
-
ubuntu-touch-coreapps-reviewers team
-
Mailing list archive
-
Message #06295
Re: [Merge] lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-remember-last-formula into lp:ubuntu-calculator-app
Hi Bartosz,
thanks for working on this.
Unfortunately, I don't think this is the right way to fix the problem: Qt.labs.settings writes on a file the setting, so you add a very big overhead: every time user presses a button there is an I/O operation, that is very expensive.
IMO the right direction to fix this bug is saving the actual formula only when app is closed (using Component.onDestruction) and when it losts focus (creating a bind on Qt.application.state[0]).
Let me know what you think.
[0]: http://doc.qt.io/qt-5/qml-qtqml-qt.html#application-prop
--
https://code.launchpad.net/~gang65/ubuntu-calculator-app/ubuntu-calculator-app-remember-last-formula/+merge/278900
Your team Ubuntu Calculator Developers is requested to review the proposed merge of lp:~gang65/ubuntu-calculator-app/ubuntu-calculator-app-remember-last-formula into lp:ubuntu-calculator-app.
References