Music lens preview does not show play/pause status

Bug #1043997 reported by Francis Ginther
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Unity
Fix Released
Medium
Nick Dedekind
unity (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

[Impact]
The music lens preview only allows for play. It does not show status or allow for pause as indicated by the unity manual-test. See screenshot-music-preview.

[Test Case]
From unity/manual-tests/Preview.txt:
Preview Music Play
note: Should be automated.
------------
This tests the dash music preview track play

Setup:
#. Open music lens (Super+M)
#. Enter arbitrary search string and ensure a result is available.
#. Open a preview for a result which contains tracks.

Actions:
#. Play first track.

Expected Result:
  Music Player daemon will start playing track and update status of the track
  in the preview to playing as well as update it's progress.

[Observed Results]
Music player shows no track status.

[Test Case 2]
Preview Music Pause
note: Should be automated.
------------
This tests the dash preview preview navigate right animation

Setup:
#. Open music lens (Super+M)
#. Enter arbitrary search string and ensure a result is available.
#. Open a preview for a result which contains tracks.
#. Play first track.

Actions:
#. Pause playing track.

Expected Result:
  Music Player daemon will pause the playing track and update it's status
  to paused. Progress will stop incresing.

[Observed Results]
There was no pause control through the preview.

[Packages under test]
From ppa:unity-team/release:
 - bamf - 0.2.122+bzr482ubuntu0+437
 - compiz - 1:0.9.8.0-0ubuntu1~test8
 - nux - 3.2.0+bzr651ubuntu0+361
 - unity - 6.2.0+bzr2633ubuntu0+762

[Hardware config]
Nvidia hardware with nouveau driver.

ProblemType: Bug
DistroRelease: Ubuntu 12.10
Package: unity 6.2.0+bzr2633ubuntu0+762 [origin: LP-PPA-unity-team-release]
ProcVersionSignature: Ubuntu 3.5.0-13.14-generic 3.5.3
Uname: Linux 3.5.0-13-generic x86_64
NonfreeKernelModules: wl
ApportVersion: 2.5.1-0ubuntu3
Architecture: amd64
CompizPlugins: [core,composite,opengl,compiztoolbox,decor,vpswitch,snap,mousepoll,resize,place,move,wall,grid,regex,imgpng,session,gnomecompat,animation,fade,unitymtgrabhandles,workarounds,scale,expo,ezoom,unityshell]
CrashDB: unity
Date: Thu Aug 30 13:25:47 2012
EcryptfsInUse: Yes
InstallationMedia: Ubuntu 12.10 "Quantal Quetzal" - Alpha amd64 (20120627)
ProcEnviron:
 TERM=xterm
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: unity
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Revision history for this message
Francis Ginther (fginther) wrote :
Revision history for this message
Neil J. Patel (njpatel) wrote :

If this was on a new install, you might have been missing the mp3 codec required for playback. We've raised this as an issue to design (that there is nothing in the spec to show errors), but there's not much we can do about it until they've got a resolution.

Revision history for this message
Paweł Stołowski (stolowski) wrote :

Missing mp3 codec may be one explanation, but I got an impression that it's not obvious that:

* one has to mouse-over the track number (!) in the preview for the built-in music-player controls to show up; so: mouse-over "1" at the left of the track title should reveal small play button that will activate music-preview-player daemon. That's the subject of the test.

* 'Play' button at the bottom plays song in Rhythmbox with no progress report (and no way to pause) via the preview;

Francis, is this the case? Perhaps the test should be reworded?

Revision history for this message
Nick Dedekind (nick-dedekind) wrote :

If this is the case, I will update the manual tests with more specific instructions.

"Hover mouse over track number for play icon"
"Click play icon"
"When music is playing, track should show a pause icon in place of track number"

Changed in unity:
assignee: nobody → Nick Dedekind (nick-dedekind)
tags: added: 1210b1blockers
Changed in unity:
importance: Undecided → Medium
Changed in unity (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Francis Ginther (fginther) wrote :

Pawel Stolowski (stolowski) and Nick Dedekind (nick-dedekind),

After having this explained to me in detail, this feature is functioning. I recommend rewording the test case, since it's not obvious there is a second play control revealed by hovering over the track icon.

Revision history for this message
Olli Ries (ories) wrote :

not blocking Beta1, works (supposedly) as designed, design needs an update

Changed in unity:
assignee: Nick Dedekind (nick-dedekind) → John Lea (johnlea)
tags: removed: 1210b1blockers
John Lea (johnlea)
Changed in unity:
assignee: John Lea (johnlea) → nobody
Changed in unity:
assignee: nobody → Nick Dedekind (nick-dedekind)
status: New → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in unity (Ubuntu):
status: New → Confirmed
Omer Akram (om26er)
Changed in unity (Ubuntu):
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (9.2 KiB)

This bug was fixed in the package unity - 6.6.0-0ubuntu1

---------------
unity (6.6.0-0ubuntu1) quantal-proposed; urgency=low

  * New upstream release.
    - Fixes non-escaped character sequences in dash previews (LP: #1039020)
    - Updated background layer for preview cover-art and details panels
      to be 10% low-light
    - Expand a PlacesGroup if it is the only category that contains results
      (LP: #950710)
    - Update unity autopilot tests to match autopilot API
    - Updated the convert files to fix some typos in the key names
    - Add gmodule dependency
    - Activate proper result if the categories aren't displayed in-order
      (LP: #1040101)
    - Refactor device launcher icons (LP: #713423)
    - LauncherController: make the controller enable the launcher struts,
      based on hide-mode option (LP: #1044005)
    - Launcher: make always possible to drag an icon to the bottom or top
      of its sub list (LP: #1043968)
    - Don't desat bfb/hud icon in DNDReset (LP: #1043963)
    - Progressively adjust the speed of the minimize animation. First
      time it is used is slower, then speeds up the more it is used.
      (LP: #1017510)
    - Implement new ordering of categories for home lens. (LP: #1043915)
    - UnityWindow now implements ScaleWindowInterface (LP: #876017)
    - Launcher: restore an icon position after that the dragging has been
      cancelled (LP: #955561)
    - LauncherDragWindow: cancel drag on window mapped/unmapped
      (LP: #1044723)
    - Now there is a check of an override color in RefreshColor (which is
      called when a PropertyNotify event happens). Also added a check in
      FullySaturateColor to a void division by zero. (LP: #975350)
    - Queue redraw after cover-art texture is updated from a url/file source.
      (LP: #1043947)
    - Fixed ability to delete glib::Source wrapper during its callback
      (LP: #1044823)
    - Close preview when dash is hidden. (LP: #1045298)
    - LauncherModel: rewrite the Reordering functions to keep the icon
      priority deltas (LP: #761155)
    - Make sure we can pass extra hints when activating preview actions.
      (LP: #1046352)
    - UnityWindow: scale window code improved (LP: #1033935)
    - The mouse will now cause the HUD buttons to change selection
      (LP: #1042692)
    - "Alt+Space" shortcut to reveal the window menu is not hardcoded, but a
       Compiz key option. " (Hold)" should also be translated. Made all
       Compiz plug-in names and all Compiz plug-in option names in
       unityshell.cpp static constants.
    - remove unity --reset, it's not anymore really needed now that we are
      in stable days of unity and we moved to gsettings
    - Removed the variables 'oldPrev' and 'oldNext' which got assigned the
      value NULL, but then were never used
    - Fixed the size of the previews to 770x380 pixels. (LP: #1045243)
    - UnityWindow: use smart pointers, use static close_icon (with dynamic
      state) and PanelStyle context (LP: #1033935) (LP: #1045127)
      (LP: #1046124) (LP: #1046126)
    - Remove everything in the #ifndef USE_MODERN_COMPIZ_GL ifdefs and remove
      the ifdefs alltogether. unity now requires compiz...

Read more...

Changed in unity (Ubuntu):
status: In Progress → Fix Released
Changed in unity:
status: In Progress → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.