Modify

Ticket #14 (closed enhancement: fixed)

Opened 7 years ago

Last modified 6 years ago

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 7 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 7 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 7 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 7 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.

comment:5 Changed 7 years ago by anna

  • Owner changed from andycat to anna

comment:6 Changed 7 years ago by anna

  • Status changed from new to closed
  • Resolution set to fixed

comment:7 Changed 6 years ago by anonymous

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

comment:8 Changed 6 years ago by anonymous

Thank you.  ed hardy

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'reopened'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.