Skip to content

Fixes #2109. MenuBar has extra space on left. Refactors MenuTests. Better Menu API docs.#2117

Merged
tig merged 18 commits intogui-cs:developfrom
tig:fix_2109_menubar_spacing
Oct 21, 2022
Merged

Fixes #2109. MenuBar has extra space on left. Refactors MenuTests. Better Menu API docs.#2117
tig merged 18 commits intogui-cs:developfrom
tig:fix_2109_menubar_spacing

Conversation

@tig
Copy link
Copy Markdown
Member

@tig tig commented Oct 21, 2022

Fixes #2109

Also revamps most of the MenuTests to be less dependent on the formatting of expected within the source file, and thus less fragile.

@tig tig requested a review from migueldeicaza as a code owner October 21, 2022 00:16
@tig
Copy link
Copy Markdown
Member Author

tig commented Oct 21, 2022

@BDisp - please review this! Thanks.

Comment thread Terminal.Gui/Core/ContextMenu.cs Outdated
Comment thread Terminal.Gui/Views/Menu.cs
Comment thread Terminal.Gui/Views/Menu.cs Outdated
Comment thread Terminal.Gui/Views/Menu.cs Outdated
Comment thread Terminal.Gui/Views/Menu.cs Outdated
Comment thread Terminal.Gui/Views/Menu.cs Outdated
Comment thread Terminal.Gui/Views/Menu.cs Outdated
Comment thread Terminal.Gui/Core/ContextMenu.cs Outdated
@tig tig changed the title Fixes #2109. MenuBar has extra space on left Fixes #2109. MenuBar has extra space on left. Refactors MenuTests. Better Menu API docs. Oct 21, 2022
@tig tig merged commit 8cbd597 into gui-cs:develop Oct 21, 2022
This was referenced Nov 4, 2022
@tig
Copy link
Copy Markdown
Member Author

tig commented Nov 4, 2022

Also fixes #2126

@tig tig deleted the fix_2109_menubar_spacing branch April 3, 2024 01:47
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.

MenuBar has an extra space on left. Seems odd.

2 participants