Opened 14 years ago
Closed 13 years ago
Last modified 13 years ago
#7425 closed patch (fixed)
Mythvideo doesn't properly behaves with "Video Tree remembers last position" flag
| Reported by: | JYA | Owned by: | paulh |
|---|---|---|---|
| Priority: | minor | Milestone: | 0.25 |
| Component: | Plugin - MythVideo | Version: | head |
| Severity: | low | Keywords: | |
| Cc: | Ticket locked: | yes |
Description
Mythvideo can be set to remember the last position when exiting. So when you go back into mythvideo you are were you last left.
The setting: Setup -> Media Settings -> Videos -> General Settings -> Page 2 "Video tree remembers last position"
When going into Mythvideo, always back at the root of the directory.
Gallery view or Listing view
Attachments (4)
Change History (23)
comment:1 Changed 14 years ago by
| Component: | MythTV - General → Plugin - MythVideo |
|---|---|
| Milestone: | 0.22 → unknown |
| Owner: | changed from Isaac Richards to Anduin Withers |
| Severity: | medium → low |
| Version: | 0.22rc1 → head |
comment:2 Changed 14 years ago by
comment:3 Changed 14 years ago by
Please don't remove the setting, but fix it instead.
I have a large video collection stored in a hierarchy of folders, and re-navigating every time I switch between videos and recordings is frustrating (more-so because my wife complains).
I really enjoyed that setting, and I am currently subscribed to updates from the 0.22-fixes through mythbuntu, so if it got fixed I could appreciate it immediately.
Is anyone able to work on this? Is this something I could reasonable create a patch for if I were to download/compile the source?
Changed 14 years ago by
| Attachment: | 0001-Fixes-7425.-Update-tree-position-remmeber-functiona.patch added |
|---|
Fixes "Save tree position" functionality in MythVideo?
Changed 14 years ago by
| Attachment: | 0001-Fixes-7425 - updated.patch added |
|---|
comment:5 Changed 14 years ago by
| Ticket locked: | unset |
|---|
comment:6 Changed 14 years ago by
I am very interested in this patch, since this feature is beyond useful to me, and I've missed it dearly in 0.22-fixes. If I can help out by testing it, let me know what I need to do (in 20 words or less, I'll figure out the details.) Thanks.
comment:7 Changed 14 years ago by
Hi thanks heaps for the patch i tried it but it didnt seem to work
comment:9 Changed 14 years ago by
| Milestone: | unknown → 0.23 |
|---|---|
| Type: | defect → patch |
comment:10 follow-up: 11 Changed 14 years ago by
| Milestone: | 0.23 → 0.24 |
|---|
comment:11 Changed 14 years ago by
The attached patch is against trunk revision 24017. It includes the patch from above but expands the functionality. The video position is now also remembered for the gallery view (which I use).
Robert S
Changed 14 years ago by
| Attachment: | mythvideo-remember-tree-and-gallery-position.patch added |
|---|
Remember video position also in gallery view mode
Changed 14 years ago by
| Attachment: | mythvideo-remember-tree-position.patch added |
|---|
Improved patch against trunk 24740
comment:12 Changed 14 years ago by
| Milestone: | 0.24 → unknown |
|---|
comment:13 Changed 14 years ago by
| Owner: | changed from Anduin Withers to robertm |
|---|---|
| Status: | new → assigned |
comment:14 Changed 14 years ago by
| Milestone: | unknown → 0.25 |
|---|
comment:16 Changed 13 years ago by
Do any of these patches work with .24 fixes i tryed mythvideo-remember-tree-position.patch which applied cleanly but it didnt seem to work and 0001-Fixes-7425 - updated.patch doesnt apply cleanly which is what i was using with .23 fixes.
comment:17 Changed 13 years ago by
| Ticket locked: | set |
|---|
Oobe, you've been spoken to about ignoring trac rules before.
comment:18 Changed 13 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
MythVideo?: Fix the remember position setting.
This fixes the setting to restore the same position in browser, gallery and tree views when returning to MythVideo? or when switching views. Fixes #7425.
Branch: master Changeset: b03a6b6acecaf1dbd6b28c3d0ca75a66a6f0feaa
comment:19 Changed 13 years ago by
| Owner: | changed from robertm to paulh |
|---|
Then that option should be removed from the settings if it doesn't do anything in 0.22