[vlc-devel] [PATCH] commented the lines for item removal to remove the option "Remove selected"

[Avani Sharma]

Because it is misleading according to issue #16551

issue url:
https://trac.videolan.org/vlc/ticket/16551#comment:1

Avani


On Fri, Feb 26, 2016 at 10:01 PM, Jean-Baptiste Kempf <jb at videolan.org>
wrote:

> Why?
>
>
> On 25 Feb, Avani wrote :
> > ---
> >  modules/gui/qt4/components/playlist/standardpanel.cpp | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/modules/gui/qt4/components/playlist/standardpanel.cpp
> b/modules/gui/qt4/components/playlist/standardpanel.cpp
> > index 1178647..d1c25c0 100644
> > --- a/modules/gui/qt4/components/playlist/standardpanel.cpp
> > +++ b/modules/gui/qt4/components/playlist/standardpanel.cpp
> > @@ -232,10 +232,10 @@ bool StandardPLPanel::popup( const QPoint &point )
> >      menu.addSeparator();
> >
> >      /* Item removal */
> > -
> > +    /* Uncomment the following lines to activate the Remove Selected
> option
> >      ADD_MENU_ENTRY( QIcon( ":/buttons/playlist/playlist_remove" ),
> qtr(I_POP_DEL),
> >                      VLCModelSubInterface::ACTION_REMOVE );
> > -
> > +    */
> >      ADD_MENU_ENTRY( QIcon( ":/toolbar/clear" ), qtr("Clear the
> playlist"),
> >                      VLCModelSubInterface::ACTION_CLEAR );
> >
> > --
> > 2.1.4
> >
> > _______________________________________________
> > vlc-devel mailing list
> > To unsubscribe or modify your subscription options:
> > https://mailman.videolan.org/listinfo/vlc-devel
>
> --
> With my kindest regards,
>
> --
> Jean-Baptiste Kempf
> http://www.jbkempf.com/ - +33 672 704 734
> Sent from my Electronic Device
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20160226/6d55ad5c/attachment.html>