Junior Job - external MySQL config

[John Atkinson]

> Is the combobox with “MySQL” needed?

Strictly, no. I kept it in though in though as (in my opinion) the checkbox
looks odd on its own. Also, it would obviously be required should Amarok
support other engines in the future (like Amarok 1.4 did PostgreSql).

That is subjective though. If there are no plans for other engines then it
would make sense to remove it. In that case, perhaps the dialog should be
more specific as well and note being for "MySQL Configuration" rather than
the generic "database".

- John
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok/attachments/20090829/7f8635e8/attachment.html>