Ticket #3916 (closed patch: fixed)
Opened 5 years ago
Last modified 5 years ago
Patch for Netflix Browse Menu
| Reported by: | Britney Fransen <britney.fransen@…> | Owned by: | kkuphal |
|---|---|---|---|
| Priority: | minor | Milestone: | unknown |
| Component: | mythflix | Version: | 0.20-fixes |
| Severity: | medium | Keywords: | |
| Cc: | Ticket locked: | no |
Description
The attached patch to mythflix.cpp and mythflix.h adds the following:
- Added a Browse Menu that activates when M is pressed
- Added "Show Netflix Page" item to the Browse menu that launches Mythbrowser to show the Netflix page for the selected movie
Attachments
Change History
Changed 5 years ago by Britney Fransen <britney.fransen@…>
- Attachment mythflix.cpp.browseMenu.diff added
Changed 5 years ago by Britney Fransen <britney.fransen@…>
- Attachment mythflix.h.browseMenu.diff added
comment:1 Changed 5 years ago by kkuphal
- Status changed from new to closed
- Resolution set to fixed
(In [14390]) Fixes #3912 and Fixes #3916 by applying slightly modified patch which adds Show Netflix page to queue actions to display page in MythBrowser?. Modified patch to display menu on SELECT in Browse instead of adding MENU button. Seems more intuitive since there was no visual feedback to SELECT previously.
comment:2 Changed 5 years ago by Britney Fransen <britney.fransen@…>
Right now in Queue both SELECT and MENU will show the menu. However in Browse only SELECT will show the menu. For consistency I think that the behavior should be the same in both Browse and Queue. If I get a vote I would like to have both SELECT and MENU display the menu in both Browse and Queue. If only one is to be used I would use MENU as that seems to be the convention across other parts of Myth and the plugins.
