Code review comment for lp:~feng-kylin/unity8/fix-lp1431566

Revision history for this message
Michael Zanetti (mzanetti) wrote :

Thanks for the contribution. Unfortunately this is not really how it should be done. "usageModeSettings" is not part of Panel.qml. It is bad practice to reach out of context like this and will cause a mess in the long run.

Also, there are more issues with those buttons and I already have a branch upcoming that improves the situation. I'm very sorry. I should have marked the bug as "in progress".

« Back to merge proposal