Ticket #14 (closed enhancement: fixed)
Download button on video_view template - change text to "Download Hi-Res / Screening Version"
| Reported by: | and | Owned by: | anna |
|---|---|---|---|
| Priority: | major | Milestone: | 0.2 |
| Component: | ATVideo | Severity: | |
| Keywords: | Cc: | ||
| Who will test this: |
Description
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.
Attachments
Change History
comment:1 Changed 6 years ago by and
- Summary changed from Download button confusion to Download button confusion - add text "high-res"
- Milestone changed from 0.3 to 0.2
comment:2 Changed 6 years ago by anna
- Summary changed from Download button confusion - add text "high-res" to Download button on video_view template - change text to "Download Hi-Res / Screening Version"
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()" />
comment:3 Changed 6 years ago by 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()" />
comment:4 Changed 6 years ago by 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.
