Opened 17 years ago
Closed 17 years ago
Last modified 17 years ago
#3916 closed patch (fixed)
Patch for Netflix Browse Menu
Reported by: | 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 (2)
Change History (4)
Changed 17 years ago by
Attachment: | mythflix.cpp.browseMenu.diff added |
---|
Changed 17 years ago by
Attachment: | mythflix.h.browseMenu.diff added |
---|
comment:1 Changed 17 years ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
comment:2 Changed 17 years ago by
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.
Note: See
TracTickets for help on using
tickets.
(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.