> On Sept. 23, 2014, 12:08 a.m., Joshua Cohen wrote: > > I'm sure it's fine, but in the future it might be helpful to include > > screenshots comparable to those in the ticket showing the proper alignment?
Ack. Will add before/after screenshots for this type of work in the future. - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25913/#review54209 ----------------------------------------------------------- On Sept. 22, 2014, 11:23 p.m., David McLaughlin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25913/ > ----------------------------------------------------------- > > (Updated Sept. 22, 2014, 11:23 p.m.) > > > Review request for Aurora, Joshua Cohen, Kevin Sweeney, and Bill Farner. > > > Bugs: AURORA-436 > https://issues.apache.org/jira/browse/AURORA-436 > > > Repository: aurora > > > Description > ------- > > HTML Grid Fixes in Scheduler UI. > > > Got rid of custom self-closing (void) tags: > https://github.com/angular/angular.js/issues/1953. > Got rid of nested container-fluids (caused inconsistent margins on role/env > pages). > Got rid of redundant nested if/else on errors. > > > Diffs > ----- > > src/main/resources/org/apache/aurora/scheduler/http/ui/breadcrumb.html > 5c4fe96e9c6cb816e497f9dc145f99b6568de887 > src/main/resources/org/apache/aurora/scheduler/http/ui/error.html > 5b03acaf593ee9d0c32e70f7178fb0e54162c7b8 > src/main/resources/org/apache/aurora/scheduler/http/ui/job.html > ca1a0f666a1eea7c225f017918f4755c28c57502 > src/main/resources/org/apache/aurora/scheduler/http/ui/role.html > c3a2fd9f12fe6b99a5253f123002a7a02e51a9ec > src/main/resources/org/apache/aurora/scheduler/http/ui/update.html > b6cf4f0ed44f33bd5048c5ba9267eabb837dab84 > > Diff: https://reviews.apache.org/r/25913/diff/ > > > Testing > ------- > > Manual testing. These are effectively code cleanup no-ops. The main goal is > to avoid people cargo-culting existing bad practices in the name of > consistency (which I did with the time-display directives). > > > Thanks, > > David McLaughlin > >