]> git.sesse.net Git - vlc/blobdiff - modules/gui/wxwindows/playlist.cpp
include services discovery in interface
[vlc] / modules / gui / wxwindows / playlist.cpp
index ffbeaa2bca2c030e512b57d4062346b594e82abd..a95125384ef910d5d7fd22e71968c8ee552a5e32 100644 (file)
@@ -2,11 +2,12 @@
  * playlist.cpp : wxWindows plugin for vlc
  *****************************************************************************
  * Copyright (C) 2000-2004 VideoLAN
- * $Id: playlist.cpp,v 1.38 2004/01/16 00:01:19 gbazin Exp $
+ * $Id$
  *
  * Authors: Olivier Teulière <ipkiss@via.ecp.fr>
+ *          Clément Stenac <zorglub@videolan.org>
  *
- * This program is free software; you can redistribute it and/or modify
+ * This program is free software; you can redistribute it and/OR MODIFy
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 2 of the License, or
  * (at your option) any later version.
 #include <vlc/intf.h>
 
 #include "wxwindows.h"
-#include <wx/listctrl.h>
+
+#include "bitmaps/shuffle.xpm"
+#include "bitmaps/repeat.xpm"
+#include "bitmaps/loop.xpm"
+
+#include "bitmaps/type_unknown.xpm"
+#include "bitmaps/type_net.xpm"
+#include "bitmaps/type_card.xpm"
+#include "bitmaps/type_disc.xpm"
+#include "bitmaps/type_directory.xpm"
+#include "bitmaps/type_playlist.xpm"
+
+#include <wx/dynarray.h>
+#include <wx/imaglist.h>
+
+#define HELP_SHUFFLE N_( "Shuffle" )
+#define HELP_LOOP N_( "Loop" )
+#define HELP_REPEAT N_( "Repeat" )
 
 /* Callback prototype */
-int PlaylistChanged( vlc_object_t *p_this, const char *psz_variable,
-                     vlc_value_t old_val, vlc_value_t new_val, void *param );
-int PlaylistNext( vlc_object_t *p_this, const char *psz_variable,
-                     vlc_value_t old_val, vlc_value_t new_val, void *param );
-int ItemChanged( vlc_object_t *p_this, const char *psz_variable,
-                     vlc_value_t old_val, vlc_value_t new_val, void *param );
+static int PlaylistChanged( vlc_object_t *, const char *,
+                            vlc_value_t, vlc_value_t, void * );
+static int PlaylistNext( vlc_object_t *, const char *,
+                         vlc_value_t, vlc_value_t, void * );
+static int ItemChanged( vlc_object_t *, const char *,
+                        vlc_value_t, vlc_value_t, void * );
 
 /*****************************************************************************
  * Event Table.
@@ -56,8 +74,6 @@ enum
     RSortTitle_Event,
     SortAuthor_Event,
     RSortAuthor_Event,
-    SortGroup_Event,
-    RSortGroup_Event,
     Randomize_Event,
 
     EnableSelection_Event,
@@ -70,14 +86,12 @@ enum
     Repeat_Event,
     SelectAll_Event,
 
-    EnableGroup_Event,
-    DisableGroup_Event,
-
     Up_Event,
     Down_Event,
     Infos_Event,
 
     PopupPlay_Event,
+    PopupPlayThis_Event,
     PopupDel_Event,
     PopupEna_Event,
     PopupInfo_Event,
@@ -86,12 +100,27 @@ enum
     Search_Event,
 
     /* controls */
-    ListView_Event,
+    TreeCtrl_Event,
 
     Browse_Event,  /* For export playlist */
+
+    /* custom events */
+    UpdateItem_Event,
+
+    MenuDummy_Event = wxID_HIGHEST + 999,
+
+    FirstView_Event = wxID_HIGHEST + 1000,
+    LastView_Event = wxID_HIGHEST + 1100,
+
+    FirstSD_Event = wxID_HIGHEST + 2000,
+    LastSD_Event = wxID_HIGHEST + 2100,
 };
 
+DEFINE_LOCAL_EVENT_TYPE( wxEVT_PLAYLIST );
+
 BEGIN_EVENT_TABLE(Playlist, wxFrame)
+    EVT_SIZE(Playlist::OnSize)
+
     /* Menu events */
     EVT_MENU(AddFile_Event, Playlist::OnAddFile)
     EVT_MENU(AddMRL_Event, Playlist::OnAddMRL)
@@ -103,8 +132,6 @@ BEGIN_EVENT_TABLE(Playlist, wxFrame)
     EVT_MENU(RSortTitle_Event, Playlist::OnSort)
     EVT_MENU(SortAuthor_Event, Playlist::OnSort)
     EVT_MENU(RSortAuthor_Event, Playlist::OnSort)
-    EVT_MENU(SortGroup_Event, Playlist::OnSort)
-    EVT_MENU(RSortGroup_Event, Playlist::OnSort)
 
     EVT_MENU(Randomize_Event, Playlist::OnSort)
 
@@ -114,25 +141,25 @@ BEGIN_EVENT_TABLE(Playlist, wxFrame)
     EVT_MENU(DeleteSelection_Event, Playlist::OnDeleteSelection)
     EVT_MENU(SelectAll_Event, Playlist::OnSelectAll)
     EVT_MENU(Infos_Event, Playlist::OnInfos)
-    EVT_CHECKBOX(Random_Event, Playlist::OnRandom)
-    EVT_CHECKBOX(Repeat_Event, Playlist::OnRepeat)
-    EVT_CHECKBOX(Loop_Event, Playlist::OnLoop)
 
-    EVT_MENU(EnableGroup_Event, Playlist::OnEnDis)
-    EVT_MENU(DisableGroup_Event, Playlist::OnEnDis)
+    EVT_MENU_OPEN( Playlist::OnMenuOpen )
+    EVT_MENU( -1, Playlist::OnMenuEvent )
 
-    /* Listview events */
-    EVT_LIST_ITEM_ACTIVATED(ListView_Event, Playlist::OnActivateItem)
-    EVT_LIST_COL_CLICK(ListView_Event, Playlist::OnColSelect)
-    EVT_LIST_KEY_DOWN(ListView_Event, Playlist::OnKeyDown)
-    EVT_LIST_ITEM_RIGHT_CLICK(ListView_Event, Playlist::OnPopup)
+    EVT_TOOL(Random_Event, Playlist::OnRandom)
+    EVT_TOOL(Repeat_Event, Playlist::OnRepeat)
+    EVT_TOOL(Loop_Event, Playlist::OnLoop)
 
     /* Popup events */
     EVT_MENU( PopupPlay_Event, Playlist::OnPopupPlay)
+    EVT_MENU( PopupPlayThis_Event, Playlist::OnPopupPlay)
     EVT_MENU( PopupDel_Event, Playlist::OnPopupDel)
     EVT_MENU( PopupEna_Event, Playlist::OnPopupEna)
     EVT_MENU( PopupInfo_Event, Playlist::OnPopupInfo)
 
+    /* Tree control events */
+    EVT_TREE_ITEM_ACTIVATED( TreeCtrl_Event, Playlist::OnActivateItem )
+
+    EVT_CONTEXT_MENU( Playlist::OnPopup )
 
     /* Button events */
     EVT_BUTTON( Search_Event, Playlist::OnSearch)
@@ -144,53 +171,67 @@ BEGIN_EVENT_TABLE(Playlist, wxFrame)
 
     EVT_TEXT(SearchText_Event, Playlist::OnSearchTextChange)
 
+    /* Custom events */
+    EVT_COMMAND(-1, wxEVT_PLAYLIST, Playlist::OnPlaylistEvent)
+
     /* Special events : we don't want to destroy the window when the user
      * clicks on (X) */
     EVT_CLOSE(Playlist::OnClose)
 END_EVENT_TABLE()
 
-
-/* Event Table for the Newgroup class */
-BEGIN_EVENT_TABLE(NewGroup, wxDialog)
-    EVT_BUTTON( wxID_OK, NewGroup::OnOk)
-    EVT_BUTTON( wxID_CANCEL, NewGroup::OnCancel)
-END_EVENT_TABLE()
-
+/*****************************************************************************
+ * PlaylistItem class
+ ****************************************************************************/
+class PlaylistItem : public wxTreeItemData
+{
+public:
+    PlaylistItem( playlist_item_t *_p_item ) : wxTreeItemData()
+    {
+        p_item = _p_item;
+    }
+protected:
+    playlist_item_t *p_item;
+friend class Playlist;
+};
 
 /*****************************************************************************
  * Constructor.
  *****************************************************************************/
 Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     wxFrame( p_parent, -1, wxU(_("Playlist")), wxDefaultPosition,
-             wxDefaultSize, wxDEFAULT_FRAME_STYLE )
+             wxSize(345,400), wxDEFAULT_FRAME_STYLE )
 {
+    vlc_value_t val;
+
     /* Initializations */
-    iteminfo_dialog = NULL;
     p_intf = _p_intf;
-    vlc_value_t  val;
     i_update_counter = 0;
     i_sort_mode = MODE_NONE;
     b_need_update = VLC_FALSE;
-    vlc_mutex_init( p_intf, &lock );
     SetIcon( *p_intf->p_sys->p_icon );
 
+    p_view_menu = NULL;
+    p_sd_menu = SDMenu();
+
+    i_current_view = VIEW_SIMPLE;
+
     i_title_sorted = 0;
     i_author_sorted = 0;
     i_group_sorted = 0;
+    i_duration_sorted = 0;
 
-
-    var_Create( p_intf, "random", VLC_VAR_BOOL );
-    var_Change( p_intf, "random", VLC_VAR_INHERITVALUE, & val, NULL );
-    var_Create( p_intf, "loop", VLC_VAR_BOOL );
-    var_Create( p_intf, "loop", VLC_VAR_BOOL );
-    var_Change( p_intf, "repeat", VLC_VAR_INHERITVALUE, & val, NULL );
-    var_Change( p_intf, "repeat", VLC_VAR_INHERITVALUE, & val, NULL );
+    var_Create( p_intf, "random", VLC_VAR_BOOL | VLC_VAR_DOINHERIT );
+    var_Create( p_intf, "loop", VLC_VAR_BOOL | VLC_VAR_DOINHERIT );
+    var_Create( p_intf, "repeat", VLC_VAR_BOOL | VLC_VAR_DOINHERIT );;
 
     /* Create our "Manage" menu */
     wxMenu *manage_menu = new wxMenu;
     manage_menu->Append( AddFile_Event, wxU(_("&Simple Add...")) );
     manage_menu->Append( AddMRL_Event, wxU(_("&Add MRL...")) );
     manage_menu->AppendSeparator();
+    manage_menu->Append( MenuDummy_Event, wxU(_("Services discovery")),
+                         p_sd_menu );
+    manage_menu->AppendSeparator();
     manage_menu->Append( Open_Event, wxU(_("&Open Playlist...")) );
     manage_menu->Append( Save_Event, wxU(_("&Save Playlist...")) );
     manage_menu->AppendSeparator();
@@ -202,12 +243,9 @@ Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     sort_menu->Append( RSortTitle_Event, wxU(_("&Reverse sort by title")) );
     sort_menu->AppendSeparator();
     sort_menu->Append( SortAuthor_Event, wxU(_("Sort by &author")) );
-    sort_menu->Append( RSortAuthor_Event, wxU(_("&Reverse sort by author")) );
+    sort_menu->Append( RSortAuthor_Event, wxU(_("Reverse sort by author")) );
     sort_menu->AppendSeparator();
-    sort_menu->Append( SortGroup_Event, wxU(_("Sort by &group")) );
-    sort_menu->Append( RSortGroup_Event, wxU(_("&Reverse sort by group")) );
-    sort_menu->AppendSeparator();
-    sort_menu->Append( Randomize_Event, wxU(_("&Randomize Playlist")) );
+    sort_menu->Append( Randomize_Event, wxU(_("&Shuffle Playlist")) );
 
     /* Create our "Selection" menu */
     wxMenu *selection_menu = new wxMenu;
@@ -215,144 +253,135 @@ Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     selection_menu->Append( DisableSelection_Event, wxU(_("&Disable")) );
     selection_menu->AppendSeparator();
     selection_menu->Append( InvertSelection_Event, wxU(_("&Invert")) );
-    selection_menu->Append( DeleteSelection_Event, wxU(_("&Delete")) );
+    selection_menu->Append( DeleteSelection_Event, wxU(_("D&elete")) );
     selection_menu->Append( SelectAll_Event, wxU(_("&Select All")) );
 
-    /* Create our "Group" menu */
-    wxMenu *group_menu = new wxMenu;
-    group_menu->Append( EnableGroup_Event, wxU(_("&Enable all group items")) );
-    group_menu->Append( DisableGroup_Event,
-                        wxU(_("&Disable all group items")) );
+    /* Create our "View" menu */
+    ViewMenu();
 
     /* Append the freshly created menus to the menu bar */
     wxMenuBar *menubar = new wxMenuBar( wxMB_DOCKABLE );
     menubar->Append( manage_menu, wxU(_("&Manage")) );
     menubar->Append( sort_menu, wxU(_("S&ort")) );
     menubar->Append( selection_menu, wxU(_("&Selection")) );
-    menubar->Append( group_menu, wxU(_("&Groups")) );
+    menubar->Append( p_view_menu, wxU(_("&View items") ) );
 
     /* Attach the menu bar to the frame */
     SetMenuBar( menubar );
 
     /* Create the popup menu */
     popup_menu = new wxMenu;
-    popup_menu->Append( PopupPlay_Event, wxU(_("Play item")) );
-    popup_menu->Append( PopupDel_Event, wxU(_("Delete item")) );
-    popup_menu->Append( PopupEna_Event, wxU(_("Toggle enabled")) );
-    popup_menu->Append( PopupInfo_Event, wxU(_("Info on item")) );
+    popup_menu->Append( PopupPlay_Event, wxU(_("Play")) );
+    popup_menu->Append( PopupPlayThis_Event, wxU(_("Play this branch")) );
+    popup_menu->Append( PopupDel_Event, wxU(_("Delete")) );
+    popup_menu->Append( PopupEna_Event, wxU(_("Enable/Disable")) );
+    popup_menu->Append( PopupInfo_Event, wxU(_("Info")) );
 
     /* Create a panel to put everything in */
     wxPanel *playlist_panel = new wxPanel( this, -1 );
     playlist_panel->SetAutoLayout( TRUE );
 
-    /* Create the Random checkbox */
-    wxCheckBox *random_checkbox =
-        new wxCheckBox( playlist_panel, Random_Event, wxU(_("Random")) );
+    /* Create the toolbar */
+    wxToolBar *toolbar =
+        CreateToolBar( wxTB_HORIZONTAL | wxTB_FLAT | wxTB_DOCKABLE );
+
+    /* Create the random tool */
+    toolbar->AddTool( Random_Event, wxT(""), wxBitmap(shuffle_on_xpm),
+                       wxBitmap(shuffle_on_xpm), wxITEM_CHECK,
+                       wxU(_(HELP_SHUFFLE) ) );
     var_Get( p_intf, "random", &val );
-    vlc_bool_t b_random = val.b_bool;
-    random_checkbox->SetValue( b_random == VLC_FALSE ? 0 : 1 );
+    toolbar->ToggleTool( Random_Event, val.b_bool );
 
-    /* Create the Loop Checkbox */
-    wxCheckBox *loop_checkbox =
-        new wxCheckBox( playlist_panel, Loop_Event, wxU(_("Loop")) );
+    /* Create the Loop tool */
+    toolbar->AddTool( Loop_Event, wxT(""), wxBitmap( loop_xpm),
+                      wxBitmap( loop_xpm), wxITEM_CHECK,
+                      wxU(_(HELP_LOOP )  ) );
     var_Get( p_intf, "loop", &val );
-    int b_loop = val.b_bool ;
-    loop_checkbox->SetValue( b_loop );
+    toolbar->ToggleTool( Loop_Event, val.b_bool );
 
     /* Create the Repeat one checkbox */
-    wxCheckBox *repeat_checkbox =
-        new wxCheckBox( playlist_panel, Repeat_Event, wxU(_("Repeat one")) );
+    toolbar->AddTool( Repeat_Event, wxT(""), wxBitmap( repeat_xpm),
+                      wxBitmap( repeat_xpm), wxITEM_CHECK,
+                      wxU(_(HELP_REPEAT )  ) );
     var_Get( p_intf, "repeat", &val );
-    int b_repeat = val.b_bool ;
-    repeat_checkbox->SetValue( b_repeat );
+    toolbar->ToggleTool( Repeat_Event, val.b_bool ) ;
 
     /* Create the Search Textbox */
-    search_text =
-        new wxTextCtrl( playlist_panel, SearchText_Event, wxT(""),
-                        wxDefaultPosition, wxSize(140, -1),
-                        wxTE_PROCESS_ENTER);
+    search_text = new wxTextCtrl( toolbar, SearchText_Event, wxT(""),
+                                  wxDefaultPosition, wxSize(100, -1),
+                                  wxTE_PROCESS_ENTER);
 
     /* Create the search button */
-    search_button =
-        new wxButton( playlist_panel, Search_Event, wxU(_("Search")) );
-
-
-    /* Create the listview */
-    /* FIXME: the given size is arbitrary, and prevents us from resizing
-     * the window to smaller dimensions. But the sizers don't seem to adjust
-     * themselves to the size of a listview, and with a wxDefaultSize the
-     * playlist window is ridiculously small */
-    listview = new wxListView( playlist_panel, ListView_Event,
-                               wxDefaultPosition, wxSize( 500, 300 ),
-                               wxLC_REPORT | wxSUNKEN_BORDER );
-    listview->InsertColumn( 0, wxU(_("Name")) );
-    listview->InsertColumn( 1, wxU(_("Author")) );
-    listview->InsertColumn( 2, wxU(_("Group")) );
-    listview->InsertColumn( 3, wxU(_("Duration")) );
-    listview->SetColumnWidth( 0, 270 );
-    listview->SetColumnWidth( 1, 150 );
-    listview->SetColumnWidth( 2, 80 );
+    search_button = new wxButton( toolbar , Search_Event, wxU(_("Search")) );
+
+    toolbar->AddControl( new wxControl( toolbar, -1, wxDefaultPosition,
+                         wxSize(16, 16), wxBORDER_NONE ) );
+    toolbar->AddControl( search_text );
+    toolbar->AddControl( new wxControl( toolbar, -1, wxDefaultPosition,
+                         wxSize(5, 5), wxBORDER_NONE ) );
+    toolbar->AddControl( search_button );
+    search_button->SetDefault();
+    toolbar->Realize();
+
+    /* Create the tree */
+    treectrl = new wxTreeCtrl( playlist_panel, TreeCtrl_Event,
+                               wxDefaultPosition, wxDefaultSize,
+                               wxTR_HIDE_ROOT | wxTR_LINES_AT_ROOT|
+                               wxTR_NO_LINES |
+                               wxTR_HAS_BUTTONS | wxTR_TWIST_BUTTONS |
+                               wxTR_MULTIPLE | wxTR_EXTENDED );
+
+    /* Create image list */
+
+    wxImageList *p_images = new wxImageList( 16 , 16, TRUE);
+
+    wxIcon icons[10];
+    icons[ ITEM_TYPE_UNKNOWN ] = wxIcon( type_unknown_xpm );
+    icons[ ITEM_TYPE_DISC ] = wxIcon( type_disc_xpm );
+    icons[ ITEM_TYPE_DIRECTORY ] = wxIcon( type_directory_xpm );
+    icons[ ITEM_TYPE_PLAYLIST ] = wxIcon( type_playlist_xpm );
+    icons[ ITEM_TYPE_NET ] = wxIcon( type_net_xpm );
+    icons[ ITEM_TYPE_CARD ] = wxIcon( type_card_xpm );
+
+    for( unsigned int i = 0; i< WXSIZEOF( icons ) ; i++ )
+    {
+         p_images->Add( wxIcon( icons[i] ));
+    }
+
+    treectrl->AssignImageList( p_images );
+
+    treectrl->AddRoot( wxU(_("root" )), -1, -1, NULL );
+
+    /* Reduce font size */
+    wxFont font= treectrl->GetFont();
+    font.SetPointSize(8);
+    treectrl->SetFont( font );
 
     /* Create the Up-Down buttons */
+#if 0
     wxButton *up_button =
         new wxButton( playlist_panel, Up_Event, wxU(_("Up") ) );
-
     wxButton *down_button =
         new wxButton( playlist_panel, Down_Event, wxU(_("Down") ) );
 
-    /* Create the iteminfo button */
-    wxButton *iteminfo_button =
-        new wxButton( playlist_panel, Infos_Event, wxU(_("Item info") ) );
-
-    /* Place everything in sizers */
-    wxBoxSizer *button_sizer = new wxBoxSizer( wxHORIZONTAL );
-    button_sizer->Add( iteminfo_button, 0, wxALIGN_CENTER|wxLEFT , 5);
-    button_sizer->Layout();
-
     wxBoxSizer *updown_sizer = new wxBoxSizer( wxHORIZONTAL );
-    updown_sizer->Add( up_button, 0, wxALIGN_LEFT|wxRIGHT, 3);
-    updown_sizer->Add( down_button, 0, wxALIGN_LEFT|wxLEFT, 3);
     updown_sizer->Layout();
-
-    wxBoxSizer *checkbox_sizer = new wxBoxSizer( wxHORIZONTAL );
-    checkbox_sizer->Add( random_checkbox, 0,
-                         wxEXPAND | wxALIGN_RIGHT | wxALL, 5);
-    checkbox_sizer->Add( loop_checkbox, 0,
-                         wxEXPAND | wxALIGN_RIGHT | wxALL, 5);
-    checkbox_sizer->Add( repeat_checkbox, 0,
-                         wxEXPAND | wxALIGN_RIGHT | wxALL, 5);
-    checkbox_sizer->Layout();
-
-    wxBoxSizer *search_sizer = new wxBoxSizer( wxHORIZONTAL );
-    search_sizer->Add( search_text, 0, wxRIGHT|wxALIGN_CENTER, 3);
-    search_sizer->Add( search_button, 0, wxLEFT|wxALIGN_CENTER, 3);
-    search_sizer->Layout();
-
     /* The top and bottom sizers */
-    wxBoxSizer *top_sizer = new wxBoxSizer( wxHORIZONTAL );
-    top_sizer->Add( checkbox_sizer, 1, wxLEFT|wxRIGHT|wxALIGN_LEFT, 4 );
-    top_sizer->Add( search_sizer, 1, wxLEFT|wxRIGHT|wxALIGN_RIGHT, 4 );
-    top_sizer->Layout();
-
     wxBoxSizer *bottom_sizer = new wxBoxSizer( wxHORIZONTAL );
-    bottom_sizer->Add( updown_sizer, 0, wxEXPAND |wxRIGHT | wxLEFT | wxALIGN_LEFT, 4);
-    bottom_sizer->Add( button_sizer , 0, wxEXPAND|wxLEFT | wxRIGHT | wxALIGN_RIGHT, 4 );
+    bottom_sizer->Add( up_button, 0, wxALIGN_LEFT | wxRIGHT, 3);
+    bottom_sizer->Add( down_button, 0, wxALIGN_LEFT | wxLEFT, 3);
     bottom_sizer->Layout();
-
-    wxBoxSizer *main_sizer = new wxBoxSizer( wxVERTICAL );
-
+#endif
     wxBoxSizer *panel_sizer = new wxBoxSizer( wxVERTICAL );
-    panel_sizer->Add( top_sizer, 0, wxEXPAND | wxALL, 5 );
-    panel_sizer->Add( listview, 1, wxEXPAND | wxALL, 5 );
-    panel_sizer->Add( bottom_sizer, 0 , wxEXPAND | wxALL, 5);
+    panel_sizer->Add( treectrl, 1, wxEXPAND | wxALL, 5 );
+#if 0
+    panel_sizer->Add( bottom_sizer, 0, wxALL, 5);
+#endif
     panel_sizer->Layout();
 
     playlist_panel->SetSizerAndFit( panel_sizer );
-    main_sizer->Add( playlist_panel, 1, wxGROW, 0 );
-    main_sizer->Layout();
-    SetSizerAndFit( main_sizer );
 
-#if !defined(__WXX11__)
+#if wxUSE_DRAG_AND_DROP
     /* Associate drop targets with the playlist */
     SetDropTarget( new DragAndDrop( p_intf, VLC_TRUE ) );
 #endif
@@ -376,13 +405,23 @@ Playlist::Playlist( intf_thread_t *_p_intf, wxWindow *p_parent ):
     /* One item has been updated */
     var_AddCallback( p_playlist, "item-change", ItemChanged, this );
 
-
     vlc_object_release( p_playlist );
 
     /* Update the playlist */
     Rebuild();
 }
 
+void Playlist::OnSize( wxSizeEvent& event)
+{
+#if 0
+    wxSize size = GetClientSize();
+    if( listview )
+        listview->SetColumnWidth( 0, size.x - listview->GetColumnWidth(1)
+                        - 15 /* margins */ );
+#endif
+    event.Skip();
+}
+
 Playlist::~Playlist()
 {
     playlist_t *p_playlist =
@@ -393,8 +432,8 @@ Playlist::~Playlist()
         return;
     }
 
-    delete iteminfo_dialog;
-
+    var_DelCallback( p_playlist, "item-change", ItemChanged, this );
+    var_DelCallback( p_playlist, "playlist-current", PlaylistNext, this );
     var_DelCallback( p_playlist, "intf-change", PlaylistChanged, this );
     vlc_object_release( p_playlist );
 }
@@ -402,56 +441,217 @@ Playlist::~Playlist()
 /**********************************************************************
  * Update one playlist item
  **********************************************************************/
+void Playlist::UpdateNode( playlist_t *p_playlist, playlist_item_t *p_node,
+                           wxTreeItemId node )
+{
+    long cookie;
+    wxTreeItemId child;
+    for( int i = 0; i< p_node->i_children ; i++ )
+    {
+        if( i == 0 )
+        {
+            child = treectrl->GetFirstChild( node, cookie);
+        }
+        else
+        {
+            child = treectrl->GetNextChild( node, cookie );
+        }
+
+        if( !child.IsOk() )
+        {
+            /* Not enough children */
+            CreateNode( p_playlist, p_node->pp_children[i], node );
+            /* Keep the tree pointer up to date */
+            child = treectrl->GetNextChild( node, cookie );
+        }
+        else
+        {
+        }
+    }
+
+}
+/* Creates the node p_node as last child of parent */
+void Playlist::CreateNode( playlist_t *p_playlist, playlist_item_t *p_node,
+                           wxTreeItemId parent )
+{
+    long cookie;
+    wxTreeItemId node =
+        treectrl->AppendItem( parent, wxL2U( p_node->input.psz_name ),
+                              -1,-1, new PlaylistItem( p_node ) );
+    treectrl->SetItemImage( node, p_node->input.i_type );
+
+    for( int i = 0; i< p_node->i_children ; i++ )
+    {
+        /* Append the item */
+        if( p_node->pp_children[i]->i_children == -1 )
+        {
+            wxTreeItemId item =
+                treectrl->AppendItem( node,
+                    wxL2U( p_node->pp_children[i]->input.psz_name ), -1,-1,
+                           new PlaylistItem( p_node->pp_children[i]) );
+
+            treectrl->SetItemImage( item,
+                                    p_node->pp_children[i]->input.i_type );
+        }
+        else
+        {
+            CreateNode( p_playlist, p_node->pp_children[i],
+                        node );
+        }
+    }
+}
+
+wxTreeItemId Playlist::FindItem( wxTreeItemId root, playlist_item_t *p_item )
+{
+    long cookie;
+    PlaylistItem *p_wxcurrent;
+    wxTreeItemId search;
+    wxTreeItemId item = treectrl->GetFirstChild( root, cookie );
+    wxTreeItemId child;
+
+    while( item.IsOk() )
+    {
+        p_wxcurrent = (PlaylistItem *)treectrl->GetItemData( item );
+        if( p_wxcurrent->p_item == p_item )
+        {
+            return item;
+        }
+        if( treectrl->ItemHasChildren( item ) )
+        {
+            wxTreeItemId search = FindItem( item, p_item );
+            if( search.IsOk() )
+            {
+                return search;
+            }
+        }
+        item = treectrl->GetNextChild( root, cookie);
+    }
+    /* Not found */
+    wxTreeItemId dummy;
+    return dummy;
+}
+
+/*wxTreeItemId Playlist::FindItemByName( wxTreeItemId root, wxString search_string, wxTreeItemId current )
+{
+    long cookie;
+    PlaylistItem *p_wxcurrent;
+    wxTreeItemId search;
+    wxTreeItemId item = treectrl->GetFirstChild( root, cookie );
+    wxTreeItemId child;
+
+    while( item.IsOk() )
+    {
+        if( treectrl->GetItemText( item).Lower().Contains(
+                                                 search_string.Lower() ) )
+        {
+            return item;
+        if( treectrl->ItemHasChildren( item ) )
+        {
+            wxTreeItemId search = FindItem( item, p_item );
+            if( search.IsOk() )
+            {
+                return search;
+            }
+        }
+        item = treectrl->GetNextChild( root, cookie);
+    }
+  */  /* Not found */
+    /*wxTreeItemId dummy;
+    return dummy;
+}
+*/
+
+
+
+void Playlist::SetCurrentItem( wxTreeItemId item )
+{
+    if( item.IsOk() )
+    {
+        treectrl->SetItemBold( item, true );
+        treectrl->EnsureVisible( item );
+    }
+}
+
 void Playlist::UpdateItem( int i )
 {
+    if( i < 0 ) return; /* Sanity check */
+    playlist_item_t *p_item;
+
     playlist_t *p_playlist =
         (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
                                        FIND_ANYWHERE );
+
     if( p_playlist == NULL )
     {
         return;
     }
-    if( i < 0 || i >= p_playlist->i_size || !p_playlist->pp_items[i] )
-    {
-        vlc_object_release(p_playlist);
+
+    p_item = playlist_ItemGetById( p_playlist, i );
+
+    wxTreeItemId item = FindItem( treectrl->GetRootItem(), p_item);
+
+    UpdateTreeItem( p_playlist, item );
+
+    vlc_object_release(p_playlist);
+
+}
+
+void Playlist::UpdateTreeItem( playlist_t *p_playlist ,wxTreeItemId item )
+{
+    playlist_item_t *p_item;
+
+   if( !item.IsOk() )
+   {
         return;
-    }
-    listview->SetItem( i, 0, wxL2U(p_playlist->pp_items[i]->psz_name) );
-    listview->SetItem( i, 1, wxU( playlist_GetInfo( p_playlist, i,
-                                       _("General") , _("Author") ) ) );
-    char *psz_group = playlist_FindGroup(p_playlist,
-                                         p_playlist->pp_items[i]->i_group);
-    listview->SetItem( i, 2,
-             wxL2U( psz_group ? psz_group : _("Normal") ) );
+   }
 
-    if( p_playlist->pp_items[i]->b_enabled == VLC_FALSE )
+    p_item  =  ((PlaylistItem *)treectrl->GetItemData( item ))->p_item;
+
+    if( !p_item )
     {
-        wxListItem listitem;
-        listitem.m_itemId = i;
-        listitem.SetTextColour( *wxLIGHT_GREY);
-        listview->SetItem(listitem);
+        return;
     }
 
+    wxString msg;
+    char *psz_author = playlist_ItemGetInfo( p_item, _("Meta-information"),
+                                                         _("Artist"));
     char psz_duration[MSTRTIME_MAX_SIZE];
-    mtime_t dur = p_playlist->pp_items[i]->i_duration;
-    if( dur != -1 ) secstotimestr( psz_duration, dur/1000000 );
-    else memcpy( psz_duration , "-:--:--", sizeof("-:--:--") );
-    listview->SetItem( i, 3, wxU(psz_duration) );
+    mtime_t dur = p_item->input.i_duration;
+    if( dur != -1 )
+        secstotimestr( psz_duration, dur/1000000 );
+    else
+        memcpy( psz_duration, "-:--:--", sizeof("-:--:--") );
 
-    /* Change the colour for the currenty played stream */
-    wxListItem listitem;
-    listitem.m_itemId = i;
-    if( i == p_playlist->i_index )
+    if( !strcmp( psz_author, "" ) )
     {
-        listitem.SetTextColour( *wxRED );
+        msg.Printf( wxString( wxL2U( p_item->input.psz_name ) ) + wxU( " ( ") +
+                    wxString(wxL2U(psz_duration ) ) + wxU( ")") );
     }
     else
     {
-        listitem.SetTextColour( *wxBLACK );
+        msg.Printf( wxString(wxU( psz_author )) + wxT(" - ") +
+                    wxString(wxL2U(p_item->input.psz_name)) + wxU( " ( ") +
+                    wxString(wxL2U(psz_duration ) ) + wxU( ")") );
     }
-    listview->SetItem( listitem );
+    treectrl->SetItemText( item , msg );
 
-    vlc_object_release(p_playlist);
+    if( p_playlist->status.p_item == p_item )
+    {
+        SetCurrentItem( item );
+    }
+    else
+    {
+        treectrl->SetItemBold( item, false );
+    }
+#if 0
+    if( p_item->b_enabled == VLC_FALSE )
+    {
+        wxListItem listitem;
+        listitem.m_itemId = i;
+        listitem.SetTextColour( *wxLIGHT_GREY);
+        listview->SetItem(listitem);
+    }
+#endif
 }
 
 /**********************************************************************
@@ -459,6 +659,7 @@ void Playlist::UpdateItem( int i )
  **********************************************************************/
 void Playlist::Rebuild()
 {
+    playlist_view_t *p_view;
     playlist_t *p_playlist =
         (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
                                        FIND_ANYWHERE );
@@ -467,31 +668,38 @@ void Playlist::Rebuild()
         return;
     }
 
-    int i_focused = listview->GetFocusedItem();
-
-    /* Clear the list... */
-    listview->DeleteAllItems();
-
     /* ...and rebuild it */
     vlc_mutex_lock( &p_playlist->object_lock );
-    for( int i = 0; i < p_playlist->i_size; i++ )
+
+    p_view = playlist_ViewFind( p_playlist, i_current_view ); /* FIXME */
+
+    /* HACK we should really get new*/
+    msg_Dbg( p_intf, "rebuilding tree" );
+    treectrl->DeleteAllItems();
+    treectrl->AddRoot( wxU(_("root" )), -1, -1,
+                         new PlaylistItem( p_view->p_root) );
+
+    wxTreeItemId root = treectrl->GetRootItem();
+    UpdateNode( p_playlist, p_view->p_root, root );
+
+    wxTreeItemId item;
+    if( p_playlist->status.p_item != NULL )
     {
-        wxString filename = wxL2U(p_playlist->pp_items[i]->psz_name);
-        listview->InsertItem( i, filename );
-        UpdateItem( i );
+        item = FindItem( root, p_playlist->status.p_item );
     }
-    vlc_mutex_unlock( &p_playlist->object_lock );
-
-    if( i_focused )
+    else if( p_playlist->status.p_node != NULL )
     {
-        listview->Focus( i_focused );
-        listview->Select( i_focused );
+        item = FindItem( root, p_playlist->status.p_node );
     }
     else
     {
-        listview->Focus( p_playlist->i_index );
+        item = root;
     }
 
+    SetCurrentItem( item );
+
+    vlc_mutex_unlock( &p_playlist->object_lock );
+
     vlc_object_release( p_playlist );
 }
 
@@ -503,22 +711,14 @@ void Playlist::ShowPlaylist( bool show )
 
 void Playlist::UpdatePlaylist()
 {
-    vlc_bool_t b_need_update = VLC_FALSE;
     i_update_counter++;
 
     /* If the playlist isn't show there's no need to update it */
     if( !IsShown() ) return;
 
-    vlc_mutex_lock( &lock );
     if( this->b_need_update )
     {
-        b_need_update =VLC_TRUE;
         this->b_need_update = VLC_FALSE;
-    }
-    vlc_mutex_unlock( &lock );
-
-    if( b_need_update )
-    {
         Rebuild();
     }
 
@@ -532,14 +732,13 @@ void Playlist::UpdatePlaylist()
     {
         return;
     }
-
+#if 0
     /* Update the colour of items */
-
-    vlc_mutex_lock( &p_playlist->object_lock );
-    if( p_intf->p_sys->i_playing != p_playlist->i_index )
+    int i_playlist_index = p_playlist->i_index;
+    if( p_intf->p_sys->i_playing != i_playlist_index )
     {
         wxListItem listitem;
-        listitem.m_itemId = p_playlist->i_index;
+        listitem.m_itemId = i_playlist_index;
         listitem.SetTextColour( *wxRED );
         listview->SetItem( listitem );
 
@@ -549,16 +748,16 @@ void Playlist::UpdatePlaylist()
             listitem.SetTextColour( *wxBLACK );
             listview->SetItem( listitem );
         }
-        p_intf->p_sys->i_playing = p_playlist->i_index;
+        p_intf->p_sys->i_playing = i_playlist_index;
     }
-    vlc_mutex_unlock( &p_playlist->object_lock );
+#endif
     vlc_object_release( p_playlist );
 }
 
 /*****************************************************************************
  * Private methods.
  *****************************************************************************/
-void Playlist::DeleteItem( int item )
+void Playlist::DeleteItem( int item_id )
 {
     playlist_t *p_playlist =
         (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
@@ -568,8 +767,7 @@ void Playlist::DeleteItem( int item )
         return;
     }
 
-    playlist_Delete( p_playlist, item );
-    listview->DeleteItem( item );
+    playlist_Delete( p_playlist, item_id );
 
     vlc_object_release( p_playlist );
 }
@@ -587,7 +785,25 @@ void Playlist::OnSave( wxCommandEvent& WXUNUSED(event) )
         char *psz_module;
     } formats[] = {{ _("M3U file"), "*.m3u", "export-m3u" },
                    { _("PLS file"), "*.pls", "export-pls" }};
+
     wxString filter = wxT("");
+
+    playlist_t * p_playlist =
+                (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
+                                               FIND_ANYWHERE );
+
+    if( ! p_playlist )
+    {
+        return;
+    }
+    if( p_playlist->i_size == 0 )
+    {
+        wxMessageBox( wxU(_("Playlist is empty") ), wxU(_("Can't save")),
+                      wxICON_WARNING | wxOK, this );
+        vlc_object_release( p_playlist );
+        return;
+    }
+
     for( unsigned int i = 0; i < sizeof(formats)/sizeof(formats[0]); i++)
     {
         filter.Append( wxU(formats[i].psz_desc) );
@@ -600,23 +816,15 @@ void Playlist::OnSave( wxCommandEvent& WXUNUSED(event) )
 
     if( dialog.ShowModal() == wxID_OK )
     {
-
-        playlist_t * p_playlist =
-            (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                           FIND_ANYWHERE );
-
-        if( p_playlist )
+        if( dialog.GetPath().mb_str() )
         {
-            if( dialog.GetFilename().mb_str() )
-            {
-                playlist_Export( p_playlist, dialog.GetFilename().mb_str(),
-                                 formats[dialog.GetFilterIndex()].psz_module );
-            }
+            playlist_Export( p_playlist, dialog.GetPath().mb_str(),
+                             formats[dialog.GetFilterIndex()].psz_module );
         }
-        
-        vlc_object_release( p_playlist );
-        
     }
+
+    vlc_object_release( p_playlist );
+
 }
 
 void Playlist::OnOpen( wxCommandEvent& WXUNUSED(event) )
@@ -629,8 +837,8 @@ void Playlist::OnOpen( wxCommandEvent& WXUNUSED(event) )
         return;
     }
 
-    wxFileDialog dialog( this, wxU(_("Open playlist")),
-                         wxT(""), wxT(""), wxT("All playlists|*.pls;*.m3u;*.asx;*.b4s|M3U files|*.m3u"), wxOPEN );
+    wxFileDialog dialog( this, wxU(_("Open playlist")), wxT(""), wxT(""),
+        wxT("All playlists|*.pls;*.m3u;*.asx;*.b4s|M3U files|*.m3u"), wxOPEN );
 
     if( dialog.ShowModal() == wxID_OK )
     {
@@ -644,24 +852,18 @@ void Playlist::OnAddFile( wxCommandEvent& WXUNUSED(event) )
 {
     p_intf->p_sys->pf_show_dialog( p_intf, INTF_DIALOG_FILE_SIMPLE, 0, 0 );
 
-#if 0
-    Rebuild();
-#endif
 }
 
 void Playlist::OnAddMRL( wxCommandEvent& WXUNUSED(event) )
 {
     p_intf->p_sys->pf_show_dialog( p_intf, INTF_DIALOG_FILE, 0, 0 );
 
-#if 0
-    Rebuild();
-#endif
 }
 
 /********************************************************************
  * Move functions
  ********************************************************************/
-void Playlist::OnUp( wxCommandEvent& event)
+void Playlist::OnUp( wxCommandEvent& event )
 {
     playlist_t *p_playlist =
         (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
@@ -670,27 +872,20 @@ void Playlist::OnUp( wxCommandEvent& event)
     {
         return;
     }
-
+#if 0
     /* We use the first selected item, so find it */
-    long i_item = listview->GetNextItem( i_item, wxLIST_NEXT_ALL,
+    long i_item = listview->GetNextItem( -1, wxLIST_NEXT_ALL,
                                          wxLIST_STATE_SELECTED);
     if( i_item > 0 && i_item < p_playlist->i_size )
     {
-        playlist_Move( p_playlist , i_item, i_item - 1);
-        if( i_item > 1 )
-        {
-            listview->Focus( i_item - 1 );
-        }
-        else
-        {
-            listview->Focus(0);
-        }
+        playlist_Move( p_playlist, i_item, i_item - 1 );
+        listview->Focus( i_item - 1 );
     }
+#endif
     vlc_object_release( p_playlist );
-    return;
 }
 
-void Playlist::OnDown( wxCommandEvent& event)
+void Playlist::OnDown( wxCommandEvent& event )
 {
     playlist_t *p_playlist =
         (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
@@ -699,17 +894,17 @@ void Playlist::OnDown( wxCommandEvent& event)
     {
         return;
     }
-
+#if 0
     /* We use the first selected item, so find it */
-    long i_item = listview->GetNextItem( i_item, wxLIST_NEXT_ALL,
+    long i_item = listview->GetNextItem( -1, wxLIST_NEXT_ALL,
                                          wxLIST_STATE_SELECTED );
     if( i_item >= 0 && i_item < p_playlist->i_size - 1 )
     {
-        playlist_Move( p_playlist , i_item, i_item + 2 );
+        playlist_Move( p_playlist, i_item, i_item + 2 );
         listview->Focus( i_item + 1 );
     }
+#endif
     vlc_object_release( p_playlist );
-    return;
 }
 
 /********************************************************************
@@ -727,89 +922,24 @@ void Playlist::OnSort( wxCommandEvent& event )
     switch( event.GetId() )
     {
         case SortTitle_Event:
-           playlist_SortTitle( p_playlist , 0 );
+           playlist_SortTitle( p_playlist, ORDER_NORMAL );
            break;
         case RSortTitle_Event:
-           playlist_SortTitle( p_playlist , 1 );
+           playlist_SortTitle( p_playlist, ORDER_REVERSE );
            break;
         case SortAuthor_Event:
-           playlist_SortAuthor(p_playlist , 0 );
+           playlist_SortAuthor(p_playlist, ORDER_NORMAL );
            break;
         case RSortAuthor_Event:
-           playlist_SortAuthor( p_playlist , 1 );
-           break;
-        case SortGroup_Event:
-           playlist_SortGroup( p_playlist , 0 );
-           break;
-        case RSortGroup_Event:
-           playlist_SortGroup( p_playlist , 1 );
+           playlist_SortAuthor( p_playlist, ORDER_REVERSE );
            break;
         case Randomize_Event:
-           playlist_Sort( p_playlist , SORT_RANDOM, ORDER_NORMAL );
+           playlist_Sort( p_playlist, SORT_RANDOM, ORDER_NORMAL );
            break;
     }
     vlc_object_release( p_playlist );
 
     Rebuild();
-
-    return;
-}
-
-void Playlist::OnColSelect( wxListEvent& event )
-{
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-    if( p_playlist == NULL )
-    {
-        return;
-    }
-    switch( event.GetColumn() )
-    {
-        case 0:
-            if( i_title_sorted != 1 )
-            {
-                playlist_SortTitle( p_playlist, 0 );
-                i_title_sorted = 1;
-            }
-            else
-            {
-                playlist_SortTitle( p_playlist, 1 );
-                i_title_sorted = -1;
-            }
-            break;
-        case 1:
-            if( i_author_sorted != 1 )
-            {
-                playlist_SortAuthor( p_playlist, 0 );
-                i_author_sorted = 1;
-            }
-            else
-            {
-                playlist_SortAuthor( p_playlist, 1 );
-                i_author_sorted = -1;
-            }
-            break;
-        case 2:
-            if( i_group_sorted != 1 )
-            {
-                playlist_SortGroup( p_playlist, 0 );
-                i_group_sorted = 1;
-            }
-            else
-            {
-                playlist_SortGroup( p_playlist, 1 );
-                i_group_sorted = -1;
-            }
-            break;
-        default:
-            break;
-    }
-    vlc_object_release( p_playlist );
-
-    Rebuild();
-
-    return;
 }
 
 /**********************************************************************
@@ -822,24 +952,32 @@ void Playlist::OnSearchTextChange( wxCommandEvent& WXUNUSED(event) )
 
 void Playlist::OnSearch( wxCommandEvent& WXUNUSED(event) )
 {
-    wxString search_string= search_text->GetValue();
+    wxString search_string = search_text->GetValue();
 
+    bool b_ok = false;
     int i_current;
     int i_first = 0 ;
     int i_item = -1;
+}
 
-    for( i_current = 0 ; i_current <= listview->GetItemCount() ; i_current++ )
+#if 0
+    for( i_current = 0; i_current < listview->GetItemCount(); i_current++ )
     {
-        if( listview->GetItemState( i_current, wxLIST_STATE_SELECTED )
-                   == wxLIST_STATE_SELECTED )
+        if( listview->GetItemState( i_current, wxLIST_STATE_SELECTED ) ==
+              wxLIST_STATE_SELECTED )
         {
             i_first = i_current;
             break;
         }
     }
 
-    for ( i_current = i_first + 1; i_current <= listview->GetItemCount() ;
-          i_current++ )
+    if( i_first == listview->GetItemCount() )
+    {
+        i_first = -1;
+    }
+
+    for( i_current = i_first + 1; i_current < listview->GetItemCount();
+         i_current++ )
     {
         wxListItem listitem;
         listitem.SetId( i_current );
@@ -847,9 +985,48 @@ void Playlist::OnSearch( wxCommandEvent& WXUNUSED(event) )
         if( listitem.m_text.Lower().Contains( search_string.Lower() ) )
         {
             i_item = i_current;
+            b_ok = true;
+            break;
+        }
+        listitem.SetColumn( 1 );
+        listview->GetItem( listitem );
+        if( listitem.m_text.Lower().Contains( search_string.Lower() ) )
+        {
+            i_item = i_current;
+            b_ok = true;
             break;
         }
     }
+    if( !b_ok )
+    {
+        for( i_current = -1 ; i_current < i_first - 1;
+             i_current++ )
+        {
+            wxListItem listitem;
+            listitem.SetId( i_current );
+            listview->GetItem( listitem );
+            if( listitem.m_text.Lower().Contains( search_string.Lower() ) )
+            {
+                i_item = i_current;
+                b_ok = true;
+                break;
+            }
+            listitem.SetColumn( 1 );
+            listview->GetItem( listitem );
+            if( listitem.m_text.Lower().Contains( search_string.Lower() ) )
+            {
+                i_item = i_current;
+                b_ok = true;
+                break;
+            }
+        }
+    }
+
+    if( i_item < 0 || i_item >= listview->GetItemCount() )
+    {
+        return;
+    }
+
     for( long item = 0; item < listview->GetItemCount(); item++ )
     {
         listview->Select( item, FALSE );
@@ -860,31 +1037,18 @@ void Playlist::OnSearch( wxCommandEvent& WXUNUSED(event) )
     listitem.m_state = wxLIST_STATE_SELECTED;
     listview->Select( i_item, TRUE );
     listview->Focus( i_item );
-
 }
+#endif
 
 /**********************************************************************
  * Selection functions
  **********************************************************************/
 void Playlist::OnInvertSelection( wxCommandEvent& WXUNUSED(event) )
 {
-    for( long item = 0; item < listview->GetItemCount(); item++ )
-    {
-        listview->Select( item, ! listview->IsSelected( item ) );
-    }
 }
 
 void Playlist::OnDeleteSelection( wxCommandEvent& WXUNUSED(event) )
 {
-    /* Delete from the end to the beginning, to avoid a shift of indices */
-    for( long item = listview->GetItemCount() - 1; item >= 0; item-- )
-    {
-        if( listview->IsSelected( item ) )
-        {
-            DeleteItem( item );
-        }
-    }
-
     Rebuild();
 }
 
@@ -897,15 +1061,17 @@ void Playlist::OnEnableSelection( wxCommandEvent& WXUNUSED(event) )
     {
         return;
     }
-
+#if 0
     for( long item = listview->GetItemCount() - 1; item >= 0; item-- )
     {
         if( listview->IsSelected( item ) )
         {
+            /*XXX*/
             playlist_Enable( p_playlist, item );
             UpdateItem( item );
         }
     }
+#endif
     vlc_object_release( p_playlist);
 }
 
@@ -918,24 +1084,22 @@ void Playlist::OnDisableSelection( wxCommandEvent& WXUNUSED(event) )
     {
         return;
     }
-
+#if 0
     for( long item = listview->GetItemCount() - 1; item >= 0; item-- )
     {
         if( listview->IsSelected( item ) )
         {
+            /*XXX*/
             playlist_Disable( p_playlist, item );
             UpdateItem( item );
         }
     }
+#endif
     vlc_object_release( p_playlist);
 }
 
 void Playlist::OnSelectAll( wxCommandEvent& WXUNUSED(event) )
 {
-    for( long item = 0; item < listview->GetItemCount(); item++ )
-    {
-        listview->Select( item, TRUE );
-    }
 }
 
 /**********************************************************************
@@ -952,10 +1116,11 @@ void Playlist::OnRandom( wxCommandEvent& event )
     {
         return;
     }
-    var_Set( p_playlist , "random", val);
+    var_Set( p_playlist, "random", val);
     vlc_object_release( p_playlist );
 }
-void Playlist::OnLoop ( wxCommandEvent& event )
+
+void Playlist::OnLoop( wxCommandEvent& event )
 {
     vlc_value_t val;
     val.b_bool = event.IsChecked();
@@ -966,11 +1131,11 @@ void Playlist::OnLoop ( wxCommandEvent& event )
     {
         return;
     }
-    var_Set( p_playlist , "loop", val);
+    var_Set( p_playlist, "loop", val);
     vlc_object_release( p_playlist );
 }
 
-void Playlist::OnRepeat ( wxCommandEvent& event )
+void Playlist::OnRepeat( wxCommandEvent& event )
 {
     vlc_value_t val;
     val.b_bool = event.IsChecked();
@@ -981,27 +1146,51 @@ void Playlist::OnRepeat ( wxCommandEvent& event )
     {
         return;
     }
-    var_Set( p_playlist , "repeat", val);
+    var_Set( p_playlist, "repeat", val);
     vlc_object_release( p_playlist );
 }
 
+void Playlist::OnActivateItem( wxTreeEvent& event )
+{
+    playlist_item_t *p_item,*p_node;
+    playlist_t *p_playlist = (playlist_t *)vlc_object_find( p_intf,
+                                    VLC_OBJECT_PLAYLIST,FIND_ANYWHERE );
+    PlaylistItem *p_wxitem = (PlaylistItem *)treectrl->GetItemData(
+                                                        event.GetItem() );
+    wxTreeItemId parent = treectrl->GetItemParent( event.GetItem() );
 
+    PlaylistItem *p_wxparent = (PlaylistItem *)treectrl->GetItemData( parent );
 
-void Playlist::OnActivateItem( wxListEvent& event )
-{
-    playlist_t *p_playlist =
-        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
     if( p_playlist == NULL )
     {
         return;
     }
-    playlist_Goto( p_playlist, event.GetIndex() );
+
+    if( p_wxitem->p_item->i_children == -1 )
+    {
+        p_node = p_wxparent->p_item;
+        p_item = p_wxitem->p_item;
+    }
+    else
+    {
+        p_node = p_wxitem->p_item;
+        if( p_wxitem->p_item->i_children > 0 )
+        {
+            p_item = p_wxitem->p_item->pp_children[0];
+        }
+        else
+        {
+            p_item = NULL;
+        }
+    }
+
+    playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, i_current_view,
+                      p_node, p_item );
 
     vlc_object_release( p_playlist );
 }
 
-void Playlist::OnKeyDown( wxListEvent& event )
+void Playlist::OnKeyDown( wxTreeEvent& event )
 {
     long keycode = event.GetKeyCode();
     /* Delete selected items */
@@ -1013,6 +1202,14 @@ void Playlist::OnKeyDown( wxListEvent& event )
 }
 
 void Playlist::ShowInfos( int i_item )
+{
+}
+
+void Playlist::OnInfos( wxCommandEvent& WXUNUSED(event) )
+{
+}
+
+void Playlist::OnEnDis( wxCommandEvent& event )
 {
     playlist_t *p_playlist =
         (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
@@ -1021,30 +1218,29 @@ void Playlist::ShowInfos( int i_item )
     {
         return;
     }
-    if( iteminfo_dialog == NULL )
-    {
-        if( i_item >= 0 && i_item < p_playlist->i_size )
-        {
-            iteminfo_dialog = new ItemInfoDialog(
-                              p_intf, p_playlist->pp_items[i_item], this );
-            if( iteminfo_dialog->ShowModal()  == wxID_OK )
-                UpdateItem( i_item );
-            delete iteminfo_dialog;
-            iteminfo_dialog = NULL;
-        }
-    }
     vlc_object_release( p_playlist );
 }
 
-void Playlist::OnInfos( wxCommandEvent& WXUNUSED(event) )
+/**********************************************************************
+ * Menu
+ **********************************************************************/
+
+void Playlist::OnMenuOpen( wxMenuEvent& event)
 {
-    /* We use the first selected item, so find it */
-    long i_item = listview->GetNextItem( -1 , wxLIST_NEXT_ALL,
-                                         wxLIST_STATE_SELECTED );
-    ShowInfos( i_item );
+#if defined( __WXMSW__ )
+#   define GetEventObject GetMenu
+#endif
+
+    if( event.GetEventObject() == p_view_menu )
+    {
+        p_view_menu = ViewMenu();
+    }
+#if defined( __WXMSW__ )
+#   undef GetEventObject
+#endif
 }
 
-void Playlist::OnEnDis( wxCommandEvent& event )
+void Playlist::OnMenuEvent( wxCommandEvent& event )
 {
     playlist_t *p_playlist =
         (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
@@ -1054,37 +1250,167 @@ void Playlist::OnEnDis( wxCommandEvent& event )
         return;
     }
 
-    long i_item = listview->GetNextItem( i_item, wxLIST_NEXT_ALL,
-                                         wxLIST_STATE_SELECTED );
+    if( event.GetId() < FirstView_Event )
+    {
+        event.Skip();
+        return;
+    }
+    else if( event.GetId() < LastView_Event )
+    {
+
+        int i_new_view = event.GetId() - FirstView_Event;
 
-    if( i_item >= 0 && i_item < p_playlist->i_size )
+        playlist_view_t *p_view = playlist_ViewFind( p_playlist, i_new_view );
+
+        if( p_view != NULL )
+        {
+            i_current_view = i_new_view;
+            playlist_ViewUpdate( p_playlist, i_new_view );
+            Rebuild();
+            vlc_object_release( p_playlist );
+            return;
+        }
+        else if( i_new_view >= VIEW_FIRST_SORTED &&
+                 i_new_view <= VIEW_LAST_SORTED )
+        {
+            playlist_ViewInsert( p_playlist, i_new_view, "View" );
+            playlist_ViewUpdate( p_playlist, i_new_view );
+
+            i_current_view = i_new_view;
+
+            Rebuild();
+        }
+    }
+    else if( event.GetId() >= FirstSD_Event && event.GetId() < LastSD_Event )
     {
-       switch( event.GetId() )
-       {
-           case EnableGroup_Event:
-               playlist_EnableGroup( p_playlist ,
-                                  p_playlist->pp_items[i_item]->i_group );
-               break;
-           case DisableGroup_Event:
-               playlist_DisableGroup( p_playlist ,
-                                  p_playlist->pp_items[i_item]->i_group );
-               break;
-       }
-       Rebuild();
+        if( !playlist_IsServicesDiscoveryLoaded( p_playlist,
+                                pp_sds[event.GetId() - FirstSD_Event] ) )
+        {
+            playlist_ServicesDiscoveryAdd( p_playlist,
+                            pp_sds[event.GetId() - FirstSD_Event] );
+        }
+        else
+        {
+            playlist_ServicesDiscoveryRemove( p_playlist,
+                            pp_sds[event.GetId() - FirstSD_Event] );
+        }
     }
+    vlc_object_release( p_playlist );
+}
 
+wxMenu * Playlist::ViewMenu()
+{
+    playlist_t *p_playlist =
+        (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
+                                       FIND_ANYWHERE );
+    if( p_playlist == NULL )
+    {
+        return NULL;
+    }
+
+    if( !p_view_menu )
+    {
+        p_view_menu = new wxMenu;
+    }
+    else
+    {
+        wxMenuItemList::Node *node = p_view_menu->GetMenuItems().GetFirst();
+        for( ; node; )
+        {
+            wxMenuItem *item = node->GetData();
+            node = node->GetNext();
+            p_view_menu->Delete( item );
+        }
+    }
+
+    /* FIXME : have a list of "should have" views */
+    p_view_menu->Append( FirstView_Event + VIEW_CATEGORY,
+                           wxU(_("By category") ) );
+    p_view_menu->Append( FirstView_Event + VIEW_SIMPLE,
+                           wxU(_("Manually added") ) );
+    p_view_menu->Append( FirstView_Event + VIEW_ALL,
+                           wxU(_("All items, unsorted") ) );
+    p_view_menu->Append( FirstView_Event + VIEW_S_AUTHOR,
+                           wxU(_("Sorted by author") ) );
+#if 0
+    for( int i = 0; i< p_playlist->i_views; i++ )
+    {
+        p_view_menu->Append( FirstView_Event + p_playlist->pp_views[i]->i_id,
+                             wxU( p_playlist->pp_views[i]->psz_name ) );
+    }
+#endif
+
+    vlc_object_release( p_playlist);
+
+    return p_view_menu;
+}
+
+wxMenu *Playlist::SDMenu()
+{
+
+    playlist_t *p_playlist = (playlist_t *)vlc_object_find( p_intf,
+                              VLC_OBJECT_PLAYLIST, FIND_ANYWHERE );
+    if( !p_playlist )
+    {
+        return NULL;
+    }
+    vlc_value_t val, val_list, text_list;
+    p_sd_menu = new wxMenu;
+
+    vlc_list_t *p_list = vlc_list_find( p_playlist, VLC_OBJECT_MODULE,
+                                        FIND_ANYWHERE );
+
+    int i_number = 0;
+    for( int i_index = 0; i_index < p_list->i_count; i_index++ )
+    {
+        module_t * p_parser = (module_t *)p_list->p_values[i_index].p_object ;
+
+        if( !strcmp( p_parser->psz_capability, "services_discovery" ) )
+        {
+            p_sd_menu->AppendCheckItem( FirstSD_Event + i_number ,
+                       wxU( p_parser->psz_longname ? p_parser->psz_longname :
+                            p_parser->psz_shortname ) );
+
+            if( playlist_IsServicesDiscoveryLoaded( p_playlist,
+                                    p_parser->psz_shortname ) )
+            {
+                p_sd_menu->Check( FirstSD_Event + i_number, TRUE );
+            }
+
+            INSERT_ELEM( (void**)pp_sds, i_number, i_number,
+                         (void*)p_parser->psz_shortname );
+        }
+    }
+    vlc_list_release( p_list );
     vlc_object_release( p_playlist );
+    return p_sd_menu;
 }
 
+
 /*****************************************************************************
  * Popup management functions
  *****************************************************************************/
-void Playlist::OnPopup( wxListEvent& event )
+void Playlist::OnPopup( wxContextMenuEvent& event )
 {
-    i_popup_item = event.GetIndex();
-    Playlist::PopupMenu( popup_menu , ScreenToClient( wxGetMousePosition() ) );
-}
+    wxPoint pt = event.GetPosition();
 
+    i_popup_item = treectrl->HitTest( ScreenToClient( pt ) );
+    if( i_popup_item.IsOk() )
+    {
+        PlaylistItem *p_wxitem = (PlaylistItem *)treectrl->GetItemData(
+                                                            i_popup_item );
+        PlaylistItem *p_wxparent= (PlaylistItem *) treectrl->GetItemData(
+                                   treectrl->GetItemParent( i_popup_item ) );
+        p_popup_item = p_wxitem->p_item;
+        p_popup_parent = p_wxparent->p_item;
+        treectrl->SelectItem( i_popup_item );
+        Playlist::PopupMenu( popup_menu,
+                             ScreenToClient( wxGetMousePosition() ) );
+    }
+    else
+    {
+    }
+}
 
 void Playlist::OnPopupPlay( wxMenuEvent& event )
 {
@@ -1095,16 +1421,50 @@ void Playlist::OnPopupPlay( wxMenuEvent& event )
     {
         return;
     }
-    if( i_popup_item != -1 )
+    if( p_popup_item != NULL )
     {
-        playlist_Goto( p_playlist, i_popup_item );
+        if( p_popup_item->i_children > -1 )
+        {
+            if( event.GetId() == PopupPlay_Event &&
+                p_popup_item->i_children > 0 )
+            {
+                playlist_Control( p_playlist, PLAYLIST_VIEWPLAY,
+                                  i_current_view, p_popup_item,
+                                  p_popup_item->pp_children[0] );
+            }
+            else
+            {
+                playlist_Control( p_playlist, PLAYLIST_VIEWPLAY,
+                                  i_current_view, p_popup_item, NULL );
+            }
+        }
+        else
+        {
+            if( event.GetId() == PopupPlay_Event )
+            {
+                playlist_Control( p_playlist, PLAYLIST_VIEWPLAY,
+                                  i_current_view, p_popup_parent,
+                                  p_popup_item );
+            }
+        }
     }
     vlc_object_release( p_playlist );
 }
 
 void Playlist::OnPopupDel( wxMenuEvent& event )
 {
-    DeleteItem( i_popup_item );
+    PlaylistItem *p_wxitem;
+
+    p_wxitem = (PlaylistItem *)treectrl->GetItemData( i_popup_item );
+
+    if( p_wxitem->p_item->i_children == -1 )
+    {
+        DeleteItem( p_wxitem->p_item->input.i_id );
+    }
+    else
+    {
+        //DeleteNode( p_wxitem->p_item );
+    }
 }
 
 void Playlist::OnPopupEna( wxMenuEvent& event )
@@ -1117,22 +1477,37 @@ void Playlist::OnPopupEna( wxMenuEvent& event )
         return;
     }
 
-    if( p_playlist->pp_items[i_popup_item]->b_enabled )
-        //playlist_IsEnabled( p_playlist, i_popup_item ) )
-    {
-        playlist_Disable( p_playlist, i_popup_item );
-    }
-    else
-    {
-        playlist_Enable( p_playlist, i_popup_item );
-    }
+    p_popup_item->b_enabled = VLC_TRUE - p_popup_item->b_enabled;
+
     vlc_object_release( p_playlist);
     UpdateItem( i_popup_item );
 }
 
 void Playlist::OnPopupInfo( wxMenuEvent& event )
 {
-    ShowInfos( i_popup_item );
+    if( p_popup_item )
+    {
+        iteminfo_dialog = new ItemInfoDialog( p_intf, p_popup_item, this );
+        if( iteminfo_dialog->ShowModal() == wxID_OK )
+        {
+            UpdateItem( i_popup_item );
+        }
+        delete iteminfo_dialog;
+    }
+}
+
+
+/*****************************************************************************
+ * Custom events management
+ *****************************************************************************/
+void Playlist::OnPlaylistEvent( wxCommandEvent& event )
+{
+    switch( event.GetId() )
+    {
+    case UpdateItem_Event:
+        UpdateItem( event.GetInt() );
+        break;
+    }
 }
 
 /*****************************************************************************
@@ -1140,114 +1515,42 @@ void Playlist::OnPopupInfo( wxMenuEvent& event )
  *  We don't rebuild the playlist directly here because we don't want the
  *  caller to block for a too long time.
  *****************************************************************************/
-int PlaylistChanged( vlc_object_t *p_this, const char *psz_variable,
-                     vlc_value_t old_val, vlc_value_t new_val, void *param )
+static int PlaylistChanged( vlc_object_t *p_this, const char *psz_variable,
+                            vlc_value_t oval, vlc_value_t nval, void *param )
 {
     Playlist *p_playlist_dialog = (Playlist *)param;
-    vlc_mutex_lock( &p_playlist_dialog->lock );
     p_playlist_dialog->b_need_update = VLC_TRUE;
-    vlc_mutex_unlock( &p_playlist_dialog->lock );
     return VLC_SUCCESS;
 }
 
 /*****************************************************************************
  * Next: callback triggered by the playlist-current playlist variable
  *****************************************************************************/
-int PlaylistNext( vlc_object_t *p_this, const char *psz_variable,
-                 vlc_value_t old_val, vlc_value_t new_val, void *param )
+static int PlaylistNext( vlc_object_t *p_this, const char *psz_variable,
+                         vlc_value_t oval, vlc_value_t nval, void *param )
 {
     Playlist *p_playlist_dialog = (Playlist *)param;
-    p_playlist_dialog->UpdateItem( old_val.i_int );
-    p_playlist_dialog->UpdateItem( new_val.i_int );
+
+    wxCommandEvent event( wxEVT_PLAYLIST, UpdateItem_Event );
+    event.SetInt( oval.i_int );
+    p_playlist_dialog->AddPendingEvent( event );
+    event.SetInt( nval.i_int );
+    p_playlist_dialog->AddPendingEvent( event );
+
     return 0;
 }
 
-
 /*****************************************************************************
  * ItemChanged: callback triggered by the item-change playlist variable
  *****************************************************************************/
-int ItemChanged( vlc_object_t *p_this, const char *psz_variable,
-                 vlc_value_t old_val, vlc_value_t new_val, void *param )
+static int ItemChanged( vlc_object_t *p_this, const char *psz_variable,
+                        vlc_value_t old_val, vlc_value_t new_val, void *param )
 {
     Playlist *p_playlist_dialog = (Playlist *)param;
-    p_playlist_dialog->UpdateItem( new_val.i_int );
-    return 0;
-}
-
-
-/***************************************************************************
- * NewGroup Class
- ***************************************************************************/
-NewGroup::NewGroup( intf_thread_t *_p_intf, wxWindow *_p_parent ):
-    wxDialog( _p_parent, -1, wxU(_("New Group")), wxDefaultPosition,
-             wxDefaultSize, wxDEFAULT_FRAME_STYLE )
-{
-    /* Initializations */
-    p_intf = _p_intf;
-    psz_name = NULL;
-    SetIcon( *p_intf->p_sys->p_icon );
 
-    /* Create a panel to put everything in*/
-    wxPanel *panel = new wxPanel( this, -1 );
-    panel->SetAutoLayout( TRUE );
+    wxCommandEvent event( wxEVT_PLAYLIST, UpdateItem_Event );
+    event.SetInt( new_val.i_int );
+    p_playlist_dialog->AddPendingEvent( event );
 
-    wxStaticText *group_label =
-            new wxStaticText( panel , -1,
-                wxU(_("Enter a name for the new group")));
-
-    groupname = new wxTextCtrl(panel, -1, wxU(""),wxDefaultPosition,
-                               wxSize(80,27),wxTE_PROCESS_ENTER);
-
-    wxButton *ok_button = new wxButton(panel, wxID_OK, wxU(_("OK")) );
-    ok_button->SetDefault();
-    wxButton *cancel_button = new wxButton( panel, wxID_CANCEL,
-                                            wxU(_("Cancel")) );
-
-    wxBoxSizer *button_sizer = new wxBoxSizer( wxHORIZONTAL );
-
-    button_sizer->Add( ok_button, 0, wxALL, 5 );
-    button_sizer->Add( cancel_button, 0, wxALL, 5 );
-    button_sizer->Layout();
-
-    wxBoxSizer *panel_sizer = new wxBoxSizer( wxVERTICAL );
-    panel_sizer->Add( group_label, 0, wxEXPAND | wxALL, 5 );
-    panel_sizer->Add( groupname, 0, wxEXPAND | wxALL, 5 );
-    panel_sizer->Add( button_sizer, 0, wxEXPAND | wxALL, 5 );
-    panel_sizer->Layout();
-
-    panel->SetSizerAndFit( panel_sizer );
-
-    wxBoxSizer *main_sizer = new wxBoxSizer( wxVERTICAL );
-    main_sizer->Add( panel, 1, wxEXPAND, 0 );
-    main_sizer->Layout();
-    SetSizerAndFit( main_sizer );
-}
-
-NewGroup::~NewGroup()
-{
-}
-
-void NewGroup::OnOk( wxCommandEvent& event )
-{
-    psz_name = strdup( groupname->GetLineText(0).mb_str() );
-
-    playlist_t * p_playlist =
-          (playlist_t *)vlc_object_find( p_intf, VLC_OBJECT_PLAYLIST,
-                                       FIND_ANYWHERE );
-
-    if( p_playlist )
-    {
-        if( !playlist_CreateGroup( p_playlist, psz_name ) )
-        {
-            psz_name = NULL;
-        }
-        vlc_object_release( p_playlist );
-    }
-
-    EndModal( wxID_OK );
-}
-
-void NewGroup::OnCancel( wxCommandEvent& WXUNUSED(event) )
-{
-    EndModal( wxID_CANCEL );
+    return 0;
 }