Add context menu to current playing song [$15]
Bug #1128380 reported by
Danielle Foré
This bug affects 4 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Noise |
Fix Released
|
Wishlist
|
Robert Roth | ||
Bug Description
We should add a context menu to the current playing song label in the toolbar. It should be populated with the same menu you would expect from right-clicking a song in the list view.
Related branches
lp:~hauzer-d/noise/now-playing-label-context-menu
- Danielle Foré: Needs Fixing
-
Diff: 309 lines (+63/-28)5 files modifiedsrc/LibraryWindow.vala (+2/-2)
src/Views/ListView/ListView.vala (+1/-2)
src/Views/ListView/Lists/MusicListView.vala (+36/-17)
src/Views/Wrappers/MusicViewWrapper.vala (+5/-5)
src/Widgets/TopDisplay.vala (+19/-2)
lp:~evfool/noise/lp1128380-now-context
- Danielle Foré: Approve (ux)
- elementary Apps team: Pending (code) requested
-
Diff: 340 lines (+133/-114)4 files modifiedsrc/LibraryWindow.vala (+1/-1)
src/Views/ListView/Lists/MusicListView.vala (+104/-105)
src/Views/Wrappers/MusicViewWrapper.vala (+5/-4)
src/Widgets/TopDisplay.vala (+23/-4)
| Changed in noise: | |
| status: | New → Confirmed |
| Changed in noise: | |
| milestone: | none → feature-future |
| Changed in noise: | |
| assignee: | nobody → Victor Martinez (victored) |
| status: | Confirmed → In Progress |
Revision history for this message
| Никола Вукосављевић (hauzer-d) wrote | #1 |
| summary: |
- Add context menu to current playing song + Add context menu to current playing song [$15] |
| Changed in noise: | |
| assignee: | Victor Martinez (victored) → nobody |
| status: | In Progress → Triaged |
| Changed in noise: | |
| assignee: | nobody → Robert Roth (evfool) |
| status: | Triaged → In Progress |
| Changed in noise: | |
| status: | In Progress → Fix Committed |
| Changed in noise: | |
| milestone: | feature-future → freya-rc1 |
| Changed in noise: | |
| milestone: | freya-rc1 → 0.3.1 |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I've sent a patch which adds this feature: http://