[Patch] "No grouping" checkbox in playlist layout

Tim Bocek tim.bocek at gmail.com
Thu Sep 3 16:28:04 UTC 2009


OK, I deleted the old merge request and created a new one.  Hopefully this
will merge more cleanly!

On Thu, Sep 3, 2009 at 1:53 AM, Nikolaj Hald Nielsen <
nhnfreespirit at gmail.com> wrote:

> Tim
>
> I just tried your patch (merging your repo) and it causes a few
> conflicts with current master. Could you please rebase and fix the
> conflicts? Then I will be happy to push this to master (after a bit of
> testing)
>
> - Nikolaj
>
> On Thu, Sep 3, 2009 at 12:25 AM, Ian Monroe<ian.monroe at gmail.com> wrote:
> > On Wed, Sep 2, 2009 at 4:49 PM, Tim Bocek<tim.bocek at gmail.com> wrote:
> >> I just finished a patch that implements tying grouping schemes to
> playlist
> >> layouts, and created a merge request at:
> >>
> >> http://www.gitorious.org/amarok/amarok/merge_requests/1415
> >>
> >> If this is too much given the state of the freeze (it does add some
> extra
> >> strings), I'm happy to just let this sit until 2.2.1.
> >
> > Also meant to mention: delete the commented out code. Commented code
> > is confusing and pointless.
> >
> > Ian
> > _______________________________________________
> > Amarok mailing list
> > Amarok at kde.org
> > https://mail.kde.org/mailman/listinfo/amarok
> >
> _______________________________________________
> Amarok mailing list
> Amarok at kde.org
> https://mail.kde.org/mailman/listinfo/amarok
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/amarok/attachments/20090903/cb03fc1b/attachment.html>


More information about the Amarok mailing list