Dash - previews should be fixed height and width irrespective of the Dash being windowed or maximised (see visual designs for details)

Bug #1045243 reported by John Lea
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ayatana Design
Fix Released
Critical
John Lea
Unity
Fix Released
High
Nick Dedekind
unity (Ubuntu)
Fix Released
High
Unassigned

Bug Description

Previews should be fixed height and width irrespective of the Dash being windowed or maximised

See designs in https://chinstrap.canonical.com/~sabdfl/12_10/desktop_and_netbook/Dash/ for details

e.g.

https://chinstrap.canonical.com/~sabdfl/12_10/desktop_and_netbook/Dash/Files/Preview_file_01.png

Related branches

John Lea (johnlea)
tags: added: dashm udp
Changed in ayatana-design:
assignee: nobody → John Lea (johnlea)
Changed in unity:
milestone: none → backlog
Changed in ayatana-design:
importance: Undecided → Critical
Changed in unity (Ubuntu):
importance: Undecided → High
Changed in unity:
importance: Undecided → High
Changed in ayatana-design:
status: New → Triaged
Changed in unity:
status: New → Triaged
Changed in unity (Ubuntu):
status: New → Triaged
Changed in ayatana-design:
status: Triaged → Fix Committed
Changed in unity:
assignee: nobody → Nick Dedekind (nick-dedekind)
Changed in unity:
status: Triaged → In Progress
Changed in unity:
status: In Progress → Fix Committed
Omer Akram (om26er)
Changed in unity (Ubuntu):
status: Triaged → Fix Committed
Changed in unity:
milestone: backlog → 6.6
Revision history for this message
Winael (vinzjobard) wrote :

Hi,

We cannot access to the design to see more explaination for this decision, but I think it's a bad decision. With fixed high and width you'll not have the same experiment on little screen as netbooks or big screen.

If you use a big screen, the previews will be flown in the dash, that's very chocking, and very not nice to use.

Can you explain us more why you take this decision ?

BR,
Winael

Revision history for this message
Michal Predotka (mpredotka) wrote :

Links in the bug description are not accessible for most of us.

Revision history for this message
Happy R (razz-dulce) wrote :

Hi!
This change (although probably the design team knows better, related to what they intend to implement) is IMO a bad decission.

This will definitely remove most of exciting new functionalities provided by the previews (especially for 3-party developers), that, you can't read anymore large pieces of text, see clear big images, scroll becomes not so usable anymore, etc

Revision history for this message
Mark Shuttleworth (sabdfl) wrote : Re: [Bug 1045243] Re: Dash - previews should be fixed height and width irrespective of the Dash being windowed or maximised (see visual designs for details)

Hmm... I don't think previews should be a fixed height. John, can we
discuss?

Mark

Changed in unity:
milestone: 6.6 → 7.0
Omer Akram (om26er)
no longer affects: unity/6.0
Changed in unity:
milestone: 7.0 → 6.6
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: Fix Committed → Fix Released
Changed in unity:
status: Fix Committed → Fix Released
John Lea (johnlea)
Changed in ayatana-design:
status: Fix Committed → Fix Released
tags: added: reviewedbydesignq
removed: dashm udp
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.