Skip to content

Commit

Permalink
PV inverter: make meaning of limiting option clear
Browse files Browse the repository at this point in the history
Add some text to indicate what the limiter option is for. It
is so a user can disable it where it interferes with normal
operation.

Also, hide the limiter option if the PV-inverter is not to
be shown to the user, ie if it belongs to another installation
and is not used here.
  • Loading branch information
izak committed Oct 24, 2024
1 parent e836be6 commit eae55f7
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion pages/settings/PageSettingsFroniusInverter.qml
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,7 @@ Page {
}

ListRadioButtonGroupNoYes {
id: isActive
//% "Show"
text: qsTrId("page_settings_fronius_inverter_show")
dataItem.uid: bindPrefix + "/IsActive"
Expand All @@ -63,11 +64,17 @@ Page {
//% "Power limiting"
text: qsTrId("page_settings_fronius_inverter_power_limiting")
dataItem.uid: bindPrefix + "/EnableLimiter"
allowed: limiterSupportedItem.value === 1
allowed: isActive.dataItem.value === 1 && limiterSupportedItem.value === 1
optionModel: [
{ display: CommonWords.disabled, value: 0 },
{ display: CommonWords.enabled, value: 1 }
]
bottomContentChildren: [
ListLabel {
//% "This PV inverter has support for power limiting. Disable this setting if it interferes with normal operation."
text: qsTrId("page_settings_fronius_inverter_power_limiting_label")
}
]
}
}
}
Expand Down

0 comments on commit eae55f7

Please sign in to comment.