Skip to content
This repository has been archived by the owner on May 23, 2023. It is now read-only.

Small fix and DriverModeSetting #7331

Merged
merged 1 commit into from
Jul 24, 2021
Merged

Small fix and DriverModeSetting #7331

merged 1 commit into from
Jul 24, 2021

Conversation

schwiti6190
Copy link
Contributor

@schwiti6190 schwiti6190 commented Jul 5, 2021

@schwiti6190 schwiti6190 requested a review from pvaiko July 5, 2021 11:37
base.lua Outdated Show resolved Hide resolved
settings.lua Outdated Show resolved Hide resolved
settings.lua Show resolved Hide resolved
settings.lua Outdated Show resolved Hide resolved
settings.lua Outdated Show resolved Hide resolved
start_stop.lua Outdated Show resolved Hide resolved
settings.lua Outdated Show resolved Hide resolved
@schwiti6190 schwiti6190 requested a review from pvaiko July 16, 2021 11:09
base.lua Outdated
@@ -334,7 +331,7 @@ function courseplay:onDraw()
local isDriving = self:getIsCourseplayDriving();

--WORKWIDTH DISPLAY
if self.cp.mode ~= 7 and self.cp.timers.showWorkWidth and self.cp.timers.showWorkWidth > 0 then
if self.cp.settings.driverMode:get() ~= 7 and self.cp.timers.showWorkWidth and self.cp.timers.showWorkWidth > 0 then
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pvaiko Is it correct, that the work width isn't shown for the bale collector ai driver ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is, it does not make any sense there. While you are at it, could replace the magic number with the constant, btw

- Small global company fix, where the production overlay automatically opens while loading. #7313
- Added DriverModeSetting
- Adds more available helpers than 10 #7335
- Drive unload now improvements.
@schwiti6190 schwiti6190 merged commit 468d8cc into master Jul 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants