Skip to content

Commit 1a3a949

Browse files
authored
Fix GNSS status icon when fix is attained without sat info (#4461)
Fix GNSS status when fix is attained without sat info
1 parent 27791d9 commit 1a3a949

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

src/js/sensor_helpers.js

+8-10
Original file line numberDiff line numberDiff line change
@@ -76,19 +76,17 @@ export function sensor_status(sensors_detected = 0, gps_fix_state = 0) {
7676
$(".magicon", eSensorStatus).removeClass("active");
7777
}
7878

79-
if (have_sensor(sensors_detected, "gps")) {
79+
const gnssSensorDetected = have_sensor(sensors_detected, "gps");
80+
const hasGnssFix = gps_fix_state > 0;
81+
82+
if (gnssSensorDetected || hasGnssFix) {
8083
$(".gps", eSensorStatus).addClass("on");
81-
if (gps_fix_state) {
82-
$(".gpsicon", eSensorStatus).removeClass("active");
83-
$(".gpsicon", eSensorStatus).addClass("active_fix");
84-
} else {
85-
$(".gpsicon", eSensorStatus).removeClass("active_fix");
86-
$(".gpsicon", eSensorStatus).addClass("active");
87-
}
84+
$(".gpsicon", eSensorStatus)
85+
.toggleClass("active", gnssSensorDetected && !hasGnssFix)
86+
.toggleClass("active_fix", gnssSensorDetected && hasGnssFix);
8887
} else {
8988
$(".gps", eSensorStatus).removeClass("on");
90-
$(".gpsicon", eSensorStatus).removeClass("active");
91-
$(".gpsicon", eSensorStatus).removeClass("active_fix");
89+
$(".gpsicon", eSensorStatus).removeClass("active active_fix");
9290
}
9391

9492
if (have_sensor(sensors_detected, "sonar")) {

0 commit comments

Comments
 (0)