-
Notifications
You must be signed in to change notification settings - Fork 52
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
publish: support pictures #85
Comments
one big question here is whether there's mf2 markup to identify pictures that are the primary content of a post, as opposed to just decoration. if not, I can start with just the first picture(s) inside the e-content. |
@snarfed how about using the u-photo of the h-entry and as a fallback use the first image. FB uses the |
sgtm! |
I'm marking up h-entries as h-as-image for posts which are primarily an image post. I stole it from others doing it, along with h-as-page, h-as-article, h-as-bookmark http://microformats.org/wiki/activitystreams It is an experimental markup, I believe. There is also a proposal for a u-photo in an h-entry noted as an option for marking up the primary image. |
for anyone interested in tackling this, the changes would (probably) be isolated to just |
lots of good discussion on irc today on how to determine whether a post is a "photo" post, and if so, which photo(s). they've started wikifying it on Photos#Discussion. |
I'm pretty well convinced that a note with a photo should be syndicated like a photo, full stop. In the discussion today, I could only come up with one counter-example https://kylewm.com/note/2014/05/05/1 and even that is kind of a toss up. |
done for twitter! by @notenoughneon in snarfed/granary#16. it just uses the first u-photo. |
moving back to now since i'd like to do this for facebook too. |
I've started working on facebook support. |
awesome, glad to hear it! |
@notenoughneon exciting! there's a little bit of code in redwind for posting photos to facebook, feel free to reference or ignore :) |
thanks again, @notenoughneon! done in snarfed/granary#20. |
needs new API usage for both FB and twitter, but should be pretty straightforward.
cc @kartikprabhu
The text was updated successfully, but these errors were encountered: