Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37137/#review94293 --- This is being nit-picky, but is it a glitch that the video makes it

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37137/#review94299 --- Master (61c63ea) is red with this patch.

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37137/ --- (Updated Aug. 5, 2015, 8:28 p.m.) Review request for Aurora and Zameer Manji.

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Joshua Cohen
On Aug. 5, 2015, 7:38 p.m., Bill Farner wrote: This is being nit-picky, but is it a glitch that the video makes it seem like the first pulse is really sudden compared to the subsqeuent ones? Probably just an artifact of the screen recording. The pulse speed is constant in browser. -

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37137/#review94302 --- Ship it! Ship It! - Bill Farner On Aug. 5, 2015, 8:28 p.m.,

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37137/#review94301 --- Master (61c63ea) is green with this patch.

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37137/#review94303 --- src/main/resources/scheduler/assets/css/app.css (line 436)

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Joshua Cohen
On Aug. 5, 2015, 8:49 p.m., Zameer Manji wrote: src/main/resources/scheduler/assets/css/app.css, line 436 https://reviews.apache.org/r/37137/diff/2/?file=1032840#file1032840line436 If we are going to add the -webkit prefix then I think we should do the same for -moz. Otherwise I

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37137/#review94308 --- Ship it! Ship It! - Zameer Manji On Aug. 5, 2015, 1:28 p.m.,

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Joshua Cohen
On Aug. 5, 2015, 8:49 p.m., Zameer Manji wrote: src/main/resources/scheduler/assets/css/app.css, line 436 https://reviews.apache.org/r/37137/diff/2/?file=1032840#file1032840line436 If we are going to add the -webkit prefix then I think we should do the same for -moz. Otherwise I

Re: Review Request 37137: Add indications of continued activity to the Update page.

2015-08-05 Thread Zameer Manji
On Aug. 5, 2015, 1:49 p.m., Zameer Manji wrote: src/main/resources/scheduler/assets/css/app.css, line 436 https://reviews.apache.org/r/37137/diff/2/?file=1032840#file1032840line436 If we are going to add the -webkit prefix then I think we should do the same for -moz. Otherwise I