on xenial help sign in
add a quirk for the battery indicator
commit5540a5030b16e201c546108b3cc399efe7d6df09 authorLukáš Tinkl <lukas.tinkl@canonical.com> dateMon 16 Nov 2015, 15:19 (11 years ago) parentf698e80
1 file changed+10−2
plugins/Unity/Indicators/indicatorsmanager.cpp+10−2
plugins/Unity/Indicators/indicatorsmanager.cpp+10−2
@@ -286,11 +286,19 @@ Indicator::Ptr IndicatorsManager::indicator(const QString& indicator_name)
286286 data->m_indicator = new_indicator;
287287 QSettings settings(data->m_fileInfo.absoluteFilePath(), QSettings::IniFormat, this);
288288 new_indicator->init(data->m_fileInfo.fileName(), settings);
289− if (new_indicator->identifier() == QStringLiteral("indicator-session") && m_platform.isMultiSession()) {
290− new_indicator->setProfile("desktop"); // convergence: enable session indicator conditionally
289+
290+ // convergence:
291+ // 1) enable session indicator conditionally, typically when running on desktop
292+ // 2) switch the battery/power indicator to desktop mode, can't control brightness for now and phone-on-desktop broken (FIXME)
293+ //
294+ // The rest of the indicators respect their default profile (which is "phone", even on desktop PCs)
295+ if ((new_indicator->identifier() == QStringLiteral("indicator-session") && m_platform.isMultiSession())
296+ || (new_indicator->identifier() == QStringLiteral("indicator-power") && m_platform.isPC())) {
297+ new_indicator->setProfile("desktop");
291298 } else {
292299 new_indicator->setProfile(m_profile);
293300 }
301+
294302 QObject::connect(this, &IndicatorsManager::profileChanged, new_indicator.data(), &Indicator::setProfile);
295303 return new_indicator;
296304}