[PATCH] kcmaudiocd - Ogg Vorbis settings

[Jason Lane]

Attached is an improved version of my patch -- addressing the points below. 

On Thursday 29 August 2002 1:09 pm, Ryan Cumming wrote:
> 1) Having widgets disappear and reappear based on other widgets is
> generally considered bad form. It'd be better if both groups were always
> displayed, but appropriately disabled. I'll look in to fixing the MP3 tab
> to do the same.
I was following the 'example' of the MP3 tab... but now fixed. The hide() and 
show() calls changed to setEnabled(). I also took the liberty of changing the 
hide()/show() calls for the MP3 tab, too, for consistency. It does have the 
side effect of making the dialogue require a scroll bar on an 800x600 
display, though.

> 2) As per the styleguide, radiobuttons should use sentence-style
> capitalization, not title.
A subtle thing to overlook... fixed.

> 3) Yes, we're in a feature freeze, but if the UI issues are cleaned up, it
> seems small enough to fit in.
:-)

Thanks for spending the time to have a look at my patch. Cheers,

Jason
-------------- next part --------------
A non-text attachment was scrubbed...
Name: vorbis-kcmaudiocd.patch.bz2
Type: application/x-bzip2
Size: 2768 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-multimedia/attachments/20020829/138294b6/attachment.bin>