These are my code Peman, And has been merged from master, somehow. (Original code is not in this screen and are in SettingsHome.qml) As usual it helps to immediately identify that toFixed() function has not been forgotten and been used to set decimal point by 2 digits.
Please see my above and linked to by Behrouz. To avoid future merge conflicts, I think it'd be best to ensure the styling issues are fixed only on Behrouz's branch instead of here too. If you agree, I think this and the above comment can be resolved.
Why these two are not defined in the message list spreadsheet? Where is the definition? Please collaborate with FW Team to have correct length here even if it has not been defined it needs a definition to have a placeholder.
There was no Fall through in the original code. Please, do not remove codes even if it's the same code. Even if we are going to (which we are not) have fall through there should be Q_FALLTHROUGH(); in the code as has been stated in the C++ coding standard.
we are not handling timeouts and timers in application. It makes application and HD out of sync. HD will send the times if required each second. That's how it has been discussed and designed. There should be a message later which has the timeout as a data and UI only updates itself by that value. If it's not for animation, please remove it.