lightning-browser-extension icon indicating copy to clipboard operation
lightning-browser-extension copied to clipboard

Website screen logos are slightly off

Open MoritzKa opened this issue 3 years ago • 6 comments

Describe the bug

The logos of app on the "Website" screen do not fill the whole place properly.

Screenshots of the changes [optional]

image

To Reproduce

Steps to reproduce the behavior:

  1. Visit the "Websites" screen in the Alby extension

Expected behavior

logos fill the place properly without a white space.

Alby information

  • Alby Version: 1.17

Device information [optional]

  • OS: Windows
  • Browser Chrome

Are you working on this issue?

No

MoritzKa avatar Oct 06 '22 14:10 MoritzKa

The broken logo thumbnails have a transparent gap at the bottom of each SVG image. This is the reason for the incorrect display. In my screenshots you can see the spacing at the bottom.

To display the images without visually errors, all thumbs should be square and completely filled with a (background) color.

Maybe @auwalrg8 can optimize affected thumbnails.

👎

  • Thumb 1

wrong 2

  • Thumb 2

wrong

👍 perfect

rottingcleaner avatar Oct 06 '22 17:10 rottingcleaner

@rottingcleaner is it possible to use a fill property via css to always fill the square? That means maybe 1-2px might be cut of but at least we solve the general issue no matter what exact icon proportions?

escapedcat avatar Oct 07 '22 04:10 escapedcat

@escapedcat I don't think, it will solve the problem. But I can try it. Which color I should use to fill the squares?

From my perspective we should use prepared thumbnails without spacing around. That will increase the quality.

rottingcleaner avatar Oct 07 '22 09:10 rottingcleaner

@escapedcat I don't think, it will solve the problem. But I can try it. Which color I should use to fill the squares?

Um, no, I mean something like this: https://developer.mozilla.org/en-US/docs/Web/CSS/background-size background-size: cover;
Although on this issues' case it's not a background-image.

From my perspective we should use prepared thumbnails without spacing around. That will increase the quality.

Sure, that's the nicer solution

escapedcat avatar Oct 07 '22 09:10 escapedcat

Ah, ok. I tried out but also this solution is not really an option as long the thumbnail/image itself has a transparent space.

I think we have no posibilities to fix the issue clean on the CSS side.

Screenshot 2022-10-07 at 12 14 17 Screenshot 2022-10-07 at 12 17 40

rottingcleaner avatar Oct 07 '22 10:10 rottingcleaner

not really an option as long the thumbnail/image itself has a transparent space

Right, thanks for checking!

escapedcat avatar Oct 07 '22 10:10 escapedcat

This seems to be resolved, right?

stackingsaunter avatar Mar 16 '23 00:03 stackingsaunter

Yes. That looks good if I look at the rendered ratio.

chrome_zTqCzayD4H

MoritzKa avatar Mar 16 '23 09:03 MoritzKa