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.

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

chatline: Split creation of unit links from tile/city links #2525

Merged
merged 3 commits into from
Feb 12, 2025

Conversation

blabber
Copy link
Collaborator

@blabber blabber commented Feb 5, 2025

Splits the functionality to create unit links for the chatline from the functionality to create tile/city links. This allows the user to create a tile link for a tile, even if it is occupied by a unit.

Shortcut Old behavior New behavior
Ctrl-Alt-RMB Link to (unit -> city -> tile) Link to (city ->tile)
Ctrl-Alt-Shift-RMB ./. Link to (unit -> city -> tile)

It is possible to restore the old behavior by rebinding the shortcuts.

No documentation changes are required, as this matches the existing chatline documentation in data/helpdata.txt.

Closes #2524

Reported by clangd.
Splits the functionality to create unit links for the chatline from
the functionality to create tile/city links. This allows the user to
create a tile link for a tile, even if it is occupied by a unit.

| Shortcut           | Old behavior                   | New behavior                   |
| ------------------ | ------------------------------ | ------------------------------ |
| Ctrl-Alt-RMB       | Link to (unit -> city -> tile) | Link to (city ->tile)          |
| Ctrl-Alt-Shift-RMB | ./.                            | Link to (unit -> city -> tile) |

It is possible to restore the old behavior by rebinding the shortcuts.

No documentation changes are required, as this matches the existing
chatline documentation in `data/helpdata.txt`.
@lmoureaux lmoureaux requested a review from jwrober February 7, 2025 20:10
Copy link
Collaborator

@jwrober jwrober left a comment

Choose a reason for hiding this comment

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

This changes the Shortcut Options as documented in docs/Manuals/Game/shortcut-options.rst. Need to add to this PR for approval. Code works great.

@blabber
Copy link
Collaborator Author

blabber commented Feb 12, 2025

This changes the Shortcut Options as documented in docs/Manuals/Game/shortcut-options.rst.

Good catch.

@blabber blabber closed this Feb 12, 2025
@blabber blabber reopened this Feb 12, 2025
@jwrober jwrober self-requested a review February 12, 2025 19:47
Copy link
Collaborator

@jwrober jwrober left a comment

Choose a reason for hiding this comment

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

Wonderful

@jwrober jwrober enabled auto-merge (rebase) February 12, 2025 19:47
@jwrober jwrober merged commit a86cacd into longturn:master Feb 12, 2025
24 of 33 checks passed
@blabber blabber deleted the bugfix/unit_links branch February 12, 2025 19:52
@jwrober jwrober added the back-port back-port candidate label Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-port back-port candidate
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

Linking in chatline does not match documentation
3 participants