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

include img alt text in photo property #56

Closed
kartikprabhu opened this issue Feb 17, 2018 · 5 comments
Closed

include img alt text in photo property #56

kartikprabhu opened this issue Feb 17, 2018 · 5 comments

Comments

@kartikprabhu
Copy link
Owner

Originally by @snarfed

microformats#83

@kartikprabhu
Copy link
Owner Author

kartikprabhu commented May 23, 2018

only relevant while parsing u-photo; p-photo will pick up the alt and not the url anyway.

Major change so for version 2.0.0 (if it goes into spec). Put under a flag for now alt_in_photo ?

@snarfed any "better" suggestions for the flag name?

@snarfed
Copy link

snarfed commented May 23, 2018

thanks! nah, i'm agreeable.

@snarfed
Copy link

snarfed commented May 23, 2018

however, i'd strongly encourage standardizing on microformats/mf2py for issues and turn them off here. 😄 tracking issues across multiple repos is a recipe for confusion and skew.

@kartikprabhu
Copy link
Owner Author

@snarfed yes the actual issues on the main repo are the "real" ones. These are just for my notes and memory since it seems right now I am the only one actively coding this thing!

@snarfed
Copy link

snarfed commented May 23, 2018

right, i understand...but they're still public, and some are duplicated, and some will skew, all of which confuses readers.

take notes for memory on the main repo issues! totally appropriate. notes are useful for other people too, collaboration and all. no one will complain. 😁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants