** Also affects: gnome-shell-extension-desktop-icons (Ubuntu Focal)
   Importance: Undecided
       Status: New

** Changed in: gnome-shell-extension-desktop-icons (Ubuntu Focal)
       Status: New => Fix Committed

** Description changed:

+ [ Impact ]
+ 
+ Image thumbnails are stretched if the aspect ratio is less than the one
+ of the container
+ 
+ [ Test case ]
+ 
+ To reproduce, create an image that has aspect ratio, e.g., ~0.5 (e.g.
+ width = 250 and height = 500)
+ 
+ Save it in the desktop and ensure that the image has proper proportions.
+ 
+ 
+ [ Regression potential ]
+ 
+ Image thumbnails are distorted
+ 
+ ----
+ 
  This was discovered when investigating fix for bug #1868529.
  
  Aspect ratio is still not correct for images that are more narrow than
  the container. To reproduce, create an image that has aspect ratio,
  e.g., ~0.5 (e.g. width = 250 and height = 500).
  
  Line in fileItem.js where we calculate icon width does not count the
  margins (zero is used instead of actual margins):
  
  let containerWidth = Prefs.getDesiredWidth(scaleFactor, 0);
  
  Here is how the margins are being calculated in desktopGrid.js:
  
-         this._extra_width = themeNode.get_margin(St.Side.LEFT) +
-                              themeNode.get_margin(St.Side.RIGHT) +
-                              themeNode.get_border_width(St.Side.LEFT) +
-                              themeNode.get_border_width(St.Side.RIGHT) +
-                              themeNode.get_horizontal_padding();
-         this._extra_height = themeNode.get_margin(St.Side.TOP) +
-                              themeNode.get_margin(St.Side.BOTTOM) +
-                              themeNode.get_border_width(St.Side.TOP) +
-                              themeNode.get_border_width(St.Side.BOTTOM) +
-                              themeNode.get_vertical_padding();
+         this._extra_width = themeNode.get_margin(St.Side.LEFT) +
+                              themeNode.get_margin(St.Side.RIGHT) +
+                              themeNode.get_border_width(St.Side.LEFT) +
+                              themeNode.get_border_width(St.Side.RIGHT) +
+                              themeNode.get_horizontal_padding();
+         this._extra_height = themeNode.get_margin(St.Side.TOP) +
+                              themeNode.get_margin(St.Side.BOTTOM) +
+                              themeNode.get_border_width(St.Side.TOP) +
+                              themeNode.get_border_width(St.Side.BOTTOM) +
+                              themeNode.get_vertical_padding();

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1880031

Title:
  Thumbnail is stretched if its aspect ratio is less than that of the
  container

To manage notifications about this bug go to:
https://bugs.launchpad.net/gnome-shell-extension-desktop-icons/+bug/1880031/+subscriptions

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to