-
Notifications
You must be signed in to change notification settings - Fork 987
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
Fixes #27537 - Show tooltip for vm memory size #6964
Conversation
Issues: #27537 |
this is fine but just a workaround for broken spinner field, I think @ezr-ondrej looked at the issue a while ago so he may have some lead on how to fix it properly |
Yeah, this will work, because different javascript is catching it, but it doesn't solve the core issue. |
app/helpers/form_helper.rb
Outdated
@@ -220,7 +220,7 @@ def byte_size_f(f, attr, options = {}) | |||
options[:label_help] = _("When specifying custom value, add 'MB' or 'GB' at the end. Field is not case sensitive and MB is default if unspecified.") | |||
options[:help_block] ||= soft_limit_warning_block | |||
options[:help_block] += f.hidden_field(attr, :class => "real-hidden-value", :id => nil) | |||
options[:label_help_options] = { :rel => 'popover-modal' } | |||
options[:label_help_options] = { :data => { :placement => 'right' } } |
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.
Maybe we can just delete it, to be consistent with other fields whatever we decide to use globaly? Or is there a reason to be always on the right?
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.
It's just that other tooltips in the form are on the right. But it is a default, so I deleted it as suggested.
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.
ACK 👍 Thanks @xprazak2 merging.
failures of travis just an fetch failure on one version, ignoring... |
No description provided.