[Approved] Option to disable 'Stop live TV' dialog (1 Viewer)

elliottmc

Retired Team Member
  • Premium Supporter
  • August 7, 2005
    14,927
    6,061
    Cardiff, UK
    Home Country
    United Kingdom United Kingdom
    To avoid confusion:
    should we include @Oxan 's branch in the next testbuild and delete @Sebastiii 's branch? Feature itself works nice. There just needs to be one config screenshot and a little text updated in Wiki.

    Thanks every dev for taking care! :)

    I have included Oxan's branch in a build. I will simply test here and if it all works fine, we should merge Oxan's branch and then delete both branches.
     

    Holzi

    Super Moderator
  • Team MediaPortal
  • April 21, 2010
    7,934
    2,235
    Ba-Wü
    Home Country
    Germany Germany
    I have included Oxan's branch in a build. I will simply test here and if it all works fine, we should merge Oxan's branch and then delete both branches.

    Thanks!
    Could you provide a screenshot of the config window to add to wiki? :)
     

    elliottmc

    Retired Team Member
  • Premium Supporter
  • August 7, 2005
    14,927
    6,061
    Cardiff, UK
    Home Country
    United Kingdom United Kingdom
    Okay, tested Oxan's branch. Working perfectly. Configuration is fine. Changing settings inside MP and checking in config is fine. Fix works fine.

    All sorted. Simple enough, so can be merged to master as soon as docs are sorted!

    Thanks @Oxan !
     

    Holzi

    Super Moderator
  • Team MediaPortal
  • April 21, 2010
    7,934
    2,235
    Ba-Wü
    Home Country
    Germany Germany
    I haven't changed the string number yet though, because I don't know which branch conflicts

    I think the string number of this branch doesn't need changes anymore. @Scythe42 took care of the string numbers in his branch. So everything should be fine with your branch. :)
     

    elliottmc

    Retired Team Member
  • Premium Supporter
  • August 7, 2005
    14,927
    6,061
    Cardiff, UK
    Home Country
    United Kingdom United Kingdom
    To avoid confusion:
    should we include @Oxan 's branch in the next testbuild and delete @Sebastiii 's branch? Feature itself works nice. There just needs to be one config screenshot and a little text updated in Wiki.

    Thanks every dev for taking care! :)

    I have included Oxan's branch in a build. I will simply test here and if it all works fine, we should merge Oxan's branch and then delete both branches.
    I have included Oxan's branch in a build. I will simply test here and if it all works fine, we should merge Oxan's branch and then delete both branches.

    Thanks!
    Could you provide a screenshot of the config window to add to wiki? :)

    Here you go!
     

    Attachments

    • Capture.JPG
      Capture.JPG
      61.5 KB

    elliottmc

    Retired Team Member
  • Premium Supporter
  • August 7, 2005
    14,927
    6,061
    Cardiff, UK
    Home Country
    United Kingdom United Kingdom

    Oxan

    Retired Team Member
  • Premium Supporter
  • August 29, 2009
    1,730
    1,124
    Home Country
    Netherlands Netherlands
    Ok, great. Awesome work guys.

    @Holzi: I already provided a screenshot in my post, but you probably missed that :p I think the skin changes aren't strictly required, but needed if the skin want to show the configuration option.
     

    Holzi

    Super Moderator
  • Team MediaPortal
  • April 21, 2010
    7,934
    2,235
    Ba-Wü
    Home Country
    Germany Germany
    Yeah, I missed that one and saw it too late. :)
    Yes, great work and big thanks to @jonywee for providing the patch!
     

    Users who are viewing this thread

    Top Bottom