Opened 13 years ago
Last modified 13 years ago
#336 closed defect (FIXED)
Inform people they need a better browser if missing <video>, webm
Reported by: | Christopher Allan Webber | Owned by: | Karen Rustad |
---|---|---|---|
Priority: | major | Milestone: | 0.2.1 |
Component: | programming | Keywords: | |
Cc: | Parent Tickets: |
Description
We should add some messaging for people to know that they should get a better browser if or webm is not enabled. We could use `http://www.modernizr.com/ <http://www.modernizr.com/>`_ to detect this.
Attachments (2)
Change History (11)
comment:2 by , 13 years ago
Oh hey Inconexo, great research! Yes, we don't need the whole bulk of modernizr if that's the case, just the detection part. If yepnope provides that subset, then we should just use yepnope for now. Thanks!
comment:3 by , 13 years ago
Milestone: | 0.2.0 → 0.2.1 |
---|
comment:3 by , 13 years ago
Owner: | set to |
---|---|
Status: | New → In Progress |
Currently working on this bug, using yepnope :)
by , 13 years ago
Attachment: | 0001-Added-a-you-don-t-have-HTML5-so-this-video-will-not-.patch added |
---|
0001-Added-a-you-don-t-have-HTML5-so-this-video-will-not-.patch
comment:4 by , 13 years ago
As far as I could tell, the way we wanted to use it, Modernizr was a dependency of yepnope anyway! And neither were behaving the way that their documentation said they should, grr. I was able to get a JS version of the warning working, though. But **then**... Actually, `http://fortuito.us/diveintohtml5/video.html#ie <http://fortuito.us/diveintohtml5/video.html#ie>`_ and `http://camendesign.com/code/video\_for\_everybody <http://camendesign.com/code/video_for_everybody>`_ suggest that because of how the tag is designed, we don't actually need any fancy html5 detection or javascript at all! We just put our warning inside the video tag, after any and all s, and browsers that don't understand this html5 nonsense will happily show our warning to the world. Besides having less js to load and run, this implementation also makes it super easy to translate our warning (whereas before, the warning text was inside the js file, where i18n couldn't touch it!). Here's a patch that implements the javascript-less version of this feature. I tested it in Lynx and got the desired behavior; others should test it in other old-fashioned browsers!
comment:5 by , 13 years ago
Merged! Thanks! Thanks to Chris for reminding to use "git am"! Bonus points to earn: Translation tags around the text, and making things fit within 80 columns. You could change the bug status to "Feedback", that might help find issues with mergeable patches.
by , 13 years ago
Attachment: | 0001-added-translation-tags-to-html5-wwarning.patch added |
---|
0001-added-translation-tags-to-html5-wwarning.patch
comment:6 by , 13 years ago
Status: | In Progress → Feedback |
---|
Here's an additional patch with translation tags and (I think) 80-char column compatibility (unfortunately my text editor doesn't make setting the # of columns easy...)
Note:
See TracTickets
for help on using tickets.