[mythtv-commits] Ticket #2533: setting-revamp : review + test/fix + merge

[MythTV]

#2533: setting-revamp : review + test/fix + merge
---------------------+------------------------------------------------------
 Reporter:  danielk  |        Owner:  danielk
     Type:  task     |       Status:  new    
 Priority:  trivial  |    Milestone:  unknown
Component:  mythtv   |      Version:  head   
 Severity:  low      |   Resolution:         
---------------------+------------------------------------------------------
Changes (by danielk):

  * priority:  minor => trivial

Comment:

 I've done one review+cleanup pass over all the significantly modified
 files in the settings-revamp branch. Most of the broken parts have TODO's
 attached.

 An exception is the settings.h classes which will need to have multiple
 QObject inheritence and virtual QObject inheritence removed. This has to
 be fixed whether this branch is ever merged or not.

 I have not verified that there are no missing configuration controls in
 globalsettings.{cpp,h} by comparing it closely with the current version,
 but I added back any controls I noticed were missing.

 Also, the keybinding code in this branch is broken.

 Finally, the expert mode shows shows you the expert options for all
 screens when you pull it up for any screen, with no filtering available.
 Because of this, I've disabled the expert options for now and placed them
 on the last advanced pane[s] of each configuration screen.

 This isn't really usable at the moment, but it is probably worth it to get
 it working rather than re-implementing the idea. However, I'm putting this
 on my "to merge" queue after the mythtv-vid and multiuser branches.

-- 
Ticket URL: <http://cvs.mythtv.org/trac/ticket/2533#comment:11>
MythTV <http://www.mythtv.org/>
MythTV