After working on this for a couple weeks and adding a ton of changes, I've decided to create one patch for all the changes I've made, rather than create a separate patch for each change. I'd like to get any feedback from users before committing this.
Here are the changes in the patch:
Code cleanup:
Cleans up javascript (removed unneeded complexity of using jquery plugin and reliance on jquery UI).
Separates out recorder types into three different js files for better maintainability.
Simplified CSS styling and removed images. Theming should happen in themes, not the module.
Features:
Adds video/audio stream recording for firefox (video only works in firefox nightly). This will cut down considerable on upload times.
Media recorder only asks for mic permission on record (https://drupal.org/node/2232601).
Better UI with help messages.
Better error handling on both server and client sides.
Added theme template for more customizability.
Added better audio visualization and mic meter.
Removed:
Removes youtube upload widget. I plan to move this into another module or project if a lot of people are using this.
Removed width/height settings as these should be handled by theme.
Removed time limit setting. Down the road this shouldn't be needed if we are streaming the recording. If this is used by people though I will re-add it.
TODO:
Possibly re-add time limit setting.
Add widget settings if we want to add more settings/customization.
Add back the flash settings button.
Possible add a mic setup wizard?
If you want to test the new features, I suggest testing on firefox nightly (for streamed video), firefox (for streamed audio), chrome (HTML5 non-streamed), safari (flash), and IE (flash).
Comment | File | Size | Author |
---|---|---|---|
#4 | media_recorder-2277977-4.patch | 92.23 KB | kenianbei |
Comments
Comment #1
kenianbei CreditAttribution: kenianbei commentedComment #2
Anonymous (not verified) CreditAttribution: Anonymous commentedgreat, I will try to test the patch(es) this week.
Comment #3
kenianbei CreditAttribution: kenianbei commentedMake sure to clear the cache, there was some changes to menu routing.
Comment #4
kenianbei CreditAttribution: kenianbei commentedAdded a new patch with just one small fix for the path that files are sent to.
Comment #5
kenianbei CreditAttribution: kenianbei commentedI've added this to dev since there hadn't been any feedback.
Comment #6
Anonymous (not verified) CreditAttribution: Anonymous commentedHi Kenianbei:
Sorry for the delay, I have been busy with other projects the past few months. I will try to soak the dev release and give feedback if needed early next month.
Comment #7
kenianbei CreditAttribution: kenianbei commentedI've just added an alpha5 release with all the new changes. Please test and let me know if it breaks anything with your site setup.
Comment #8
kenianbei CreditAttribution: kenianbei commentedComment #9
Anonymous (not verified) CreditAttribution: Anonymous commentedJust an FYI in case you are exploring WebRTC support ideas:
http://simplewebrtc.com/
Comment #10
kenianbei CreditAttribution: kenianbei commentedThanks for this... FYI, I have another project in early dev that uses webRTC, but instead uses the PeerJS library:
https://www.drupal.org/project/peerjs