airfranceklm / ng-lazy-image

Angular directive for loading responsive image when container (which is preventing reflow) is in viewport.
http://afklm.github.io/ng-lazy-image/
MIT License
350 stars 64 forks source link

Adding alt/title support for SEO purposes #10

Closed simonwalsh closed 9 years ago

simonwalsh commented 9 years ago

Hey guys!

Was wondering if it were possible to add alt/title support for SEO?

We could either have those directly on the div tag (but not very semantic) or have them has a data attribute. Could also be added to the options array, but it might be a bit messy for long strings.

If you see this as a good idea, I can prepare a pull request.

Keep me posted!

SquadraCorse commented 9 years ago

We thought about alternative content and wether it should replace it or not (also take notice we have background images as well). I think alternative title as data attribute is ok since image itself is written with js as well. So now in bower latest. Thanks!

simonwalsh commented 9 years ago

Thanks for fast response time man! Will try that out tomorrow :)