----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/100758/#review1683 -----------------------------------------------------------
Ship it! Running this right now, and I like it a lot. Removes a tab that contained next to no information. Presents all required information without the useless tab, but without overcrowding the UI. Makes the minimum size of the whole dialog smaller, which is always a good thing. Verdict: The "Summary" tab layout could probably be refined a bit more, but this is a great improvement over what we had before. - Mark On Feb. 27, 2011, 11:27 a.m., Ralf Engels wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/100758/ > ----------------------------------------------------------- > > (Updated Feb. 27, 2011, 11:27 a.m.) > > > Review request for Amarok. > > > Summary > ------- > > After discussion with Markey > > Statistics pane was remove > Editing of rating and score was moved to summary page. > Other meta information (albums from same artist) were removed as this > functionality is covered by the context applets and does not belong to a tag > edit dialog > > New functionality: if editing multiple tracks from same album the album cover > can still be edited and the summary page shows the total lenght and total > size of all tracks. > > > Diffs > ----- > > src/dialogs/LabelListModel.h 45dd577 > src/dialogs/LabelListModel.cpp 565da03 > src/dialogs/TagDialog.h 887eeaf > src/dialogs/TagDialog.cpp 71eba59 > src/dialogs/TagDialogBase.ui cdf33dd > > Diff: http://git.reviewboard.kde.org/r/100758/diff > > > Testing > ------- > > Editing with one and multiple tracks. > Adding, removing labels. > Switching between "per Track" and multiple tracks mode and editing various > tags. > > > Thanks, > > Ralf > >
_______________________________________________ Amarok-devel mailing list [email protected] https://mail.kde.org/mailman/listinfo/amarok-devel
