-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50314/
-----------------------------------------------------------

(Updated July 22, 2016, 8:28 p.m.)


Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, and Yusaku Sako.


Changes
-------

address last comments


Bugs: AMBARI-17824
    https://issues.apache.org/jira/browse/AMBARI-17824


Repository: ambari


Description
-------

Following needs to be done as part of this work
view instances needs to be shown under relevant service as tabs
content of the view resources needs to be shown in iframe
Quick links location needs to be adjusted so it does not collapse with view tabs
Left service menu on the service page needs to be removed to give more space to 
view content
Increase the span for the pages under service page for larger width


Diffs (updated)
-----

  ambari-web/app/app.js 79493a1 
  ambari-web/app/messages.js 24c9cfa 
  ambari-web/app/models/stack_service.js 3e2d5b4 
  ambari-web/app/routes/main.js ebf9cb3 
  ambari-web/app/styles/application.less 200c7be 
  ambari-web/app/templates/main/service/item.hbs 43fb3dd 
  ambari-web/app/views.js 904edb7 
  ambari-web/app/views/common/widget/heatmap_widget_view.js 6c74420 
  ambari-web/app/views/main/dashboard.js 9694774 
  ambari-web/app/views/main/service/item.js 02ac426 
  ambari-web/app/views/main/views/details.js df1b19b 

Diff: https://reviews.apache.org/r/50314/diff/


Testing
-------

I verified that the patch accomplishes the tasks written in the description.
Verified that unit test passes with the patch
29343 tests complete (26 seconds)
154 tests pending


Thanks,

Manasi Maheshwari

Reply via email to