Skip to content
This repository has been archived by the owner on Jun 13, 2024. It is now read-only.

Assets generally oversized #229

Open
dnfield opened this issue Jul 22, 2020 · 3 comments
Open

Assets generally oversized #229

dnfield opened this issue Jul 22, 2020 · 3 comments
Labels
enhancement New feature or request P3 Priority 3 - Could have

Comments

@dnfield
Copy link
Contributor

dnfield commented Jul 22, 2020

See some context at flutter/flutter#61209

Turning on that setting (I in the terminal - not available through IDEs yet), you can see that most images will get inverted/flipped.

I haven't extensively probed this, but at this point I suspect the following:

  • All scaled images (i.e. images on the 1x 2x 3x 4x directories) are actually scaled too large by as much as a factor of 3.
  • All unscaled images are also too large.

It would be best to scale down the assets themselves, to avoid extra binary size/wire transfer. It would be acceptable to provide cache parameters for the images, particularly if we have to decode them at different sizes for different modes (e.g. desktop vs. mobile) and want to just ship the (perhaps slightly over-sized) version to all platforms.

I can try to take a stab at this, but would want to make sure I'm covering the expected modalities. Ideally, any widget that displays an image would be covered by a test (does not have to be a golden test, but does have to actually resolve the image and try to paint it) that paints it at its expected size(s) to verify that the image is adequately sized.

@dnfield dnfield added the enhancement New feature or request label Jul 22, 2020
@dnfield
Copy link
Contributor Author

dnfield commented Jul 22, 2020

/cc @willlarche fyi

@guidezpl
Copy link
Member

guidezpl commented Dec 9, 2021

Hey Dan, was this on a mobile or desktop device. IIRC, most assets were designed to look nice on desktop, leading them to be oversized on mobile

@guidezpl guidezpl added the P3 Priority 3 - Could have label Dec 9, 2021
@dnfield
Copy link
Contributor Author

dnfield commented Dec 9, 2021

I believe I ran this on Desktop, but it's been a little while.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request P3 Priority 3 - Could have
Projects
None yet
Development

No branches or pull requests

2 participants