-
Notifications
You must be signed in to change notification settings - Fork 51
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
support for bridgy_omit_link=maybe #659
Conversation
Not quite ready yet... |
OK now I'm done, and it's live on https://bridgy-kwm.appspot.com/ (CI checks are failing because this depends on snarfed/granary#86) |
</label> | ||
<label> | ||
<input id="omit-link-maybe" name="bridgy_omit_link" value="maybe" type="radio"> | ||
if shortened |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor, consider an alt
tooltip with a more detailed explanation?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea! Also reminded me I meant to add it to the docs too.
lgtm yay! especially if you've tested manually that interactive still works w/o js as well as with. :P thanks again! |
interpret bridgy_omit_link=maybe as "include link iff the content is shortened". fixes snarfed#561
dfc0a5a
to
80a0d46
Compare
🎵 well I just met you |
@snarfed it acts weird with the cached bridgy.js -- do you do any kind of |
sadly no, but its expiration is only 1h. |
interpret bridgy_omit_link=maybe as "include link iff the content is
shortened".
(my 2-line-fix ended up being more like 300 lines across 11 files ... nice estimation abilities, me)
fixes #561