Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-28 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21790/ --- (Updated May 28, 2014, 6:52 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-28 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21790/ --- (Updated May 28, 2014, 6:53 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-23 Thread Suman Karumuri
On May 22, 2014, 11:37 p.m., David McLaughlin wrote: src/main/resources/org/apache/aurora/scheduler/http/ui/css/app.css, line 127 https://reviews.apache.org/r/21790/diff/1/?file=588421#file588421line127 We should probably decide on a naming convention for custom CSS. I've been

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-23 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21790/ --- (Updated May 23, 2014, 7:12 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-23 Thread Suman Karumuri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21790/ --- (Updated May 23, 2014, 7:13 p.m.) Review request for Aurora, David McLaughlin

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-23 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21790/#review43883 --- Ship it! Ship It! - Bill Farner On May 23, 2014, 7:13 p.m.,

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-22 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21790/#review43760 --- I would have to object to the amount of new files added with this

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-22 Thread Suman Karumuri
On May 22, 2014, 7:33 p.m., David McLaughlin wrote: I would have to object to the amount of new files added with this review. Is there any way to tell bower to only install the files defined in main? The vast majority of these new files are superfluous. AFAICT, there is no way to

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-22 Thread David McLaughlin
On May 22, 2014, 7:33 p.m., David McLaughlin wrote: I would have to object to the amount of new files added with this review. Is there any way to tell bower to only install the files defined in main? The vast majority of these new files are superfluous. Suman Karumuri wrote:

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-22 Thread David McLaughlin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21790/#review43794 --- Ship it!

Re: Review Request 21790: Upgrade Aurora UI to bootstrap3

2014-05-22 Thread Suman Karumuri
On May 22, 2014, 7:33 p.m., David McLaughlin wrote: I would have to object to the amount of new files added with this review. Is there any way to tell bower to only install the files defined in main? The vast majority of these new files are superfluous. Suman Karumuri wrote: