-
Notifications
You must be signed in to change notification settings - Fork 0
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
Consistent way of writing #1
Comments
Oh, I actually don't use that part of the code! This comes from the original kirby-webmentions plugin by Bastian Allgeier. I have my own snippets in See examples here: I don't know if I want to ship the plugin with those icons, but I will make some changes in the default snippets to solve this better :) (Also, make sure you got the latest version of either my seblog-kirby-webmentions or Bastian's kirby-webmentions, because it fixes a nasty vulnerability. Haven't blogged about it yet, I will soon.) |
Ah ok. Thanks for clarifying. I don't think Bastian is maintaining the plugin right now. Will look deeper into your other things as well. The toolkit looks very nice. |
Nope, that's why I started my own fork. First just for myself, but I think it's nice to have a version that is useful to others also. So: thanks for using it! Feedback is appreciated :) |
Great. Again: your work is highly appreciated ;) I still ave a lot to integrate such as the possibility to reply from within my blog. I wish I could attend the IndieWebCamp I organise in Düsseldorf once, to find the time. Are you coming? |
Well, if you don't mind doing some manual work, you can actually just add: <div class="h-entry">
In reply to <a href="https://seblog.nl/2017/02/07/3/reply" class="u-in-reply-to">Sebastiaan</a>
<p class="p-content p-name">... :)</p>
</div> ... to the I use a separate field for it, because it makes sense to do so in Kirby, but it's not required! I plan on coming to Düsseldorf, really looking forward to that! |
Hey,
first of all: lovely work. Thanks a lot.
I just thought, that it might be nice to be consistent in who you write the headlines for what mention it is. Either all lower case or all upper case. Easy to solve on my end of course, but I thought you maybe want to have a consistent way of doing this in the repo already. I attached screenshots of what I mean. Maybe you also had a reason for doing it this way … ;)
Cheers,
/marc
The text was updated successfully, but these errors were encountered: