Horizontal cards with summary don't get backgrounds

[Laszlo Kudela]

If I change the WEATHER_TEMPLATE in the scopes example to:

const static string WEATHER_TEMPLATE =
        R"(
{
        "schema-version": 1,
        "template": {
        "category-layout": "carousel",
        "card-layout": "horizontal",
        "card-size": "small",
 "card-background":"color:///#0F0F0F"
        },
        "components": {
        "title": "title",
        "art" : {
        "field": "art"
        },
        "subtitle": "subtitle",
        "summary": "summary"
        }
        }
        )";

The cards are placed in carousel style, but the background color is simply omitted, the color of the card stays black.

If the card-layout is changed from "horizontal" to "vertical", the color is there, and the cards in a carousel style, but the bottom 10% of the cards seem to be clipped. (Screenshot attached)

This is my first bug report ever, so please let me know what else I should add, or if I am at the right place at all.

Reported from: http://developer.ubuntu.com/scopes/cookbook/

[David Callé (davidc3)]

Thanks a lot for the report, I'm marking unity8 as affected since it looks like a Dash bug.

[Michał Sawicz (saviq)]

Confirmed, something made horizontal card layouts lose background.

[Albert Astals Cid (aacid)]

According to
https://docs.google.com/a/canonical.com/document/d/1n880Fih5KyGPcoP5chidnHDG_8TxXUgSuij7f4rHpuk/edit
horizontal cards without a summary don't get a shape, and since they don't get a shape, don't get background either.

This is my understanding of the table intersection of "Art_Header_horizontal" + "Can card be inside shape?" in "Cards and Ubuntu Shape" that says "No" and "The ubuntu shape is also optional for card templates which support it. In this case, the card must have a background colour"

[Michał Sawicz (saviq)]

OK back to us, this should work when there *is* summary, but doesn't.

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity8 - 8.02+15.04.20150113.1-0ubuntu1

---------------
unity8 (8.02+15.04.20150113.1-0ubuntu1) vivid; urgency=low

  [ Andrea Cimitan ]
  * support background on horizontal cards with summary (LP: #1393008)

  [ Gerry Boland ]
  * DashCommunicator: replace QDBusInterface with a non-blocking
    simplification which does not introspect the service on creation
    (LP: #1403508)

  [ Leo Arias ]
  * Added an autopilot test for the edges demo.

  [ Albert Astals ]
  * Don't show the manage dash pull up arrow on temp scopes (LP:
    #1401869)
  * Remove hack, newer Qt already support keyClick(char)
  * Tests: Add Qt 5.5 removal TODOs
  * Make sure we use fPIC when compiling files for the static library
    too
  * Make sure changing a scope doesn't trigger creation/destruction of
    delegates until it's finished (LP: #1410122)

  [ Michael Zanetti ]
  * patch debian/control file before using it to make it work with mk-
    build-deps
 -- Ubuntu daily release <email address hidden> Tue, 13 Jan 2015 20:58:08 +0000

[Michał Sawicz (saviq)]

This is intertwined with bug #1411748.

[Launchpad Janitor (janitor)]

This bug was fixed in the package unity8 - 8.02+15.04.20150217~rtm-0ubuntu1

---------------
unity8 (8.02+15.04.20150217~rtm-0ubuntu1) 14.09; urgency=medium

  [ Albert astals ]
  * Preserve the aspect ratio for the card attribute images

  [ Andrea Cimitan ]
  * support background on horizontal cards with summary (LP: #1393008)
  * Background needs to be specified to be visible in horizontal cards
  (LP: #1411748)

  [ Daniel d'Andrada ]
  * Don't show() the lockscreen if it's already being shown

  [ Joshua Arenson ]
  * Create a PhysicalKeyMapper to handle all physical/hardware keys

  [ Michał Sawicz ]
  * Create a PhysicalKeyMapper to handle all physical/hardware keys

  [ Mirco Müller ]
  * Allow swipe-to-dismiss for contracted snap-decision notifications,
    interactive notifications and ephemeral notifications. (LP:
    #1355422, #1334855)
 -- CI Train Bot <email address hidden> Tue, 17 Feb 2015 10:47:19 +0000