Multiline attributes break (center flush) on return to visibility

Bug #1381092 reported by Kyle Nitzsche
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Savilerow project
Fix Released
High
Unassigned
unity8 (Ubuntu)
Fix Released
High
Albert Astals Cid
unity8 (Ubuntu RTM)
Fix Released
High
Albert Astals Cid

Bug Description

We have a scope that uses attributes to provide three lines of text.

The result layout is correct until a user clicks a result and navigates to a preview and then returns. (or simply scrolls down such that the result goes out of view and then scrolls back up bringing it into view). After this, the second and third lines are not left flush.

I'll attach screenshots.

JSON renderer:

{
    "schema-version": 1,
    "template": {
        "category-layout": "grid",
        "card-size": "small",
        "card-layout": "horizontal"
    },
    "components": {
        "title": "title",
        "subtitle": "subtitle",
        "mascot": "art",
        "attributes": {
            "field": "attributes",
            "max-count": 6
        }
    }
}

Query code:

    VariantBuilder builder;
    builder.add_tuple({
        {"value", Variant(strong(week_no).toStdString())}
    });
    builder.add_tuple({
        {"value", Variant("")}
    });
    builder.add_tuple({
        {"value", Variant(strong(QString("%1: %2").arg(sunrise).arg(sunrise_time)).toStdString())}
    });
    builder.add_tuple({
        {"value", Variant("")}
    });

    builder.add_tuple({
        {"value", Variant(strong(QString("%1: %2").arg(sunset).arg(sunset_time)).toStdString())}
    });
    builder.add_tuple({
        {"value", Variant("")}
    });
    date_res["attributes"] = builder.end();

Related branches

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

correct layout before going to preview and coming back attached.

Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

incorrect layout after going to preview and coming back

Changed in savilerow:
importance: Undecided → High
Michał Sawicz (saviq)
affects: unity-scopes-api (Ubuntu) → unity8 (Ubuntu)
Changed in unity8 (Ubuntu):
status: New → In Progress
assignee: nobody → Albert Astals Cid (aacid)
importance: Undecided → High
summary: - Multiline attributes break (center flush) on return from preview
+ Multiline attributes break (center flush) on return to visibility
description: updated
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.00+14.10.20141015.4~rtm-0ubuntu1

---------------
unity8 (8.00+14.10.20141015.4~rtm-0ubuntu1) 14.09; urgency=low

  [ Michał Sawicz ]
  * remove qml ownership confusion for caching unitymenumodels (LP:
    #1368856)

  [ josharenson ]
  * Fix lp:1370240 by making stages interactive when a snap decision is
    dismissed. (LP: #1370240)

  [ Nick Dedekind ]
  * remove qml ownership confusion for caching unitymenumodels (LP:
    #1368856)
  * Implementation of expandable panel design (LP: #1368856)

  [ Albert Astals ]
  * CardAttributes: Specify column and row since the gridlayout gets
    confused sometimes (LP: #1381092)
  * Reset VerticalJournal until the cardTool settles (LP: #1381255)

  [ Mirco Müller ]
  * Added synchronous/confirmation notification support to unity8. (LP:
    #1232633)

  [ Michael Zanetti ]
  * drop all visual indication of "pinning" (LP: #1381054)
  * Fix lp:1370240 by making stages interactive when a snap decision is
    dismissed. (LP: #1370240)

  [ Antti Kaijanmäki ]
  * Unlock all modems on boot. (LP: #1333121)
 -- Ubuntu daily release <email address hidden> Wed, 15 Oct 2014 16:44:20 +0000

Changed in unity8 (Ubuntu RTM):
status: New → Fix Released
Michał Sawicz (saviq)
Changed in unity8 (Ubuntu RTM):
assignee: nobody → Albert Astals Cid (aacid)
importance: Undecided → High
Revision history for this message
Kyle Nitzsche (knitzsche) wrote :

marking fix released on savilerow because I confirm the unity8 fix above resolves the issue. thx.

Changed in savilerow:
status: New → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
unity8 (8.00+15.04.20141030-0ubuntu1) vivid; urgency=low

  [ josharenson ]
  * Fix lp:1370240 by making stages interactive when a snap decision is
    dismissed. (LP: #1370240)

  [ Nick Dedekind ]
  * remove qml ownership confusion for caching unitymenumodels (LP:
    #1368856)
  * Implementation of expandable panel design (LP: #1368856)

  [ Albert Astals ]
  * CardAttributes: Specify column and row since the gridlayout gets
    confused sometimes (LP: #1381092)
  * Reset VerticalJournal until the cardTool settles (LP: #1381255)

  [ Mirco Müller ]
  * Added synchronous/confirmation notification support to unity8. (LP:
    #1232633)

  [ Michael Zanetti ]
  * Fix lp:1370240 by making stages interactive when a snap decision is
    dismissed. (LP: #1370240)
  * Drop all visual indication of "pinning" (LP: #1381054)

  [ Antti Kaijanmäki ]
  * Unlock all modems on boot. (LP: #1333121)
 -- Ubuntu daily release <email address hidden> Thu, 30 Oct 2014 10:51:17 +0000

Changed in unity8 (Ubuntu):
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.