-
Notifications
You must be signed in to change notification settings - Fork 18
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
Add MenuBar #97
Add MenuBar #97
Conversation
a4d4f22
to
7b8d7ae
Compare
6d6ff9d
to
ba16bd1
Compare
ba16bd1
to
023126a
Compare
Coding style
00a3511
to
cc7ccf4
Compare
LGTM for an intermediate merge so we can move forward with this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved
I would add on pending task also:
And yes please add something to show separator, leaving an empty line is not looking like a separator |
In a first try I also changed "..." to show that there is a sub menu with ">" maybe you can think to do something like that pretty better looking, I gave up just thinking to R2L languages I like the menu selection and move left and right that i haven't implemented so well. Please follow also discussion here |
This PR replaces #93.
Trello
https://trello.com/c/4GtDpmMo/
Overview
This adds a MenuBar widget to the NCurses UI.
For more details, see the main PR for this feature:
/~https://github.com/libyui/libyui/pull/169
Travis Build Failure
This doesn't build because the image used by Travis does not yet contain the libyui with the new features. This is expected.
Related PRs
Proposed Solution
There is a new class
NCMenuBar
that manages the top menu list. This class has the following responsibilities:NCPopupMenu
).The class
NCPopupMenu
was extended, gaining these responsibilities:The
NCPopupMenu
internally uses aNCTable
, and this could have some drawbacks:Pending Tasks:
NOTE: Pending tasks will come in another PR. We want to merge this now to synchronize SO version of all libyui related packages.