Quantcast
Channel: PowerDevil activity
Viewing all articles
Browse latest Browse all 1684

Jakob Petsovits opened merge request !339: kcmodule: Revert to binding curren...

$
0
0

Thanks to @broulik for finding this bug, and (in advance) for testing this fix on his system. What happened was that I tested the original patch while on battery, which apparently added just enough time for the models to initialize. While plugged in though, the KCM starts with the "On AC Power" tab and that one initialized its comboboxes incorrectly.


Commit 8ecf6430(MR !337) caused empty ComboBox selections on initial load.Turns out indexOfValue() can only be used after theComponent.completed() signal is emitted, so a purely declarativebinding worked incorrectly if queried too early by QML.

Therefore, we revert back to assigning a Qt.binding() tocurrentIndex in Component.onCompleted.

This commit also adapts PowerProfileModel and its associatedComboBox in a similar way as the other two models used in this KCM.


Viewing all articles
Browse latest Browse all 1684

Trending Articles