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
[SLE15-SP3] Place file selection buttons at the bottom #18
Conversation
src/lib/vpn/edit_gw_secrets.rb
Outdated
@@ -192,10 +192,10 @@ def mk_gw_cert_frame | |||
Frame(_("Gateway certificate"), VBox( | |||
Left(MinWidth(40, HBox( | |||
InputField(Id(:gw_cert), _("Path to certificate file"), ""), | |||
PushButton(Id(:select_gw_cert), _("Pick.."))))), | |||
Bottom(PushButton(Id(:select_gw_cert), _("Pick..")))))), |
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.
BTW I'd replace the "Pick" label with Label.Select
code - this would be more consistent with the other dialogs and it would save some translators work...
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.
Thank you @lslezak for the suggestion! I had focused on resolving what was reported in the bug. Could you please check the updated diff and review if it meets the current standard?
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.
Looks good
This mr is missing a version change and a changelog entry. |
This should allow input text boxes and file selection buttons to be aligned.
✔️ Public Jenkins job #21 successfully finished |
✔️ Internal Jenkins job #16 successfully finished |
This should allow input text boxes and file selection buttons to be
aligned.
Screenshots before change:
For Gateway Certificate
For Client Certificate
Screenshots after the change:
For Gateway Certificate
For Client Certificate
Fixes bsc#1085625