Commit c085b31e by Giulio Gratta

Merge pull request #1683 from edx/giulio/fix-video-links

Stop video skipping links from showing up when not needed and address the issue of skip links for multiple videos.
parents 07e1d361 122e0d4a
...@@ -44,7 +44,7 @@ ...@@ -44,7 +44,7 @@
<div class="focus_grabber first"></div> <div class="focus_grabber first"></div>
<div class="tc-wrapper"> <div class="tc-wrapper">
<a href="#before-transcript" class="nav-skip">${_("Skip to a navigable version of this video's transcript.")}</a> <a href="#before-transcript_${id}" class="nav-skip sr">${_("Skip to a navigable version of this video's transcript.")}</a>
<article class="video-wrapper"> <article class="video-wrapper">
<span tabindex="0" class="spinner" aria-hidden="false" aria-label="${_('Loading video player')}"></span> <span tabindex="0" class="spinner" aria-hidden="false" aria-label="${_('Loading video player')}"></span>
...@@ -83,7 +83,7 @@ ...@@ -83,7 +83,7 @@
</div> </div>
</div> </div>
</section> </section>
<a class="nav-skip" id="before-transcript" href="#after-transcript">${_('Skip to end of transcript.')}</a> <a class="nav-skip sr" id="before-transcript_${id}" href="#after-transcript_${id}">${_('Skip to end of transcript.')}</a>
</article> </article>
<ol id="transcript-captions" class="subtitles" tabindex="0" title="${_('Captions')}" role="group" aria-label="${_('Activating an item in this group will spool the video to the corresponding time point. To skip transcript, go to previous item.')}"> <ol id="transcript-captions" class="subtitles" tabindex="0" title="${_('Captions')}" role="group" aria-label="${_('Activating an item in this group will spool the video to the corresponding time point. To skip transcript, go to previous item.')}">
...@@ -91,7 +91,7 @@ ...@@ -91,7 +91,7 @@
</ol> </ol>
</div> </div>
<a class="nav-skip" id="after-transcript" href="#before-transcript">${_('Go back to start of transcript.')}</a> <a class="nav-skip sr" id="after-transcript_${id}" href="#before-transcript_${id}">${_('Go back to start of transcript.')}</a>
<div class="focus_grabber last"></div> <div class="focus_grabber last"></div>
<ul class="wrapper-downloads"> <ul class="wrapper-downloads">
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment