Skip to content

RightClickMenu enhancements#946

Merged
stavrosfa merged 4 commits into
C7-Game:Developmentfrom
stavrosfa:enhancement/right-click-menu-separators
Jun 21, 2026
Merged

RightClickMenu enhancements#946
stavrosfa merged 4 commits into
C7-Game:Developmentfrom
stavrosfa:enhancement/right-click-menu-separators

Conversation

@stavrosfa

Copy link
Copy Markdown
Contributor

Added a menu separator as is in the original game, cleaned up the text and the order of items.

@ajhalme

ajhalme commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Can't unsee the wavy justification relative to the text (OC3 vs Civ3):

menus

@stavrosfa

Copy link
Copy Markdown
Contributor Author

Yeah, I know, I tried to fix this, but as far as I could tell it needs a more holistic refactor to be able to allign the text buttons.
I tried to add simple spaces before and after the text to simulate a margin, but it felt like a very cheap solution.

Added city name for "Zoom to City"
@stavrosfa

Copy link
Copy Markdown
Contributor Author

I somehow stumbled on the solution by mistake, looking for something else.
Turns out the buttons already use a Style Box, and we can specify the margin there.

@stavrosfa

Copy link
Copy Markdown
Contributor Author

Unless I find a bug, I am done with this for now, let me know if it looks ok so it can be merged, thanks!

@ajhalme ajhalme left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@stavrosfa stavrosfa merged commit 78ac01e into C7-Game:Development Jun 21, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants