Modify

Ticket #478 (closed task: fixed)

Opened 4 years ago

Last modified 3 years ago

Other videos by this author

Reported by: and Owned by: dave
Priority: major Milestone: 4.4 .Beta - Video Publishing
Component: DesignCSS Severity:
Keywords: Cc:
Who will test this: And

Description (last modified by anna) (diff)

Currently the only way to see other videos by the same author is to click their author link.

It would be great if in the new design you could see a few of these videos in video/view page.

Add these at the bottom of the page for now, they can be slotted into re-design later.

Attachments

Change History

comment:1 Changed 3 years ago by anna

  • Owner set to dave
  • Description modified (diff)

comment:2 Changed 3 years ago by dave

  • Status changed from new to closed
  • Resolution set to ready for testing

This has been done, taking a random sampling of up to five video's of the user

comment:3 Changed 3 years ago by anna

  • Status changed from closed to reopened
  • Resolution ready for testing deleted
  1. nice, just need to fix it so the thumbnail shows up, as currently its showing a broken image:

 http://testing.plumi.org:16080/Plone/Members/reviewer/videos/test%20for%20222%20new/view

  1. and also can it exclude the actual video that we are already looking at? e.g.:

 http://testing.plumi.org:16080/Plone/Members/reviewer/videos/test%20for%20222%20new/view

the video that shows there from the same author is the same one we're already looking at.

  1. also - that video that belongs to reviewer also shows up here in a video belonging to test:

 http://testing.plumi.org:16080/Plone/Members/test/videos/test-video/view

is this because its going on owner, not creator? if so i guess we need to change it to creator, given that we haven't found a solution for the owner / owner role / multiple owner issue.

comment:4 Changed 3 years ago by anna

actually, from looking at this on staging:  http://staging.engagemedia.org/Members/anna/videos/transcoding-test-three/view

...is it possible to lay these out in a grid? so two sit side by side underneath the main video content area, as in a table.

i suppose in this case it would be better to limit it to 4 random videos.

this is just so the comments area doesn't get pushed too far down.

FYI the image thumbnails do appear in staging, but not in dave testing site.

comment:5 Changed 3 years ago by and

Hey, looks great. Just had a quick look at my page  http://staging.engagemedia.org/Members/andrew/videos/OpenVideoConf.flv/view

The Shoot on Sight and Entah | Confused videos shouldn't come up in my listing, they were uploaded by me but I put them in other people's folders.

Couple of other things

  • The titles aren't links, i think this would be good
  • Layed out in a grid might be better, though perhaps it could be quite a bit smaller and sit along side the full description taking up 1/4-1/3 of the realestate, am thinking as anna mentioned that if it's right at the bottom no one will see it, plus the comments area issue too.

comment:6 Changed 3 years ago by dave

1/ It looks as tho videos without a default image are showing broken, is there a default video image I could use for video's that have no image uploaded?

2/ will get onto this now

3/ The lookup is based on creator (there is no catalog entry for owner to lookup on), not sure how it is doing this. Was the video created by that user, then ownership changed?

And : I don't think I have access to staging, it is missing the css I added in plumi.css to do the grids.

 http://svn.plone.org/svn/collective/plumi.skin/trunk/plumi/skin/browser/stylesheets/plumi.css

In the section

/* - Authors latest video's on a video view - */

#authors_latest {
   float:left;
   width:100%;
   background:#fff;
   overflow:hidden;
   position:relative;
}

#authors_latest ul {
   clear:left;
   float:left;
   list-style:none;
   margin:0;
   padding:0;
   position:relative;
   left:50%;
   text-align:center;
}

#authors_latest ul li img {
   height:100px;
   width:130px;
}

#authors_latest ul li {
   display:block;
   width:150px;
   height:200px;
   float:left;
   list-style:none;
   margin:0;
   padding:0;
   position:relative;
   right:50%;
}

#authors_latest ul li a {
   display:block;
   margin:0 0 0 1px;
   padding:3px 10px;
   background:#ddd;
   color:#000;
   text-decoration:none;
   line-height:1.3em;
}

#authors_latest ul li a:hover {
   background:#369;
   color:#fff;
}

#authors_latest ul li a.active,
#authors_latest ul li a.active:hover {
   color:#fff;
   background:#000;
   font-weight:bold;
}

comment:7 Changed 3 years ago by and

Hey Dave, with how the vids are bring brought in, this is not just with my page but also others  http://staging.engagemedia.org/Members/dherdiany/videos/Dengar-confused.mp4/view

That for example show's videos that are actually mine, but that author shouldn't have anything at all to do with my videos.

Is there a way of just bringing in the videos that are in their videos folder. IN this case of the user above, what is at  http://staging.engagemedia.org/Members/dherdiany/videos

comment:8 Changed 3 years ago by and

Re-layout, spoke with Dave, I reckon how it is here  http://testing.plumi.org:16080/Plone/Members/manager/videos/test-ownership/view is ok for now, given we're looking at a broader redesign down the track.

comment:9 Changed 3 years ago by dave

In regard comment 7, with staging.em there is no way I can check what code version is enabled or incorrect, I am also working on changes to the code atm regarding a default image and the listings, or excluding those videos without one from display. There also appears to be some issue with searching on Creator, between doing so in video.py with self.context.Creator() vs self.context.video_file.Creator() which I think caused the issue.

comment:10 Changed 3 years ago by mike

staging is on the same server as testing is. I updated plumi.skin and plumi.content to the current trunk. Everything should be up to date now.

comment:11 Changed 3 years ago by dave

  • Status changed from reopened to closed
  • Resolution set to ready for testing

comment:12 Changed 3 years ago by and

  • Status changed from closed to reopened
  • Resolution ready for testing deleted

Looks good. Dave want to update testing.plumi with this also? Be good to see it with a bunch more videos.

comment:13 Changed 3 years ago by and

  • Status changed from reopened to closed
  • Resolution set to fixed
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.