----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47011/#review131816 -----------------------------------------------------------
Ship it! Ship It! - Oliver Szabo On May 5, 2016, 12:46 p.m., Dharmesh Makwana wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47011/ > ----------------------------------------------------------- > > (Updated May 5, 2016, 12:46 p.m.) > > > Review request for Ambari, Don Bosco Durai, Jaimin Jetly, Oliver Szabo, and > Sumit Mohanty. > > > Bugs: AMBARI-16267 > https://issues.apache.org/jira/browse/AMBARI-16267 > > > Repository: ambari > > > Description > ------- > > Patch contains > 1. Top navbar same as ambari was added in the pages after login screen, so > added to login page as well to maintain consistency. > 2. Show help flow (tour) on first time browser is used (based on > localstorage). > > > Diffs > ----- > > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/META-INF/security-applicationContext.xml > 468552b > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/login.jsp 27fd8ec > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/scripts/views/common/Header.js > 3916bff > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/styles/style.css > 495cf98 > > ambari-logsearch/ambari-logsearch-portal/src/main/webapp/styles/style_v2.css > 675695f > > Diff: https://reviews.apache.org/r/47011/diff/ > > > Testing > ------- > > Top navbar looks as expected and help flow tested on the internal test server. > > > Thanks, > > Dharmesh Makwana > >