diff --git a/ChangeLog.md b/ChangeLog.md
index 869c41fe..1c43e020 100644
--- a/ChangeLog.md
+++ b/ChangeLog.md
@@ -1,5 +1,6 @@
#2.3.5
* Ensure that hidden blueprint effects are applied when a blueprint is selected
+ * Handle display when summary values show thrusters disabled but current mass keeps them enabled
#2.3.4
* Fix crash when removing the special effect from a module
diff --git a/src/app/components/Movement.jsx b/src/app/components/Movement.jsx
index b8560db4..65b3582e 100644
--- a/src/app/components/Movement.jsx
+++ b/src/app/components/Movement.jsx
@@ -58,13 +58,13 @@ export default class Movement extends TranslatedComponent {
// Speed
- {formats.int(ship.calcSpeed(eng, fuel, cargo, boost))}m/s
+ {ship.canThrust(cargo, fuel) ? formats.int(ship.calcSpeed(eng, fuel, cargo, boost)) + 'm/s' : '-'}
// Pitch
- {formats.int(ship.calcPitch(eng, fuel, cargo, boost))}°/s
+ {ship.canThrust(cargo, fuel) ? formats.int(ship.calcPitch(eng, fuel, cargo, boost)) + '°/s' : '-'}
// Roll
- {formats.int(ship.calcRoll(eng, fuel, cargo, boost))}°/s
+ {ship.canThrust(cargo, fuel) ? formats.int(ship.calcRoll(eng, fuel, cargo, boost)) + '°/s' : '-'}
// Yaw
- {formats.int(ship.calcYaw(eng, fuel, cargo, boost))}°/s
+ {ship.canThrust(cargo, fuel) ? formats.int(ship.calcYaw(eng, fuel, cargo, boost)) + '°/s' : '-'}
);
}
diff --git a/src/app/components/ShipSummaryTable.jsx b/src/app/components/ShipSummaryTable.jsx
index 263302f8..41deed29 100644
--- a/src/app/components/ShipSummaryTable.jsx
+++ b/src/app/components/ShipSummaryTable.jsx
@@ -34,9 +34,9 @@ export default class ShipSummaryTable extends TranslatedComponent {
const sgClassNames = cn({ warning: shieldGenerator && !ship.shield, muted: !shieldGenerator });
const sgTooltip = shieldGenerator ? 'TT_SUMMARY_SHIELDS' : 'TT_SUMMARY_SHIELDS_NONFUNCTIONAL';
const timeToDrain = Calc.timeToDrainWep(ship, 4);
- const canThrust = ship.canThrust(cargo, fuel);
+ const canThrust = ship.canThrust(cargo, ship.fuelCapacity);
const speedTooltip = canThrust ? 'TT_SUMMARY_SPEED' : 'TT_SUMMARY_SPEED_NONFUNCTIONAL';
- const canBoost = ship.canBoost(cargo, fuel);
+ const canBoost = ship.canBoost(cargo, ship.fuelCapacity);
const boostTooltip = canBoost ? 'TT_SUMMARY_BOOST' : canThrust ? 'TT_SUMMARY_BOOST_NONFUNCTIONAL' : 'TT_SUMMARY_SPEED_NONFUNCTIONAL';
return
diff --git a/src/app/i18n/en.js b/src/app/i18n/en.js
index 5ef042f3..d3fd4bea 100644
--- a/src/app/i18n/en.js
+++ b/src/app/i18n/en.js
@@ -75,7 +75,7 @@ export const terms = {
PHRASE_EFFECTIVE_SDPS_ARMOUR: 'SDPS against armour',
TT_SUMMARY_SPEED: 'With full fuel tank and 4 pips to ENG',
- TT_SUMMARY_SPEED_NONFUNCTIONAL: 'Thrusters powered off or over maximum mass',
+ TT_SUMMARY_SPEED_NONFUNCTIONAL: 'Thrusters powered off or over maximum mass with full fuel and cargo loads',
TT_SUMMARY_BOOST: 'With full fuel tank and 4 pips to ENG',
TT_SUMMARY_BOOST_NONFUNCTIONAL: 'Power distributor not able to supply enough power to boost',
TT_SUMMARY_SHIELDS: 'Raw shield strength, including boosters',
diff --git a/src/app/shipyard/Ship.js b/src/app/shipyard/Ship.js
index 258bfa38..3efabf91 100755
--- a/src/app/shipyard/Ship.js
+++ b/src/app/shipyard/Ship.js
@@ -198,7 +198,7 @@ export default class Ship {
* @return {Number} Pitch
*/
calcPitch(eng, fuel, cargo, boost) {
- return Calc.calcPitch(this.unladenMass + fuel + cargo, this.pitch, this.standard[1].m, this.pipSpeed, eng, this.topBoost / this.topSpeed, boost);
+ return Calc.calcPitch(this.unladenMass + fuel + cargo, this.pitch, this.standard[1].m, this.pipSpeed, eng, this.boost / this.speed, boost);
}
/**
@@ -210,7 +210,7 @@ export default class Ship {
* @return {Number} Roll
*/
calcRoll(eng, fuel, cargo, boost) {
- return Calc.calcRoll(this.unladenMass + fuel + cargo, this.roll, this.standard[1].m, this.pipSpeed, eng, this.topBoost / this.topSpeed, boost);
+ return Calc.calcRoll(this.unladenMass + fuel + cargo, this.roll, this.standard[1].m, this.pipSpeed, eng, this.boost / this.speed, boost);
}
/**
@@ -222,7 +222,7 @@ export default class Ship {
* @return {Number} Yaw
*/
calcYaw(eng, fuel, cargo, boost) {
- return Calc.calcYaw(this.unladenMass + fuel + cargo, this.yaw, this.standard[1].m, this.pipSpeed, eng, this.topBoost / this.topSpeed, boost);
+ return Calc.calcYaw(this.unladenMass + fuel + cargo, this.yaw, this.standard[1].m, this.pipSpeed, eng, this.boost / this.speed, boost);
}
/**