----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10535/#review19454 -----------------------------------------------------------
Ship it! Rishi the style is exactly what you set out to achieve. Simple, clean and high contrast. Well done. - Cameron Goodale On April 19, 2013, 12:30 a.m., Rishi Verma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10535/ > ----------------------------------------------------------- > > (Updated April 19, 2013, 12:30 a.m.) > > > Review request for oodt, Chris Mattmann, Andrew Hart, Paul Ramirez, and > Cameron Goodale. > > > Description > ------- > > Hey all, > > Per JIRA Issue: https://issues.apache.org/jira/browse/OODT-597, I proposed we > update some of OPSUI's look and feel to make it more readable. > > The attached diff resolves all points addressed in OODT-597, so please take a > look at that for more details. I've attached some screenshots of what the new > OPSUI looks like - for your convenience. > > I know A LOT of work must have gone into the original OPSUI look and feel. So > I want to make sure my improvements to it (and in some cases changes) are > transparent and reviewed before we push. Like I said in OODT-597, the old > OPSUI was very slick. I feel it was a little bit to read though on > low-contrast screens, and that's why I proposed an improvement to the theme. > Also, I think its good if we keep some consistency between how we do OODT > logos, coloring, and themeing to make sure we are generating a consistent > product. > > Anyways, let me know what you think! I'll push the commits when I get a 'ship > it'! :D > > rishi > > > This addresses bug OODT-597. > https://issues.apache.org/jira/browse/OODT-597 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/BasePage.css > 1468173 > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/BasePage.html > 1468173 > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/BasePage.java > 1468173 > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/BasePage_cleanwhite.css > PRE-CREATION > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/BasePage_cleanwhite.html > PRE-CREATION > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/BasePage_navyblue.css > PRE-CREATION > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/DefaultSkin.css > 1468173 > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/HomePage.css > 1468173 > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/HomePage.java > 1468173 > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/HomePage_cleanwhite.css > PRE-CREATION > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/HomePage_navyblue.css > PRE-CREATION > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/OPSUIWebPage.java > PRE-CREATION > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/project_logo.png > UNKNOWN > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/java/org/apache/oodt/pcs/opsui/splash.png > UNKNOWN > > http://svn.apache.org/repos/asf/oodt/trunk/pcs/opsui/src/main/webapp/META-INF/context.xml > 1468173 > > Diff: https://reviews.apache.org/r/10535/diff/ > > > Testing > ------- > > Local testing. See screenshots. > > > Screenshots > ----------- > > Splash page > https://reviews.apache.org/r/10535/s/20/ > Workflow monitor page > https://reviews.apache.org/r/10535/s/21/ > Workflow tasks diagram > https://reviews.apache.org/r/10535/s/22/ > Workflow task info > https://reviews.apache.org/r/10535/s/23/ > File Manager browser page > https://reviews.apache.org/r/10535/s/24/ > > > Thanks, > > Rishi Verma > >