Opened 16 years ago

Closed 16 years ago

#9135 closed defect (Fixed)

MythBuntu needs plugin checkbox on MythBrowser setup screen

Reported by: Jonathan Martens <jonathan@…> Owned by: paulh
Priority: minor Milestone: unknown
Component: Plugin - MythBrowser Version: 0.24-fixes
Severity: medium Keywords:
Cc: Ticket locked: no

Description

While running the MythBuntu team with 0.24-fixes I see the following in the log (-v "all"): 

2010-10-22 14:33:44.613 XMLParseBase: Loading window theme from D:/mythtv/build/share/mythtv/themes/Mythbuntu/browser-ui.xml
2010-10-22 14:33:44.613 MythUIHelper: LoadCacheImage(cursor.png,cursor.png--1x26.png)
2010-10-22 14:33:44.613 MythUIImage(0x2d677b58): Load(), loading 'cursor.png' in foreground
2010-10-22 14:33:44.613 MythUIImage(0x2d677b58): LoadImage(cursor.png) Object cursor
2010-10-22 14:33:44.613 MythUIHelper: LoadCacheImage(cursor.png,cursor.png--1x26.png)
2010-10-22 14:33:44.613 MythUIImage(0x2d677b58): LoadImage found in cache :cursor.png--1x26.png: RefCount = 6
2010-10-22 14:33:44.613 MythUIHelper: LoadCacheImage(cursor.png,cursor.png--1x26.png)
2010-10-22 14:33:44.613 MythUIImage(0x2d69c5a8): Load(), loading 'cursor.png' in foreground
2010-10-22 14:33:44.613 MythUIImage(0x2d69c5a8): LoadImage(cursor.png) Object cursor
2010-10-22 14:33:44.613 MythUIHelper: LoadCacheImage(cursor.png,cursor.png--1x26.png)
2010-10-22 14:33:44.613 MythUIImage(0x2d69c5a8): LoadImage found in cache :cursor.png--1x26.png: RefCount = 7
2010-10-22 14:33:44.629 Theme is missing critical theme elements.

In the dev mailinglist it was concluded that this must be due to the late implementation of the enable plugins checkbox and it was concluded that it should be raised as an issue in Trac. When I remove the offending browser-ui.xml file, so MythFrontend takes the one from the default theme the checkbox is visible and the error is no longer in the log file.

Attachments (4)

mythbuntu.pluginscheck.diff​ (812 bytes ) - added by robertm 16 years ago.
mythbuntu.jpeg​ (200.7 KB ) - added by Jonathan Martens <jonathan@…> 16 years ago.
screenshot after applying patch
mythbuntu.pluginscheck.2.diff​ (812 bytes ) - added by robertm 16 years ago.
mythbuntu2.jpeg​ (200.0 KB ) - added by Jonathan Martens <jonathan@…> 16 years ago.
screenshot after applying 2nd patch

Download all attachments as: .zip

Change History (10)

comment:1 by robertm, 16 years ago

Jonathan,

Please apply this patch and test, as well as taking a screenshot if possible. Can't test from work but this ought to do the trick.

by robertm, 16 years ago

Attachment: mythbuntu.pluginscheck.diff​ added

by Jonathan Martens <jonathan@…>, 16 years ago

Attachment: mythbuntu.jpeg​ added

screenshot after applying patch

comment:2 by Jonathan Martens <jonathan@…>, 16 years ago

The error is gone and the plugin checkbox and it's description are visible, not sure if the layout is the best with current patch though.

comment:3 by robertm, 16 years ago

Hi Jonathan,

Try this one, please.

by robertm, 16 years ago

Attachment: mythbuntu.pluginscheck.2.diff​ added

by Jonathan Martens <jonathan@…>, 16 years ago

Attachment: mythbuntu2.jpeg​ added

screenshot after applying 2nd patch

comment:4 by Jonathan Martens <jonathan@…>, 16 years ago

Looks better IMHO

comment:5 by robertm, 16 years ago

(In [26953]) Add required checkbox to Mythbuntu theme. Unfortunately terminate the string with a colon to prevent new translation strings. To fix later. Refs #9135.

comment:6 by robertm, 16 years ago

Resolution: → Fixed
Status: new → closed

(In [26954]) Backport r26953 to .24-fixes. Add required checkbox to Mythbuntu theme. Fixes #9135.

Note: See TracTickets for help on using tickets.