Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#3916 closed patch (fixed)

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 (2)

mythflix.cpp.browseMenu.diff (2.7 KB) - added by Britney Fransen <britney.fransen@…> 12 years ago.
mythflix.h.browseMenu.diff (849 bytes) - added by Britney Fransen <britney.fransen@…> 12 years ago.

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by Britney Fransen <britney.fransen@…>

Changed 12 years ago by Britney Fransen <britney.fransen@…>

Attachment: mythflix.h.browseMenu.diff added

comment:1 Changed 12 years ago by kkuphal

Resolution: fixed
Status: newclosed

(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 12 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.

Note: See TracTickets for help on using tickets.