> 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 think we should remove the -webkit prefix and 
> > just stick to the standard css3 names.
> 
> Joshua Cohen wrote:
>     -moz is not necessary. Firefox has supported prefix-less keyframes since 
> version 16 (current stable is 39) which was released in 2012. Current Safari 
> does not work without the -webkit prefix, thus the reason why it's necessary.
> 
> Zameer Manji wrote:
>     Please comment that the -webkit prefix is needed for Safari users.
> 
> Joshua Cohen wrote:
>     I think that's pretty much implied by the name? I don't want to set a 
> precedent where every vendor-prefixed CSS rule requires a comment explaining 
> that it's necessary because that browser doesn't support the prefix-less 
> version.

Understood, I take back my request.


- Zameer


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37137/#review94303
-----------------------------------------------------------


On Aug. 5, 2015, 1:28 p.m., Joshua Cohen wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/37137/
> -----------------------------------------------------------
> 
> (Updated Aug. 5, 2015, 1:28 p.m.)
> 
> 
> Review request for Aurora and Zameer Manji.
> 
> 
> Bugs: AURORA-1125
>     https://issues.apache.org/jira/browse/AURORA-1125
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Add indications of continued activity to the Update page.
> 
> 
> Diffs
> -----
> 
>   src/main/resources/scheduler/assets/css/app.css 
> ba26accceb1da7dcc89739ceb4524ab6b5c4a11d 
>   src/main/resources/scheduler/assets/js/directives.js 
> 408ee21416621d5d1c195752e1b1c5c2757f0664 
>   src/main/resources/scheduler/assets/loadingFooter.html PRE-CREATION 
>   src/main/resources/scheduler/assets/update.html 
> a5c27f3db3ca41a2fdf4c396d221ff1be9d20bdf 
> 
> Diff: https://reviews.apache.org/r/37137/diff/
> 
> 
> Testing
> -------
> 
> See attached video of the progress indicators in action.
> 
> 
> File Attachments
> ----------------
> 
> Progress indicators in action
>   
> https://reviews.apache.org/media/uploaded/files/2015/08/05/b4e2fab2-9a3e-4910-b02f-e31de2631e87__update-page.mov
> 
> 
> Thanks,
> 
> Joshua Cohen
> 
>

Reply via email to