Icon width is inconsistent when height is specified

[Albert Astals Cid]

The width of an Icon {} depends if the Icon is an SVG coming from image://theme/ or a JPG.

For the SVG the width grows as the units.gu grow, but for the JPEG it does not and one has to manually add code to maitain the aspect ratio.

See the attached code and files.

[Zsombor Egri (zsombi)]

Albert, haven't you provided recently a fix for this?

[Albert Astals Cid (aacid)]

No, I haven't

just run the file with different pixel per grid unit and see how the error is still there.

[Zsombor Egri (zsombi)]

Ahm, Albert, I have worst news for you: design said we have to set the implicit width/height to 2grid units... so you will have to do more calculations if we apply that :(

Perhaps we expose the image size so you can decide whether you want to bind to that or not.

[Launchpad Janitor (janitor)]

This bug was fixed in the package ubuntu-ui-toolkit - 1.3.1984+16.10.20160527.2

---------------
ubuntu-ui-toolkit (1.3.1984+16.10.20160527.2) yakkety; urgency=medium

  [ Christian Dywan ]
  * Slimmer frame for TextFields: 0.5dp. Fixes LP: #1578190.

  [ Albert Astals Cid ]
  * Add override
    The override specifier (since C++11) specifies that a virtual function
    overrides another virtual function. In a member function declaration or
    definition, override ensures that the function is virtual and is overriding
    a virtual function from the base class.

  [ Tim Peeters ]
  * Fix reference error in PullToRefreshStyle. Fixes LP: #1582843
  * Mark Tab, Tabs, TabBar, PageHeadConfiguration, PageHeadSections,
    PageHeadState, ToolbarButton, ToolbarItems as deprecated in the
    documentation. Fixes LP: #1566735, LP: #1566741.

  [ CI Train Bot ]
  * Resync trunk.

 -- Zoltan Balogh <email address hidden> Fri, 27 May 2016 07:08:44 +0000

[Albert Astals Cid (aacid)]

As discussed in IRC a while back, this bug hsan't really been fixed.