----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57268/#review168155 -----------------------------------------------------------
ambari-web/app/views/common/configs/service_config_layout_tab_view.js Lines 185 (patched) <https://reviews.apache.org/r/57268/#comment240302> I think this two conditions can be join in one line - Andrii Babiichuk On Березень 6, 2017, 9:13 після полудня, Richard Zang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57268/ > ----------------------------------------------------------- > > (Updated Березень 6, 2017, 9:13 після полудня) > > > Review request for Ambari, Andrii Babiichuk, Jaimin Jetly, Oleg Nechiporenko, > and Yusaku Sako. > > > Bugs: AMBARI-20287 > https://issues.apache.org/jira/browse/AMBARI-20287 > > > Repository: ambari > > > Description > ------- > > Take subSectionTab into consideration on show/hide > > > Diffs > ----- > > ambari-web/app/models/configs/theme/sub_section.js 7274569 > ambari-web/app/models/configs/theme/sub_section_tab.js 2262882 > ambari-web/app/views/common/configs/service_config_layout_tab_view.js > 466a88d > ambari-web/test/models/configs/theme/sub_section_tab_test.js 0c3b98c > > > Diff: https://reviews.apache.org/r/57268/diff/1/ > > > Testing > ------- > > Manully tested on live cluster. All unit tests passed. > 20560 passing (25s) > 153 pending > > > Thanks, > > Richard Zang > >