Skip to content

Fallback to use image_src if ogp::image isn't set#9

Merged
scottmac merged 1 commit into
scottmac:masterfrom
MitchellMcKenna:feature/image_src-fallback
Mar 18, 2013
Merged

Fallback to use image_src if ogp::image isn't set#9
scottmac merged 1 commit into
scottmac:masterfrom
MitchellMcKenna:feature/image_src-fallback

Conversation

@MitchellMcKenna

Copy link
Copy Markdown
Contributor
  • Some websites (eg. imgur.com) don't have OGP tags but they do
    indicated an image associated with the page using image_src meta tag.
  • Facebook.com's oEmbed also falls back on this if OGP data isn't available (stackoverflow).

- Some websites (eg. imgur.com) don't have OGP tags but they do
  indicated an image associated with the page using image_src.
  Facebook's oEmbed falls back on this if OGP data isn't avalable.
@AramZS

AramZS commented Mar 13, 2013

Copy link
Copy Markdown
Contributor

Looks like a good idea to me.

It's be great if we can also build in a graceful degradation to the first image on the page if nothing else is available.

@MitchellMcKenna

Copy link
Copy Markdown
Contributor Author

@scottmac Thoughts on pulling this in?

@AramZS I've also been thinking about such fallback, so I've opened up a separate issue to discuss that: #11

scottmac added a commit that referenced this pull request Mar 18, 2013
Fallback to use image_src if ogp::image isn't set
@scottmac scottmac merged commit faba415 into scottmac:master Mar 18, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants