-
Notifications
You must be signed in to change notification settings - Fork 54
Update the code to better handle the dropdown menus #299
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
base: master
Are you sure you want to change the base?
Conversation
|
User's choice will be marked with ⚫ when present |
90b615f to
12df10f
Compare
afb9270 to
b879cf3
Compare
f265dd1 to
ed09767
Compare
575dbcd to
8535af0
Compare
|
Kudos, SonarCloud Quality Gate passed!
|
979beaf to
c51183b
Compare
fc2226a to
54015e6
Compare
de17bc4 to
8c5346b
Compare
9285f31 to
41d2a54
Compare
|
|
|
|











Is your feature request related to a problem? Please describe.
The current "checkbox-button" style menu and the "radio-button" style menu's have very dirty implementations.
Both can be found in the command-box.
Describe the solution you'd like
This should be cleaned up. Best solution seems to be the introduction of a "DropDownCheckbox" and "DropDownRadio" sub-classes that handle all the specifics.
Describe alternatives you've considered
Additional context
Additionally, the difference between the user's choice and the system's choice in the "target" menu should be enhanced.
Additionally, the dropdown menu's should work on touchscreens. i.e. the menu opens on 'click' instead of only 'mouseover'.