Ticket #478 (closed task: fixed)
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:2 Changed 2 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 2 years ago by anna
- Status changed from closed to reopened
- Resolution ready for testing deleted
- 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
- 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.
- 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 2 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 2 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 2 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 2 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 2 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 2 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 2 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 2 years ago by dave
- Status changed from reopened to closed
- Resolution set to ready for testing
comment:12 Changed 2 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 2 years ago by and
- Status changed from reopened to closed
- Resolution set to fixed
