Skip to content

Deprecate the Lazy Image package #2004

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
javiereguiluz opened this issue Jul 23, 2024 · 5 comments
Open

Deprecate the Lazy Image package #2004

javiereguiluz opened this issue Jul 23, 2024 · 5 comments
Labels
LazyImage Stalled

Comments

@javiereguiluz
Copy link
Member

As discussed in #1967, the solution proposed by the Lazy Image package (based on the "blurhash" library) was useful in the past. However, nowadays all browsers support many native features to improve the loading of images. See https://web.dev/learn/images/performance-issues

So, maybe we could deprecate this package to reduce the maintenance work? Thanks!
@smnandre
Copy link
Member

What is the "usual" scenario there?

  • deprecate the component (what about the doc ? should we explain other solutions?)
  • abandon it ? (after how much time?)

This is very new to me ^^

@kbond
Copy link
Member

kbond commented Jul 24, 2024

I agree, my plan (whose timeline is unknown at this point), was to create a "recipe" system on ux.symfony.com first, then deprecate the ux components that could easily be replaced by a recipe (once the recipe exists). This allow us to point users to a concrete replacement.

@javiereguiluz javiereguiluz mentioned this issue Jan 10, 2025
Merged
Kocal added a commit that referenced this issue Jan 16, 2025
@Kocal
minor #2501 [Doc] Add a warning message to not recommend Lazy Image p…
562c079 
…ackage (javiereguiluz)

This PR was merged into the 2.x branch.

Discussion
----------

[Doc] Add a warning message to not recommend Lazy Image package

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Issues        |
| License       | MIT

Related to #2004. If we don't deprecate the package, at least warn users that it's no longer recommended. Thanks.

Commits
-------

519c4ce [Doc] Add a warning message to not recommend Lazy Image package
@carsonbot
Copy link

Thank you for this issue.
There has not been a lot of activity here for a while. Has this been resolved?

@carsonbot
Copy link

Friendly reminder that this issue exists. If I don't hear anything I'll close this.

@carsonbot
Copy link

Hey,

I didn't hear anything so I'm going to close it. Feel free to comment if this is still relevant, I can always reopen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
LazyImage Stalled
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@javiereguiluz @kbond @smnandre @carsonbot