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
Hide History toolbar when in Reports Import/Export tree #1909
Hide History toolbar when in Reports Import/Export tree #1909
Conversation
@h-kataria Ready for review. |
app/controllers/report_controller.rb
Outdated
@@ -687,7 +687,7 @@ def replace_right_cell(options = {}) # :replace_trees key can be an array of tr | |||
partial = options[:partial] ? options[:partial] : set_partial_name | |||
unless @in_a_form | |||
c_tb = build_toolbar(center_toolbar_filename) | |||
h_tb = build_toolbar("x_history_tb") | |||
h_tb = build_toolbar("x_history_tb") unless (x_active_tree == "export_tree" || params[:id] == "export_accord") |
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.
@GregP please change check to unless x_active_tree == :export_tree
that should be good enough, value of x_active_tree is a symbol not string
5033cd1
to
b9afb5c
Compare
Checked commit GregP@b9afb5c with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0 |
looks good. |
Hide History toolbar when in Reports Import/Export tree (cherry picked from commit b7bf129) https://bugzilla.redhat.com/show_bug.cgi?id=1481853
Fine backport details:
|
@h-kataria Would it be ok to backport this to Euwe branch? |
@simaishi yes, good to be be backported to EUWE |
Euwe backport (to manageiq repo) details:
|
Do not display History Toolbar when in Reports => Import / Export tree, no history to be available/selected.
https://bugzilla.redhat.com/show_bug.cgi?id=1393193
Screen shot prior to fix:
===================
Screen shots post fix: