> On March 7, 2016, 10:03 p.m., Alejandro Fernandez wrote: > > ambari-web/app/templates/main/host/summary.hbs, line 71 > > <https://reviews.apache.org/r/44457/diff/1/?file=1283160#file1283160line71> > > > > In the image for "Updated Fix (Mar-07).tiff" the component name of > > "ZOOKEEPER SERVER" is missing.
Hello Alexandr, I was testing the length for long component names and I updated the "Zookeeper" text with ".........................." as you can see in the line below "Zookeeper Server". There was also a space added following component names in the original code, which I removed (you can see how the dots stop right at the right border of the <div> block). This space was originally added to separate the name from the icons appearing after it. Since we are providing <div> blocks to each of the elements, I have added padding in the medkit-icon-block to adjust this spacing. Thanks! - Keta ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/44457/#review122386 ----------------------------------------------------------- On March 7, 2016, 9:57 p.m., Keta Patel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/44457/ > ----------------------------------------------------------- > > (Updated March 7, 2016, 9:57 p.m.) > > > Review request for Ambari, Alexandr Antonenko and Di Li. > > > Bugs: AMBARI-15310 > https://issues.apache.org/jira/browse/AMBARI-15310 > > > Repository: ambari > > > Description > ------- > > In Ambari UI, navigate to Hosts and click on the hostname where the service > is installed. The summary page displays the list of services installed. > Observe that longer service names does not display well. The refresh icon > etc. spills to the next line (see attachment "Original spacing with long > service name"). > Also, note the indentation of the last button for Clients does not align well > with the buttons above it. > > > Diffs > ----- > > ambari-web/app/styles/application.less 755dff2 > ambari-web/app/templates/main/host/details/host_component.hbs 60031de > ambari-web/app/templates/main/host/summary.hbs 17a0b69 > > Diff: https://reviews.apache.org/r/44457/diff/ > > > Testing > ------- > > FIX: > This issue is simialr to AMBARI-11670 where the spacing for each component > was fixed in a <div> tag. > The fix here also uses a similar approach where every component is surrounded > by a <div> tag and the width is fixed for them so that the appear in their > respective positions regardless of the length of service names. I have also > adjusted the width of the left <div> component with "span8" instead of > "span7" and the right <div> component of the button to "span4" instead of > "span5" to better adjust the spacing. > > TESTS: > No tests are written for these changes as the fix involves updates in the > stylesheet. > However, the changes were tested by zooming in and out and checking if the > icons and names were contained in their respective blocks and not overlapping > with other <div> blocks. > > > The following is the result of running ambari-web tests: > > 24565 tests complete (28 seconds) > 145 tests pending > > > File Attachments > ---------------- > > AMBARI-15310 (patch-2) > > https://reviews.apache.org/media/uploaded/files/2016/03/07/5ade9968-bb9d-4d61-8362-740fb24c591f__AMBARI-15310_Mar07.patch > > > Thanks, > > Keta Patel > >
