Summary: | "Jump to the currently playing item" button does not update Tag Editor | ||
---|---|---|---|
Product: | [Applications] juk | Reporter: | Thomas McGuire <mcguire> |
Component: | general | Assignee: | Scott Wheeler <wheeler> |
Status: | RESOLVED FIXED | ||
Severity: | normal | ||
Priority: | NOR | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Platform: | Compiled Sources | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: | ||
Sentry Crash Report: |
Description
Thomas McGuire
2005-03-01 17:56:16 UTC
SVN commit 467719 by mpyne: Fix bug 100569 (Jump to currently playing item feature doesn't update tag editor) in KDE 3.5. CCBUG:100569 M +8 -1 playlist.cpp --- branches/KDE/3.5/kdemultimedia/juk/playlist.cpp #467718:467719 @@ -921,9 +921,16 @@ Playlist *l = playingItem()->playlist(); l->clearSelection(); + + // Raise the playlist before selecting the items otherwise the tag editor + // will not update when it gets the selectionChanged() notification + // because it will think the user is choosing a different playlist but not + // selecting a different item. + + m_collection->raise(l); + l->setSelected(playingItem(), true); l->ensureItemVisible(playingItem()); - m_collection->raise(l); } void Playlist::slotColumnResizeModeChanged() SVN commit 467720 by mpyne: Backport fix for bug 100569 (Jump to playing item doesn't update tag editor) to KDE 3.4. BUG:100569 M +8 -1 playlist.cpp --- branches/KDE/3.4/kdemultimedia/juk/playlist.cpp #467719:467720 @@ -883,9 +883,16 @@ Playlist *l = playingItem()->playlist(); l->clearSelection(); + + // Raise the playlist before selecting the items otherwise the tag editor + // will not update when it gets the selectionChanged() notification + // because it will think the user is choosing a different playlist but not + // selecting a different item. + + m_collection->raise(l); + l->setSelected(playingItem(), true); l->ensureItemVisible(playingItem()); - m_collection->raise(l); } void Playlist::slotColumnResizeModeChanged() |