Improve library sidebar UX (right-click and selection after add, rename, delete, duplicate etc.)

res/skins/Deere/style.qss

@@ -229,6 +229,14 @@ WLibrarySidebar::item:selected,
   color: #D6D6D6;
   background-color: #006596;
 }
+WLibrarySidebar::item:selected:focus {
+  outline: none;
+  border: 0px;
+}
+WLibrarySidebar::item:!selected:focus {
+  outline: none;
+  border: 1px solid white;
+}
 /* Use the native focus decoration */
 /* This is for all cells including Played and Location */
 WTrackTableView,
@@ -412,8 +420,7 @@ WSearchLineEdit {
   qproperty-layoutSpacing: 0;
 }
 
-WLibrarySidebar,
-WLibrarySidebar::item:focus {
+WLibrarySidebar {
   outline: none;
 /* Spacing between treeview and preview deck/search bar */
   margin: 0px;

res/skins/LateNight/style_classic.qss

@@ -2071,6 +2071,14 @@ WTrackTableView::item:selected,
   color: #fff;
   background-color: #5e4507;
 }
+WLibrarySidebar::item:selected:focus {
+  outline: none;
+  border: 0px;
+}
+WLibrarySidebar::item:!selected:focus {
+  outline: none;
+  border: 1px solid white;
+}
 
 /* Use the native focus decoration */
 /* This is for all cells including Played and Location */
@@ -2088,9 +2096,6 @@ WTrackTableView {
 
 WLibrarySidebar {
   show-decoration-selected: 0;
-}
-WLibrarySidebar,
-WLibrarySidebar::item:focus {
   outline: none;
 }
 

res/skins/LateNight/style_palemoon.qss

@@ -2539,16 +2539,23 @@ WLibrarySidebar {
   /* background of Color delegate in selected row */
   selection-background-color: #2c454f;
 }
+WLibrarySidebar {
+  outline: none;
+}
 /* Selected rows in Tree and Tracks table */
 WLibrarySidebar::item:selected,
 WTrackTableView::item:selected,
 #LibraryBPMButton::item:selected {
   color: #fff;
   background-color: #2c454f;
 }
-WLibrarySidebar,
-WLibrarySidebar::item:focus {
+WLibrarySidebar::item:selected:focus {
+  outline: none;
+  border: 0px;
+}
+WLibrarySidebar::item:!selected:focus {
   outline: none;
+  border: 1px solid white;
 }
 
 WTrackTableView:focus,

res/skins/Shade/style.qss

@@ -470,9 +470,16 @@ WLibrarySidebar::branch:selected,
   WSearchLineEdit::indicator {
     width:  0;
   }
-WLibrarySidebar,
-WLibrarySidebar::item:focus {
+WLibrarySidebar {
+  outline: none;
+}
+WLibrarySidebar::item:selected:focus {
+  outline: none;
+  border: 0px;
+}
+WLibrarySidebar::item:!selected:focus {
   outline: none;
+  border: 1px solid white;
 }
 
 /* Use the native focus decoration */

res/skins/Tango/style.qss

@@ -2650,6 +2650,14 @@ WLibrarySidebar::item:!selected {
   background-color: #0f0f0f;
   color: #999;
   }
+WLibrarySidebar::item:selected:focus {
+  outline: none;
+  border: 0px;
+}
+WLibrarySidebar::item:!selected:focus {
+  outline: none;
+  border: 1px solid white;
+}
 
 WTrackTableViewHeader {
   /* Don't set a font size to pick up the system font size. */

src/library/banshee/bansheeplaylistmodel.cpp

@@ -66,7 +66,7 @@ const QString kCrate = QStringLiteral(CLM_CRATE);
 BansheePlaylistModel::BansheePlaylistModel(QObject* pParent, TrackCollectionManager* pTrackCollectionManager, BansheeDbConnection* pConnection)
         : BaseSqlTableModel(pParent, pTrackCollectionManager, "mixxx.db.model.banshee_playlist"),
           m_pConnection(pConnection),
-          m_playlistId(-1) {
+          m_playlistId(kInvalidPlaylistId) {
     m_tempTableName = BANSHEE_TABLE + QString::number(sTableNumber.fetchAndAddAcquire(1));
 }
 
@@ -77,7 +77,7 @@ BansheePlaylistModel::~BansheePlaylistModel() {
 void BansheePlaylistModel::dropTempTable() {
     if (m_playlistId >= 0) {
         // Clear old playlist
-        m_playlistId = -1;
+        m_playlistId = kInvalidPlaylistId;
         QSqlQuery query(m_database);
         QString strQuery("DROP TABLE IF EXISTS %1");
         if (!query.exec(strQuery.arg(m_tempTableName))) {

src/library/baseexternallibraryfeature.cpp

@@ -114,7 +114,7 @@ void BaseExternalLibraryFeature::slotImportAsMixxxPlaylist() {
 
     int playlistId = playlistDao.createUniquePlaylist(&playlist);
 
-    if (playlistId != -1) {
+    if (playlistId != kInvalidPlaylistId) {
         playlistDao.appendTracksToPlaylist(trackIds, playlistId);
     } else {
         // Do not change strings here without also changing strings in

src/library/baseexternalplaylistmodel.cpp

@@ -24,7 +24,7 @@ BaseExternalPlaylistModel::BaseExternalPlaylistModel(QObject* parent,
           m_playlistsTable(playlistsTable),
           m_playlistTracksTable(playlistTracksTable),
           m_trackSource(trackSource),
-          m_currentPlaylistId(-1) {
+          m_currentPlaylistId(kInvalidPlaylistId) {
 }
 
 BaseExternalPlaylistModel::~BaseExternalPlaylistModel() {
@@ -100,20 +100,20 @@ void BaseExternalPlaylistModel::setPlaylist(const QString& playlist_path) {
     }
 
     // TODO(XXX): Why not last-insert id?
-    int playlistId = -1;
+    int playlistId = kInvalidPlaylistId;
     QSqlRecord finder_query_record = finder_query.record();
     while (finder_query.next()) {
         playlistId = finder_query.value(finder_query_record.indexOf("id")).toInt();
     }
 
-    if (playlistId == -1) {
+    if (playlistId == kInvalidPlaylistId) {
         qWarning() << "ERROR: Could not get the playlist ID for playlist:" << playlist_path;
         return;
     }
 
     // Store search text
     QString currSearch = currentSearch();
-    if (m_currentPlaylistId != -1 && !currSearch.trimmed().isEmpty()) {
+    if (m_currentPlaylistId != kInvalidPlaylistId && !currSearch.trimmed().isEmpty()) {
         m_searchTexts.insert(m_currentPlaylistId, currSearch);
     }