uwescience / myria-web

Web frontend for Myria
https://demo.myria.cs.washington.edu
Other
11 stars 14 forks source link

Detect chrome, warn if other browsers #180

Closed dhalperi closed 9 years ago

dhalperi commented 9 years ago
dhalperi commented 9 years ago

@domoritz I suspect you're the one who would have the strongest opinion about this?

domoritz commented 9 years ago

Safari and Opera (webkit) are working fine as well. The only problem with firefox that I see is the visualization. I'd love to have support for firefox as well but since we don't have anyone in our group that's using it, a warning is probably a good idea to have a warning. Can we make the tone a bit different, though and say "You are using a browser that does not support all features of Myria. We recommend Google Chrome." Also, the style of the alert looks different from http://getbootstrap.com/components/#alerts

dhalperi commented 9 years ago

I'm not taking on Safari and/or Opera testing as dependencies, so I'm just going to keep warning if they're not using Chrome itself. Switched to alert style and adjusted message.

dhalperi commented 9 years ago

The new Safari look: screen shot 2014-08-05 at 10 36 40 am

domoritz commented 9 years ago

Great