ICEfaces
  1. ICEfaces
  2. ICE-9932

ace:videoPlayer mute does not function in certain configurations.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0.BETA
    • Fix Version/s: 4.0
    • Component/s: ACE-Components
    • Labels:
      None
    • Environment:
      Tomcat 7, ios 6 or 7, or android 4.1.1., IE 9
    • Assignee Priority:
      P2
    • Affects:
      Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration

      Description

      Mute attribute in dynamic attribute test does not work on ios 6 or 7, android 4.1.1, 4.4.2 or IE9.

      Test app located at:
      http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/videoPlayer
      Go to dynamic attribute test. Select rendered and muted, then play video. Audio is not muted.

        Activity

        Liana Munroe created issue -
        Liana Munroe made changes -
        Field Original Value New Value
        Description Mute attribute in dynamic attribute test does not work on ios 6 or 7, android 4.1.1. or IE9.
         It does work on Android version 4.4.2 and desktop browsers FF and Chrome.

        Test app located at:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/videoPlayer
        Go to dynamic attribute test. Select rendered and muted, then play video. Audio is not muted.
        Mute attribute in dynamic attribute test does not work on ios 6 or 7, android 4.1.1, 4.4.2 or IE9.

        Test app located at:
        http://dev.icesoft.com/svn/repo/qa/trunk/Regression-Icefaces4/Sparkle/Manual/videoPlayer
        Go to dynamic attribute test. Select rendered and muted, then play video. Audio is not muted.
        Ken Fyten made changes -
        Assignee Judy Guglielmin [ judy.guglielmin ]
        Fix Version/s 4.0 [ 11382 ]
        Affects Version/s 4.0.BETA [ 10770 ]
        Assignee Priority P2 [ 10011 ]
        Hide
        Judy Guglielmin added a comment -

        The following link shows the attributes of the html5 video tag and where they are supported:-
        http://www.jwplayer.com/html5/autoloop/

        This is something that must be taken into account when using the media tags for html5.
        From
        https://developer.apple.com/library/safari/documentation/audiovideo/conceptual/using_html5_audio_video/device-specificconsiderations/device-specificconsiderations.html
        you can see that autoPlay is specifically disabled for the following reason:-
        ".... autoplay is disabled to prevent unsolicited cellular download."

        what the iOS docs say regarding muted/volume control is:-
        "...On iOS devices, the audio level is always under the user’s physical control. The volume property is not settable in JavaScript. Reading the volume property always returns 1."

        So...this should probably be documented for iOS, android, and any other mobile devices. Since ace:videoPlayer is not a mobile tag, it does support these attributes for the desktop browsers that are html5 compliant (very important to note that), however, when used in mobility, they are only supported as per the video tag on the mobile devices as well ...all for very good reason.

        Show
        Judy Guglielmin added a comment - The following link shows the attributes of the html5 video tag and where they are supported:- http://www.jwplayer.com/html5/autoloop/ This is something that must be taken into account when using the media tags for html5. From https://developer.apple.com/library/safari/documentation/audiovideo/conceptual/using_html5_audio_video/device-specificconsiderations/device-specificconsiderations.html you can see that autoPlay is specifically disabled for the following reason:- ".... autoplay is disabled to prevent unsolicited cellular download." what the iOS docs say regarding muted/volume control is:- "...On iOS devices, the audio level is always under the user’s physical control. The volume property is not settable in JavaScript. Reading the volume property always returns 1." So...this should probably be documented for iOS, android, and any other mobile devices. Since ace:videoPlayer is not a mobile tag, it does support these attributes for the desktop browsers that are html5 compliant (very important to note that), however, when used in mobility, they are only supported as per the video tag on the mobile devices as well ...all for very good reason.
        Judy Guglielmin made changes -
        Assignee Judy Guglielmin [ judy.guglielmin ] Ken Fyten [ ken.fyten ]
        Ken Fyten made changes -
        Security Private [ 10001 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #42494 Wed Sep 03 13:08:05 MDT 2014 judy.guglielmin ICE-9932 notes in tlddoc for muted
        Files Changed
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/videoplayer/VideoPlayerMeta.java
        Commit graph MODIFY /icefaces4/trunk/icefaces/ace/component/src/org/icefaces/ace/component/audioplayer/AudioPlayerMeta.java
        Hide
        Judy Guglielmin added a comment -

        notes added to tlddoc to confirm that muted attribute is only going to work on html5 compliant desktop browsers.

        Show
        Judy Guglielmin added a comment - notes added to tlddoc to confirm that muted attribute is only going to work on html5 compliant desktop browsers.
        Judy Guglielmin made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten made changes -
        Affects Documentation (User Guide, Ref. Guide, etc.),Compatibility/Configuration [ 10003, 10002 ]
        Hide
        Liana Munroe added a comment - - edited

        Verified ICEfaces 4 trunk r42510. Changed to html5 tags in ace:videoPlayer Dynamic attribute QA test application.

        Show
        Liana Munroe added a comment - - edited Verified ICEfaces 4 trunk r42510. Changed to html5 tags in ace:videoPlayer Dynamic attribute QA test application.
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Ken Fyten
            Reporter:
            Liana Munroe
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: