Ubuntu
ubuntu-settings-components package

avatar in notification has wrong aspect ratio

Bug #1450229 reported by Bill Filler
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Canonical System Image
Fix Released
High
Kevin Gunn
Canonical System Image ww40-2015 "ota7"
ubuntu-settings-components (Ubuntu)
Fix Released
Undecided
Nick Dedekind
unity8 (Ubuntu)
Fix Released
Undecided
Unassigned

Bug Description

the avatar thumbnail is wrong in the notifciation bubble and doesn't have correct aspect ration.

Steps to reproduce

1) take a picture on the phone and associate with a contact that will text the phone
2) text the phone from the contact that has the avatar

Expected results:
notification bubble will show properly scaled avatar

Actual results:
avatar is not scaled correctly

Related branches

lp://staging/~fboucault/unity8/notifications_icon_respect_ratio
PS Jenkins bot (community): Needs Fixing (continuous-integration)
Mirco Müller (community): Approve
Michael Zanetti (community): Approve
Diff: 24 lines (+4/-3)
1 file modified
qml/Notifications/ShapedIcon.qml (+4/-3)
lp://staging/~nick-dedekind/ubuntu-settings-components/icon-aspect-ratio-crop
Albert Astals Cid: Pending requested
Sebastien Bacher: Pending requested
PS Jenkins bot: Pending (continuous-integration) requested
Diff: 52 lines (+4/-8)
3 files modified
plugins/Ubuntu/Settings/Components/MessageHeader.qml (+0/-1)
plugins/Ubuntu/Settings/Components/UbuntuShapeForItem.qml (+4/-6)
plugins/Ubuntu/Settings/Menus/TransferMenu.qml (+0/-1)
lp://staging/ubuntu/wily-proposed/ubuntu-settings-components
Bill Filler (bfiller)
summary: - avatar in notification has wrong aspect ration
+ avatar in notification has wrong aspect ratio
Revision history for this message
Bill Filler (bfiller) wrote :

the list displayed in indicator-messages is not correct either, the avatar has wrong aspect ratio

Changed in canonical-devices-system-image:
assignee: nobody → Kevin Gunn (kgunn)
importance: Undecided → High
Sebastien Bacher (seb128)
affects: indicator-messages (Ubuntu) → unity8 (Ubuntu)
Michał Sawicz (saviq)
affects: unity-notifications (Ubuntu) → ubuntu-settings-components (Ubuntu)
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in ubuntu-settings-components (Ubuntu):
status: New → Confirmed
Changed in unity8 (Ubuntu):
status: New → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package unity8 - 8.10+15.10.20150616.1-0ubuntu1

---------------
unity8 (8.10+15.10.20150616.1-0ubuntu1) wily; urgency=medium

  [ Albert Astals Cid ]
  * Bind running to the proper object.visible
  * Do not use template in generated card creator code
  * Fix regression introduced by the concierge change
  * Take into account the extra margin on top when calculating the
    display margin
  * workaround oxide 1.7 issue (LP: #1459142)

  [ CI Train Bot ]
  * New rebuild forced.

  [ Federico Gimenez ]
  * On the autopilot helpers, swipe if the item to open from the dash is
    not visible. unity8-autopilot dependencies now require ubuntu-ui-
    toolkit-autopilot (>= 1.2.1485+15.04.20150417.1-0ubuntu1)

  [ Florian Boucault ]
  * Notifications: shaped icon was stretching the icon instead of
    respecting its aspect ratio and filling the whole shape. (LP:
    #1450229)

  [ Josh Arenson ]
  * Implement bare-bones shellMode=greeter and shellMode=shell

  [ Leo Arias ]
  * On the autopilot helpers, swipe if the item to open from the dash is
    not visible. unity8-autopilot dependencies now require ubuntu-ui-
    toolkit-autopilot (>= 1.2.1485+15.04.20150417.1-0ubuntu1)

  [ Leonardo Arias Fonseca ]
  * On the autopilot helpers, swipe if the item to open from the dash is
    not visible. unity8-autopilot dependencies now require ubuntu-ui-
    toolkit-autopilot (>= 1.2.1485+15.04.20150417.1-0ubuntu1)

  [ Michael Zanetti ]
  * add info on how to run a single function in make test

  [ Richard Huddie ]
  * Slow down the speed of autopilot helpers used to swipe to a
    different scope to improve the reliability. (LP: #1461059, #1457047)

 -- CI Train Bot <email address hidden> Tue, 16 Jun 2015 15:12:59 +0000

Changed in unity8 (Ubuntu):
status: Confirmed → Fix Released
Nick Dedekind (nick-dedekind)
Changed in ubuntu-settings-components (Ubuntu):
assignee: nobody → Nick Dedekind (nick-dedekind)
status: Confirmed → In Progress
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-settings-components - 0.6+15.10.20150915-0ubuntu1

---------------
ubuntu-settings-components (0.6+15.10.20150915-0ubuntu1) wily; urgency=medium

  [ CI Train Bot ]
  * New rebuild forced.

  [ Daniel d'Andrada ]
  * Avoid creating a deluge of deprecation warnings

  [ Lukáš Tinkl ]
  * Use the standalone month name for the calendar caption

  [ Nick Dedekind ]
  * Crop icons to aspect ratio inside UbuntuShapes. (LP: #1450229)

 -- Nick Dedekind <email address hidden> Tue, 15 Sep 2015 09:42:35 +0000

Changed in ubuntu-settings-components (Ubuntu):
status: In Progress → Fix Released
Pat McGowan (pat-mcgowan)
Changed in canonical-devices-system-image:
milestone: none → ww40-2015
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.