[mythtv] [mythtv-commits] Ticket #7336: Re-implement "Delete Recordings" after MythUI port

[Nigel Pearson]

> I guess the frustrating part of this is that it is no more optional
> than any other screen in myth, then.

True, although I was more frustrated by two menu buttons
doing exactly the same thing than the theme clash.


Question... should themers be able to override the function
of the appliance, or just the eye-candy? If the former,
then the code could use *menu.xml from the themes?

Thought... if people really hate the Delete Recordings screen,
maybe discussion or a ticket should be raised to remove
it from the default menu?



> If a theme is missing that window,
> it falls back to watch recordings - the current behaviour."
> That doesn't seem to be the case.

It was for the first change [23293], but then I went
and committed the default and default-wide changes.

...
> Personally, I see no advantage to the screen existing at all,
> especially in a theme like Arclight where the space is displayed in
> Watch Recordings.

Well, not all themes do display the usage, and my household was
annoyed at having to go into System Status, scrolling over to
Machine Status, and then scrolling down past the slaves to total.


>  I am not certain, but I think if the window
> definition was removed from default and default-wide, and only added
> to themes that specifically elect to use it, that it might behave as
> you had described, but if it exists in the defaults, loadfromxml will
> *always* return true and the fallback will never happen.

Correct.



I will consider removing the window def from default*,
and maybe adding it into Metallurgy.
Gimme a few days to get feedback (and some sleep).

--
Nigel Pearson, nigel at ind.tansu.com.au|"Reality is that which,  |
Telstra Net. Eng., Sydney, Australia | when you stop believing |
Office: 9202 3900    Fax:  9212 6348 | in it, doesn't go away."|
Mobile: 0408 664435  Home: 9792 6998 | Philip K. Dick - 'Valis'|