-
Notifications
You must be signed in to change notification settings - Fork 694
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
Pressing Alt key on a Window with only a MenuBar throws System.InvalidOperationException. #2776
Labels
Comments
BDisp
added a commit
to BDisp/Terminal.Gui
that referenced
this issue
Jul 29, 2023
…hrows System.InvalidOperationException.
8 tasks
tig
added a commit
that referenced
this issue
Oct 17, 2023
* Fixes #2776. Pressing Alt key on a Window with only a MenuBar throws System.InvalidOperationException. * Add unit test. * Fix extra bracket. * Prevents throw exception if Application.Current is null. * Fix unit test because OnLeave is now invoked on toplevel removing, preventing two views having focus. * Fix unit test method name. * Fix Window by not layout on his subviews when adding a new view after the Application.Begin was already running. * FindAndOpenMenuByHotkey now search inside Menus and inside his Children. * Add unit test for Window LayoutSubviews and FindAndOpenChildrenMenuByHotkey menu method. * Prevents button to be clear when it's invisible. * Fixes 2780. Moving a Window that is Application.Top shouldn't be allowed. * Fix condition if Window and Application.Top. * Always LayoutSubviews and PositionToplevels after Clear. * Fixes #2787. MenuItem with CanExecute returning false is select when a MenuBar is opened. * Leveraging the power of CanExecute feature. * Fixes #2789. StatusItem should have a disabled attribute if it can't execute. * Allows positioning a child window outside the limits of the menu and the status bar. * Change to a more appropriate name. * Simplifies all the run actions. * Prevents open menu bar if it's invisible and close all opened menus. * Fix mdi run loop. * Fix hot key on mdi toplevels. * Fix position on mdi toplevels. * Fix Top.Redraw by set state.Toplevel.SetNeedsDisplay if it's needed to redraw. * Fix MdiTop by repainted when a keystroke is generated by keyboard. * Rename local fields. * Force redraw if application.Top needs display. * Added more features to the scenario. * Change the scenarios to run as Application.Top instead of sub-views. * Add a new scenario similar but as Mdi Container. * Add a bunch of new unit tests to prove all this PR. * Only it's need to redraw Application.Top if it's a Mdi Container. * Remove unnecessary code. * Unit test that proves that a MDI child leaves no trace when the location is changed. * Removes unnecessary Application.Init because theses uses Run<T> which already call it. * Ensures a menu bar been closed after run an action. * Ensures that another view can be focused if not IsMenuOpen and LastFocused is null, instead of focused the menu itself. * Ensures a focused contentview subview being focused if MostFocused is null. * Ensures a MdiTop subview to have priority if it's focused and thus make it Current. * Allow a MdiChild be closed when pressing Application.QuitKey. * More unit tests proving the changes. * Ensures the top.MostFocused is focused. * Ensures MdiChild on the front if MdiTop.MostFocused isn't valid, like ContentView. * Add unit test showing MdiChild on the front. * Fix an issue where NullReferenceException can be throws everywhere while get the Application.MdiChildes property. --------- Co-authored-by: Tig <tig@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Running this unit test will fail on developer branch:
The text was updated successfully, but these errors were encountered: