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

Investigating an allied city prevents one from activating own units inside #2286

Closed
lmoureaux opened this issue May 19, 2024 · 0 comments · Fixed by #2316
Closed

Investigating an allied city prevents one from activating own units inside #2286

lmoureaux opened this issue May 19, 2024 · 0 comments · Fixed by #2316
Labels
bug Something isn't working gui This issue requires changes to the user interface
Milestone
v3.1-stable

Comments

@lmoureaux
Copy link
Contributor

Describe the bug
After investigating an allied city, the city dialog will always pop up, but the unit list is inactive. This prevents one from activating one's own units.

To Reproduce
Steps to reproduce the behavior:

  1. Move unit A to an allied city
  2. Investigate that city with a diplomat
  3. The city dialog pops up
  4. Try to activate unit A
  5. Fail to do so until you reconnect

Expected behavior
One can activate one's own units from the city dialog.

Platform and version (please complete the following information):

  • OS: n/a
  • Freeciv21 version: master
  • Ruleset/Longturn game (if applicable): LT82
@lmoureaux lmoureaux added bug Something isn't working gui This issue requires changes to the user interface labels May 19, 2024
@lmoureaux lmoureaux added this to the v3.1-stable milestone May 19, 2024
@lmoureaux lmoureaux mentioned this issue Jul 7, 2024
Closed
lmoureaux added a commit to lmoureaux/freeciv21 that referenced this issue Jul 7, 2024
@lmoureaux
Don't disable the present units list when investigating
1cac910 
This allows giving orders to one's own units in the city and scrolling
through the unit list.

Closes longturn#2302.
Closes longturn#2286.
@lmoureaux lmoureaux mentioned this issue Jul 7, 2024
Merged
@jwrober jwrober closed this as completed in 21a241a Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working gui This issue requires changes to the user interface
Projects
None yet
Milestone
v3.1-stable
Development

Successfully merging a pull request may close this issue.

Don't disable the present units list when investigating lmoureaux/freeciv21
1 participant
@lmoureaux