-----------------------------------------------------------
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
> 
>

Reply via email to