Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Warning: Undefined property: stdClass::$ip in /home/public/blog/class.comment.php on line 31
Comments!
Last updated:
The day has finally arrived! I added comments to the site. I expect I'll tweak them a lot over the next few weeks, but feel free to go crazy on them for now.
If you want to post a test comment, do so on this post.
Commenting Policy
I will delete the fuck out of your comments if you're trolling, spamming, or just plain vacuous, at my own discretion. If you don't have something worthwhile to add, don't post.
Markdown Syntax
I currently support most of Markdown in the comments. You can check the progress of the comment parser at my markdown github repo.
You can use all of the block-level elements except headings.
You can use all of the inline-level elements, including reference-style links.
Few comments about your comment system (in fact, this is my own todo/check list for the comment system, even if some items are already done):
Does it retain my credentials (cookie/localStorage) so that I don't have to type them every time?
Does it send you a mail when a new comment is added on your blog (you don't want to miss comments, do you)? Does it support guest subscription on a blog post?
Do you have a easy way to delete/modify comments out of order, if something go wrong (spam)?
If you support comment subscription, there should be a link in the mails to allow people to unsubscribe.
Does it support all unicode glyphs like ♥ and ∫sin(x).dx ? Is it <a href="about:invalid">secure?
Yay, I really liiiiike it :-) I hope to make more useful comments to your wonderful blog posts soon!
All right, credential-saving now works, woo! Uses localStorage, very trivial.
I think for comment subscription, I'll provide comment rss feeds. Easier for me, as I don't have to screw around with remembering people's emails and preferences.
I can easily delete/modify comments - I just pull down the appropriate file from FTP, modify it, and put it back up again. ^_^
Comment permalink is wrong, they miss the “b” in first “b4K90”, so now they give 403.
While the width of the blog is fluid, the width of comments is not, so adding some max-width (with some other minor fixes to take care of paddings) would be great.
It would be nice to add something more than simple space between the comment number and the date, right now it's a bit hard to read.
Thanks for the suggestions, Roman! Fixed all of them, but there's still some fluidity issues with the blog header. I need to pull out a MQ to switch the header to be vertical when it gets too small.
I find it's too hard to read the first couple of lines of linked-to comments, over the black end of the gradient. (In Firefox. Possibly the gradient gradients differently in other browsers.)
Ah, more specifically, the ‘black text on almost-black gradient’ problem only occurs on long comments, because the gradient is stretched over the height of the comment, so the black part goes down farther than on short ones.
That's... not what they're supposed to look like. It appears that FF is dropping the background declaration that should be setting that gradient to a 2px stripe along the right side. I wonder why?
Hmmm, I get the same behaviour as Firefox in Chromium 18 (WebKit 535.19). I see the intended behaviour — which looks really nice, by the way — in Chrome 21 (WebKit 537.1).
Nothing special about those versions: they just happen to be the two WebKit-based browsers I have installed. But perhaps if you can spot the difference between those, it'll give a clue to the Firefox behaviour too.
Your comment system dropped my first paragraph. The fully comment was:
Hmmm, I get the same behaviour as Firefox in Chromium 18 (WebKit 535.19). I see the intended behaviour — which looks really nice, by the way — in Chrome 21 (WebKit 537.1).
Nothing special about those versions: they just happen to be the two WebKit-based browsers I have installed. But perhaps if you can spot the difference between those, it'll give a clue to the Firefox behaviour too.
The first paragraph above is “Re #19: Text on the same line as the reference.” I'm testing a hypothesis that if the comment starts with a reference, any following text in that para gets dropped.
I'd previously thought that reply lines should be a new block type in my comment-markdown. I miswrote the regex to recognize them, though, which ended up swallowing any text following (I'd intended lines with more text to just not be replies at all).
I've now patched it in a different way - reply lines start a new paragraph with a reply-to. Your comments are now revealed!
I suspect it's a problem in parsing the background-size part of the shorthand. I'll verify that the current Chrome behavior is correct (I think it is). If so, I'll leave it as it is, because that's what standards are for. If not, I'll fix it. ^_^
Did you figure out whether this is the correct behaviour or not? I created a minimum test-case at http://codepen.io/karaken12/pen/GveEh -- it works fine on Chrome 21, but looks terrible on Firefox 15.0.1.
I'd suggest changing the "Reply?" to a link which shows a form and hiding the reply form below each comment because those forms now take most of the space and distract from the actual comments themselves.
In decent browsers, that's exactly what happens already - I'm using a <details> element to hide the comment form until you wish to see it.
I'm fine with legacy browsers not rendering my page as expected, since I'm following mature specs. ^_^
Reply?
All content on this blog, unless otherwise noted or obviously owned by someone else, is licensed under the
CC0 license
.
This means you can use, repost, translate, chop up, remix, or do anything else you want with the contents of this blog,
without asking me.
(Seriously, don't ask me. It's annoying. That's why I use this license.)
+1
Reply?
It looks good!
Reply?
Few comments about your comment system (in fact, this is my own todo/check list for the comment system, even if some items are already done):
Reply?
Oh, I know it's stupid but I like
textarea { overflow-y: auto; }
(btw, this is a test for preserving whitespace)
Reply?
Hurp durp, can't believe I forget to add white-space:pre-wrap; to the rules.
Reply?
All right, credential-saving now works, woo! Uses localStorage, very trivial.
I think for comment subscription, I'll provide comment rss feeds. Easier for me, as I don't have to screw around with remembering people's emails and preferences.
I can easily delete/modify comments - I just pull down the appropriate file from FTP, modify it, and put it back up again. ^_^
Reply?
Yay for simplicity! And for comments! Woo!
Reply?
Minimalistic and light. Looks good ! :)
Reply?
Yay!
max-width
(with some other minor fixes to take care of paddings) would be great.Reply?
Reply?
Thanks for the suggestions, Roman! Fixed all of them, but there's still some fluidity issues with the blog header. I need to pull out a MQ to switch the header to be vertical when it gets too small.
Reply?
Thank you for adding comments.
I find it's too hard to read the first couple of lines of linked-to comments, over the black end of the gradient. (In Firefox. Possibly the gradient gradients differently in other browsers.)
Reply?
Ah, more specifically, the ‘black text on almost-black gradient’ problem only occurs on long comments, because the gradient is stretched over the height of the comment, so the black part goes down farther than on short ones.
See comment #6 above for an example.
Reply?
That's... not what they're supposed to look like. It appears that FF is dropping the
background
declaration that should be setting that gradient to a 2px stripe along the right side. I wonder why?Reply?
Hmmm, I get the same behaviour as Firefox in Chromium 18 (WebKit 535.19). I see the intended behaviour — which looks really nice, by the way — in Chrome 21 (WebKit 537.1).
Nothing special about those versions: they just happen to be the two WebKit-based browsers I have installed. But perhaps if you can spot the difference between those, it'll give a clue to the Firefox behaviour too.
Reply?
Your comment system dropped my first paragraph. The fully comment was:
Hmmm, I get the same behaviour as Firefox in Chromium 18 (WebKit 535.19). I see the intended behaviour — which looks really nice, by the way — in Chrome 21 (WebKit 537.1).
Nothing special about those versions: they just happen to be the two WebKit-based browsers I have installed. But perhaps if you can spot the difference between those, it'll give a clue to the Firefox behaviour too.
Reply?
Text on the same line as the reference.
A second paragraph, following the reference.
The first paragraph above is “Re #19: Text on the same line as the reference.” I'm testing a hypothesis that if the comment starts with a reference, any following text in that para gets dropped.
Reply?
Ha, interesting. If that's happening, it's a bug. The source of your comment is still around, though. Let me poke at it.
Reply?
All right, all fixed.
I'd previously thought that reply lines should be a new block type in my comment-markdown. I miswrote the regex to recognize them, though, which ended up swallowing any text following (I'd intended lines with more text to just not be replies at all).
I've now patched it in a different way - reply lines start a new paragraph with a reply-to. Your comments are now revealed!
Reply?
I suspect it's a problem in parsing the background-size part of the shorthand. I'll verify that the current Chrome behavior is correct (I think it is). If so, I'll leave it as it is, because that's what standards are for. If not, I'll fix it. ^_^
Reply?
Testing comments with the static pages.
Reply?
testing non-logged in comments
Reply?
Did you figure out whether this is the correct behaviour or not? I created a minimum test-case at http://codepen.io/karaken12/pen/GveEh -- it works fine on Chrome 21, but looks terrible on Firefox 15.0.1.
Reply?
Yes, I verified that what I'm doing is the standards-compatible thing. Firefox just hasn't yet updated (nor have the mobile browsers).
Reply?
I'd suggest changing the "Reply?" to a link which shows a form and hiding the reply form below each comment because those forms now take most of the space and distract from the actual comments themselves.
Reply?
In decent browsers, that's exactly what happens already - I'm using a <details> element to hide the comment form until you wish to see it.
I'm fine with legacy browsers not rendering my page as expected, since I'm following mature specs. ^_^
Reply?