Download button on video_view template - change text to "Download Hi-Res / Screening Version"

[and]

Many people who see the download button under the flash player may think it's to download the flash quality version of that video. We need a way to indicate that its actually a high res version.

[anna]

Made this change in ATVideo in testing.engagemedia.org

Could not commit to svn as I don't have svn access to Plone collective.

Note: andycat has made changes to video_view.pt that are not ready to commit to svn, along with the change I made (just this line): + Download Hi-Res / Screening Version

--- video_view.pt (revision 48985) +++ video_view.pt (working copy) @@ -16,7 +16,6 @@

<div metal:use-macro="here/document_byline/macros/byline">

Get the byline - contains details about author and modification date.

</div>

- <div style="float:left; width: 70%">

<div class="videoitemtop"

tal:condition="here/Producer">

<span style="font-weight: bold;">Produced by</span>

@@ -81,8 +80,6 @@

</metal:block>

</div>

- </div> - <div style="float:right; width:30%">

<div class="videoitemimage" style="float:right; text-align: center; clear:both;">

<div tal:define="item_wf_state here/review_state|python: here.portal_workflow.getInfoFor(here, 'review_state', );

@@ -128,7 +125,7 @@

tal:define="enc here/getFileAttribs" tal:condition="python: enc[1] > 0">

<img tal:replace="structure here/download.gif" />

- Download video + Download Hi-Res / Screening Version

</a>

<br />

@@ -140,7 +137,6 @@

<img tal:replace="structure here/torrent.png" />

Download via BitTorrent

</a>

- </div>

</div>

<!--

<div tal:content="structure python: here.getMediaWidget()" />

[anna]

Last comment looked screwed up, here it is again:

Made this change in ATVideo in testing.engagemedia.org

Could not commit to svn as I don't have svn access to Plone collective.

Note: andycat has made changes to video_view.pt that are not ready to commit to svn, along with the change I made (just this line):

+ Download Hi-Res / Screening Version

--- video_view.pt       (revision 48985)
+++ video_view.pt       (working copy)
@@ -16,7 +16,6 @@
        <div metal:use-macro="here/document_byline/macros/byline">
                   Get the byline - contains details about author and modification date.
        </div>
-        <div style="float:left; width: 70%">
          <div class="videoitemtop"
              tal:condition="here/Producer">
               <span style="font-weight: bold;">Produced by</span> 
@@ -81,8 +80,6 @@
             </metal:block>
 
         </div>
-        </div>
-       <div style="float:right; width:30%">
           <div class="videoitemimage" style="float:right; text-align: center;  clear:both;">
 
             <div tal:define="item_wf_state here/review_state|python: here.portal_workflow.getInfoFor(here, 'review_state', '');
@@ -128,7 +125,7 @@
                tal:define="enc here/getFileAttribs"
                tal:condition="python: enc[1] > 0">
                  <img tal:replace="structure here/download.gif" />
-               Download video
+               Download Hi-Res / Screening Version
            </a>
 
            <br />
@@ -140,7 +137,6 @@
                  <img tal:replace="structure here/torrent.png" />
                Download via BitTorrent
            </a>
-       </div>
         </div>
 <!--
         <div tal:content="structure python: here.getMediaWidget()" />

[anna]

I did have access, so I committed to Plone collective.

http://dev.plone.org/collective/changeset/49669

I also changed on stable www.engagemedia.org filesystem in order to see this change now on our live instance.

[anonymous]

Missing or invalid form token. Do you have cookies enabled?

[anonymous]

Thank you. ed hardy