-
Notifications
You must be signed in to change notification settings - Fork 350
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
fix #3867 #3878
fix #3867 #3878
Conversation
@hansva sorry for the wrong label name I've not noticed :-( . I will fix it |
@hansva, unfortunately, the only thing I'm able to fix is the label. Multiple selection (that I agree with you is not completed in the EnterSelectionDialog therefore it cannot be used. About the hieght of the dialog I'm to find out a way to set the height of the dialog but it seems not that easy. Let me know your opinion about confirming to move forward with this improvement or if we want to throw everything away and leave the things as they are. |
As stated in my previous comment, nothing you can do about it in this ticket. Just some personal frustration flowing out. |
Yeah I understand and I agree with you |
right, a mixup happened in the issue number, for reference this is for #3867 |
fix #3867 - Unit test action: dialog improvement
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
mvn clean install apache-rat:check
to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.git rebase -i
.addresses #123
), if applicable.To make clear that you license your contribution under the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.