[TOPBLOCKER] [Dash] [design] Preview images display as black thumbnails

This happens for the zoomable image, because it gets scaled down to fit, while the corners are rounded at 1:1 scale...

Not sure how to solve this design-wise, we probably don't want to round the corners "after" scaling, maybe we simply need to always apply the same corner radius...

I'm reliably informed that there's two or three corner radius' available that depend on image size. I recommend checking the SDK documentation or asking Jouni for more details

http://developer.ubuntu.com/api/devel/ubuntu-13.04/qml/ui-toolkit/qml-ubuntu-components0-ubuntushape.html

Let me know if the medium radius property is still too small, and we can look at adding another one that's larger

It's not about setting the radius. The problem is this is a *zoomable* image (well, it can be, isn't by default).

Which means you can take two fingers and use that to zoom the image in and out. What that means (currently) is the shape radius/highlight/shadow is zoomed with it. We need a UX+visual solution for the zoomable use case, please.

The spec https://sites.google.com/a/canonical.com/unity8dash/toolkit/14-previews has been updated with:

    Tapping on image opens it full screen
    Image can then be scrolled or zoomed (via pinch and spread)
    Used for product images and artwork that requires zoom
    Can contain optional credit below video. Left aligned. "Credit: Lorem ipsum dolor" where credit text can be a link to the source

Spec is updated:

Tapping on image opens it full screen
Image can then be scrolled or zoomed (via pinch and spread)
Used for product images and artwork that requires zoom
Can contain optional credit below video. Left aligned. "Credit: Lorem ipsum dolor" where credit text can be a link to the source

Think that solves the issue?

E-mailed Jouni regarding this bug. Can't assign him, not sure if he has an lp account.

Ignore my last comment, I must have not refreshed the page.

Raising this to Critical as fixing this is required in order to fix bug no 1377965 (which is a critical bug).

@saviq - just assigning so it doesn't get lost, since pat seems to request it make the next flight 10/23
also, not totally sure we have everything we need from design ?

This bug was fixed in the package unity8 - 8.01+15.04.20141030~rtm-0ubuntu1

---------------
unity8 (8.01+15.04.20141030~rtm-0ubuntu1) 14.09; urgency=medium

  [ Michael Terry ]
  * Provide 'passphrase' as a field of Components.Lockscreen
  * Fix a race between Qml loading and DBus registration that caused
    problems when jenkins tried to unlock the phone.
  * Set domain explicitly for the Dialogs component because the welcome
    wizard wants to import it. (LP: #1381731)
  * When greeter or lockscreen has focus, show active call panel. (LP:
    #1378872)

  [ Ted Gould ]
  * Set the default OOM score for the dash (LP: #1379786)

  [ Michał Sawicz ]
  * Revert lp:~unity-team/unity8/flickables-speed-workaround to avoid
    risk in RTM.
  * Updated behaviour for zoomable image, workaround for sourcesize (LP:
    #1333187)

  [ Michael Frey ]
  * Added a check for Proximity to determine if we show the lock screen.
    (LP: #1378012)

  [ Ying-Chun Liu ]
  * Add non-interactive code into GenericScopeView. (LP: #1384441)

  [ Mirco Müller ]
  * Also use modal nature of snap-decision notifications when
    greeter/lockscreen is shown. This fixes LP: #1378827. (LP: #1378827)

  [ Michael Zanetti ]
  * Make the launcher update on dconf changes (LP: #1376707)
  * exit spread on background tap (LP: #1368261)
  * Use an index instead of a scope id in DashCommunicator (LP:
    #1376044)

  [ Andrea Cimitan ]
  * Updated behaviour for zoomable image, workaround for sourcesize (LP:
    #1333187)

  [ Daniel d'Andrada ]
  * Make TouchGate synthesize QMouseEvents for mouse-based target items
  * Don't specify a distanceThreshold as it conflicts with
    hintDisplacment
 -- Ubuntu daily release <email address hidden> Thu, 30 Oct 2014 21:43:42 +0000

This bug was fixed in the package unity8 - 8.01+15.04.20141030-0ubuntu1

---------------
unity8 (8.01+15.04.20141030-0ubuntu1) vivid; urgency=medium

  [ Michael Terry ]
  * Provide 'passphrase' as a field of Components.Lockscreen
  * Fix a race between Qml loading and DBus registration that caused
    problems when jenkins tried to unlock the phone.
  * Set domain explicitly for the Dialogs component because the welcome
    wizard wants to import it. (LP: #1381731)
  * When greeter or lockscreen has focus, show active call panel. (LP:
    #1378872)

  [ Ted Gould ]
  * Set the default OOM score for the dash (LP: #1379786)

  [ Michał Sawicz ]
  * Revert lp:~unity-team/unity8/flickables-speed-workaround to avoid
    risk in RTM.
  * Updated behaviour for zoomable image, workaround for sourcesize (LP:
    #1333187)

  [ Michael Frey ]
  * Added a check for Proximity to determine if we show the lock screen.
    (LP: #1378012)

  [ Ying-Chun Liu ]
  * Add non-interactive code into GenericScopeView. (LP: #1384441)

  [ Mirco Müller ]
  * Also use modal nature of snap-decision notifications when
    greeter/lockscreen is shown. This fixes LP: #1378827. (LP: #1378827)

  [ Michael Zanetti ]
  * Make the launcher update on dconf changes (LP: #1376707)
  * exit spread on background tap (LP: #1368261)
  * Use an index instead of a scope id in DashCommunicator (LP:
    #1376044)

  [ Andrea Cimitan ]
  * Updated behaviour for zoomable image, workaround for sourcesize (LP:
    #1333187)

  [ Daniel d'Andrada ]
  * Make TouchGate synthesize QMouseEvents for mouse-based target items
  * Don't specify a distanceThreshold as it conflicts with
    hintDisplacment
 -- Ubuntu daily release <email address hidden> Thu, 30 Oct 2014 21:43:42 +0000