Skip to content
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.

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

[css-images] lazyload #3659

Open
Malvoz opened this issue Feb 19, 2019 · 5 comments
Open

[css-images] lazyload #3659

Malvoz opened this issue Feb 19, 2019 · 5 comments
Assignees
Labels
css-images-4 css-values-5 topic: urls Tracking issues that'll need edits when CSS URLs are defined on top of Fetch.

Comments

@Malvoz
Copy link
Contributor

Malvoz commented Feb 19, 2019

Hello CSSWG folks!

With no personal preference I ask you to consider CSS delegation for the lazyload loading attribute: whatwg/html#3752

@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed lazyload.

The full IRC log of that discussion <dael> Topic: lazyload
<dael> github: https://github.com//issues/3659
<dael> Rossen_: Ask from WHATWG. fantasai you want to introduce?
<dael> fantasai: Mostly I wanted to call attention to the thread to make sure we respond in a timely manner
<dael> Rossen_: Action for everyone to read up and engage in github. I'll add to agenda next week if not high response

@heycam
Copy link
Contributor

heycam commented Apr 30, 2019

Is this issue asking for a new mechanism that works like the lazyload attribute but which affects CSS images? (After skimming through the changes in the linked PR it doesn't look like it affects CSS images currently.)

@gregwhitworth
Copy link
Contributor

Having read through the agonizingly long PR, the thing I think is being raised is on the Intent to Ship Google group here:

The resulting issue is that currently background images are not being loaded due to this change which is considered a bug and should be fixed prior to shipping per this comment.

Also it seems that @svgeesus pointed out that #1603 should be able to address this issue but it doesn't seem any modifiers (that I can currently find) have been defined, thus this won't be loaded lazily from what I can tell.

But... if that isn't the case it would be good to have clarification @Malvoz

@atanassov atanassov removed the Agenda+ label May 15, 2019
@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed lazyload.

The full IRC log of that discussion <dael> Topic: lazyload
<dael> github: https://github.com//issues/3659
<dael> Rossen_: Don't know if gregwhitworth made it
<dael> gregwhitworth: I'm here
<dael> gregwhitworth: This isn't my issue, but I dug into what I think this person wanted that bg images weren't be loaded by lazyloading. I'm not sure what this person wanted so I suggest we close until more clarifiction. Chris pointed out there is already an issue from AmeliaBR about modifiers. I'd close as dup and ask for more details.
<dael> Rossen_: Close the issue back to the owner and ask for more details?
<dael> gregwhitworth: I pinged him and asked to clarify. I'd close as dup to the one Chris L referenced.
<dael> Rossen_: Any other members that read or want to discuss this?
<dael> Rossen_: If not we can do that
<dael> Rossen_: No hearing takers. I'll clear up the labels and move the issue back to the owner

@atanassov atanassov assigned atanassov and Malvoz and unassigned atanassov May 15, 2019
@rik
Copy link

rik commented Apr 11, 2020

whatwg/html#3752 has been merged and the feature is shipping in several browsers. Is it a good time to have another look at it in this WG?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
css-images-4 css-values-5 topic: urls Tracking issues that'll need edits when CSS URLs are defined on top of Fetch.
Projects
None yet
Development

No branches or pull requests

7 participants