Merge pull request #2920 from himdel/widget-fullscreen

Dashboard - fullscreen - open in new tab (cherry picked from commit a5bb860)
ManageIQ · Dec 5, 2017 · ebdf125 · ebdf125
1 parent 5f6a67f
commit ebdf125
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/app/presenters/widget_presenter.rb b/app/presenters/widget_presenter.rb
@@ -64,7 +64,7 @@ def widget_buttons
                    :confirm     => confirm,
                    :href        => "/dashboard/report_only?rr_id=#{@widget.id}&type=#{@widget.content_type == "chart" ? 'hybrid' : 'tabular'}",
                    :fonticon    => 'fa fa-arrows-alt fa-fw',
-                   :clickAction => "return miqClickAndPop(this);")
+                   :target      => "_blank")
       if PdfGenerator.available?
         buttons.push(:id       => "w_#{@widget.id}_pdf",
                      :title    => _("Download the full report (all rows) as a PDF file"),

diff --git a/app/views/static/dropdown-menu.html.haml b/app/views/static/dropdown-menu.html.haml
@@ -15,6 +15,7 @@
          'ng-attr-data-remote' => "{{button.dataRemote}}",
          'data-confirm'        => "{{button.confirm}}",
          'href'                => "{{button.href}}",
-         'data-miq_sparkle_on' => "{{button.sparkleOn ? 'true' : 'false'}}"}
+         'data-miq_sparkle_on' => "{{button.sparkleOn ? 'true' : 'false'}}",
+         'ng-attr-target'      => "{{button.target}}"}
         %span{'ng-class' => "button.fonticon"}
         {{button.name}}