Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support adventure mode look cursor #5230

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Support adventure mode look cursor #5230

wants to merge 2 commits into from
+111 −43

Conversation

Bumber64
Copy link
Contributor

Make the fns in gui/dwarfmode.lua modify the adv cursor if in adv mode.
Do the same for Gui::getCursorPos, Gui::getCursorCoords, and Gui::setCursorCoords.
Clean up some direct uses of the cursor.

@Bumber64
Update changelog.txt
d88277f
@Bumber64
Support adv look cursor
e6e1972 
* Update dwarfmode.lua - Make cursor fns support adv mode look
* Update Gui.cpp - Make cursor fns support adv mode look
* Update changevein.cpp - Use getCursorPos
* Update createitem.cpp - Use getCursorPos
* Update cursecheck.cpp - Doesn't use cursor
* Update stripcaged.cpp - Doesn't use cursor
* Update zone.cpp - Use getCursorPos
@Bumber64 Bumber64 mentioned this pull request Jan 31, 2025
Open
@Bumber64
Copy link
Contributor Author

Bumber64 commented Jan 31, 2025 •
edited
Loading

A list of scripts that should be potentially be using the guidm fns instead:

devel/light.lua
devel/query.lua
devel/tree-info.lua
gui/advfort.lua
gui/blueprint.lua
gui/companion-order.lua
gui/tiletypes.lua
modtools/create-unit.lua
colonies.lua
hfs-pit.lua
launch.lua
position.lua
putontable.lua
source.lua
stripcaged.lua
teleport.lua

toggle-kbd-cursor.lua can use guidm.clearCursorPos() instead of guidm.setCursorPos() in one spot.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
51.03-r2
Status: Todo
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Bumber64