fu: Close-up of Fu, bringing a scoop of water to her mouth (Default)
fu ([personal profile] fu) wrote in [site community profile] dw_beta2012-09-23 11:54 am

Comment Pages in S2: the old becomes new

One of our long-running projects is to remove the old crufty code written in BML (our old templating system) and replace it with better alternatives throughout the site. And the coding for talkread.bml -- which is what is used any time you load up a site-skinned comment page -- is a particularly painful example.

Add to that, having the site-skinned comment pages be written one way, and custom comment pages another, makes it likely for one to lag behind the other. For example: site-skinned comment pages show the link for entries with screened comments as: "x visible | y screened". But S2 comment pages didn't have that information until this code push!

So I'm really happy to say that we have a new version of the comments page which uses pure S2, so now site-skinned comment pages and journal-styled comment pages are formed the same way on the backend even if they don't look the same. [personal profile] exor674 worked hard on the backend to make this possible, then [personal profile] momijizukamori came through with some spit and polish (with some help from [personal profile] kunzite).

The idea is for the new version to look as much as possible like the old version, though there might be a few discrepancies, because the backend is so different. [personal profile] momijizukamori has done her best to mimic the look and feel of the old comment pages. But the comments page is one of the most used pages here, so we'd like you all to come and give it a good shakedown.

Turn on beta testing for the "New S2 Comment Pages" -- and let us know in comments if you find anything out of the ordinary.

List of known issues.

ahappylie: (Thinking)

comment title link feedback

[personal profile] ahappylie 2012-09-23 04:43 am (UTC)(link)
Just a visual tic of mine, but having three ways to link to a single comment seems a bit redundant to me. The comment title turning into a link changes the visuals of the site in a detrimental way to me, what with the extra underline on the title doubling up against the journal name and its underline and the overabundance of red in the header. Leaving the header as is with a black non-linkable title and the (Link) seems fine to me.

I've heard this is an issue with the S2 system that the developers weren't sure if they wanted to keep in, so I understand the reasoning, but that's my two cents on it so far.
musyc: Silver flute resting diagonally across sheet music (Default)

Re: comment title link feedback

[personal profile] musyc 2012-09-23 04:57 am (UTC)(link)
Seconding this. Leaving the subject headers as they were is preferable. Making them clickable links is very distracting in site scheme (Tropo Purple, here).
momijizukamori: Rufus Shinra from Final Fantasy 7 - hubric incarnate. (Rufus | hubris)

Re: comment title link feedback

[personal profile] momijizukamori 2012-09-24 07:12 pm (UTC)(link)
They've now been changed back to plain text!
chagrined: Marvel comics: zombie!Spider-Man, holding playing cards, saying "Brains?" (brains?)

Re: comment title link feedback

[personal profile] chagrined 2012-09-23 05:02 am (UTC)(link)
I also really dislike the anchor-text-colored and underlined subjects on comments. I don't mind the fact that they link to the comment? But wish they'd lose the underline and revert to the regular non-link-text color. (For example, a lot of DW journal styles do that same thing for entry subjects. They are links to the entry, but they don't have the underline and they're the same color as the regular text, not the links.) (eta: this is when viewing with ?style=site, and I'm using tropo red)
Edited 2012-09-23 05:03 (UTC)
momijizukamori: Grey tabby cat with paws on keyboard and mouse. The text reads 'code cat is on the job', lolcats-style (CODE CAT)

[personal profile] momijizukamori 2012-09-23 05:04 am (UTC)(link)
Oh, that's a method I hadn't thought of! Though it's generally not great practice to have a link indistinguishable from text...

But I'll definitely keep it on the table *g*
chagrined: Marvel comics: zombie!Spider-Man, holding playing cards, saying "Brains?" (brains?)

[personal profile] chagrined 2012-09-23 05:12 am (UTC)(link)
DW does already do it all over the site in a lot of their journal styles, though? But if you mean like, in the case of it being the site style, I see the point. (If it's changed to the new way, for myself I'd prolly just write a stylish script to make them look how I want and all.)

(Some other things I noticed I'm gonna put in a new comment now, heh.)
momijizukamori: (dreamsheep | styles)

[personal profile] momijizukamori 2012-09-23 05:14 am (UTC)(link)
A lot of them distinguish on hover (ie, change color or the underline appears), though yeah, a few don't. And yeah, not so great to have it on site scheme - but as I said, it's definitely an option for us to consider :)
marahmarie: my initials (MM) (Carlos Estevez)

Sure...

[personal profile] marahmarie 2012-09-23 06:46 am (UTC)(link)
No linkies: http://i.imgur.com/CZs98.jpg

Beta testing on: http://i.imgur.com/TN8k6.jpg

With this I can officially crown myself the queen of online errors. It never stops!
momijizukamori: Green icon with white text - 'I do believe in phosphorylation! I do!' with a string of DNA basepairs on the bottom (Default)

Re: Sure...

[personal profile] momijizukamori 2012-09-23 06:53 am (UTC)(link)
Hmm. It actually looks like you're not getting the beta style at all - you don't have any of the differences others have reported. If you look at the source, is it a huge table, or stacked divs for the comments?
chagrined: Marvel comics: zombie!Spider-Man, holding playing cards, saying "Brains?" (brains?)

Re: Sure...

[personal profile] chagrined 2012-09-23 10:34 am (UTC)(link)
I had to throw ?style=site on mine for it to show up. Even w/ beta feature on and viewing this entry which uses site style for comments, I got the old style otherwise. But I figured that was intentionally for testing or whatnot.
montuos: cartoon portrait of myself (Default)

Re: Sure...

[personal profile] montuos 2012-09-23 01:12 pm (UTC)(link)
Ditto here; no evidence of beta style after turning it on. Source shows divs, not table. Tried clearing cache, turning off all Greasemonkey scripts, signing out and in again, restarting browser; nothing worked but [personal profile] chagrined's suggestion of ?style=site. Tropo Red, Nouveau Oleanders, Firefox 15.0.1, Kubuntu 12.04. Edit: Also not seeing it in Chrome 21.0.1180.89.
Edited 2012-09-23 13:16 (UTC)
marahmarie: my initials (MM) (Default)

Re: Sure...

[personal profile] marahmarie 2012-09-24 01:40 am (UTC)(link)
Fu dropped a comment below saying she fixed it (there was an error apparently that as usual, *sigh* affected me). Thanks for looking into it, though. :)
marahmarie: my initials (MM) (Default)

Re: Sure...

[personal profile] marahmarie 2012-09-24 01:39 am (UTC)(link)
Fu, love you, thanks so much! :)
momijizukamori: Jyuushirou Ukitake from Bleach, grinning happily (Ukitake | niko niko)

Re: comment title link feedback

[personal profile] momijizukamori 2012-09-24 07:14 pm (UTC)(link)
Fu worked out a version of the code she was happier with, so thankfully no need to resort to CSS hacking - they're now straight-up plain text~
chagrined: Marvel comics: zombie!Spider-Man, holding playing cards, saying "Brains?" (brains?)

Re: comment title link feedback

[personal profile] chagrined 2012-09-24 07:16 pm (UTC)(link)
I saw when I came to reply to your other comments! :D Cool, glad it worked out. They work a lot more easily for my eyes now again, heh.
asklepios: ᴀᴄᴛʀᴇss ɴᴀᴛᴀʟɪᴇ ᴅᴏʀᴍᴇʀ; ʙʟᴏɴᴅᴇ (Default)

[personal profile] asklepios 2012-09-23 11:04 pm (UTC)(link)
+1 to this entire comment - the extra link doesn't actually serve any purpose except to put something in the way where I might accidentally click on it while trying to do something else to the comment. There are already two clearly marked ways of linking to the comment there, I'm not sure why anyone would want a third that doesn't add anything useful?
momijizukamori: (dreamsheep | styles)

[personal profile] momijizukamori 2012-09-24 07:02 pm (UTC)(link)
In most S2 styles, they were being used to build the page summary - which isn't a part of this style, and the 'thread' link only appears if a comment has replies. But subject lines are now unlinked, leaving just the small one in the header :)
yrbirdcanscene: blonde in a coat from a dior ad (dior coat)

[personal profile] yrbirdcanscene 2012-09-23 11:23 pm (UTC)(link)
I have to agree with all of this. I've tried it in Chrome, Firefox, and RockMelt, both with my Grease/Tamper Monkey and Stylish scripts on and off, and I find the linked titles to be ugly, quite frankly. The link is redundant, as there are multiple ways to access a comment link already.

Additionally, I like to comment from my smartphone, and don't like the mobile interface (which is neither here nor there), so I use the regular site, as it displays just fine. But with the comment titles made links, that's just one more thing I can accidentally click on while I'm just trying to scroll through the page.

Essentially, I don't think there was anything wrong with the previous comment title format (i.e. non-linked), and would be thrilled if it did not change.
boomvox: (pic#2886153)

[personal profile] boomvox 2012-09-23 11:29 pm (UTC)(link)
+1 to all of this, especially the mobile issue which I didn't even think of at first. I use the normal page through my smartphone's browser too, which is actually a lot easier than the mobile interface and just fine. But links in comment subjects are going to make scrolling and trying to click on anything else a real crap shoot.
momijizukamori: Young Vergil from the Devil May Cry doujinshi Bless. The text reads 'Turn you into stars' (kid!Vergil | turn you into stars)

[personal profile] momijizukamori 2012-09-24 07:16 pm (UTC)(link)
They've been changed! And thankfully the mobile interface should be getting a redesign to haul it into the 21st century soon (though that's not my project)
boomvox: (pic#3208725)

[personal profile] boomvox 2012-09-24 07:37 pm (UTC)(link)
Hooray to both of those things!
momijizukamori: Green icon with white text - 'I do believe in phosphorylation! I do!' with a string of DNA basepairs on the bottom (Science! | I do believe in phosphorylati)

[personal profile] momijizukamori 2012-09-24 07:08 pm (UTC)(link)
It's actually not a change from S2 - all journal styles have linked comment titles - but based on feedback we went ahead and changed it to mimic the BML style, so you should have happily unlinked comments now :)
momijizukamori: Green icon with white text - 'I do believe in phosphorylation! I do!' with a string of DNA basepairs on the bottom (Default)

Re: comment title link feedback

[personal profile] momijizukamori 2012-09-24 07:00 pm (UTC)(link)
Ask and ye shall recieve ;) They're now unlinked.
ahappylie: (Fangirly)

Re: comment title link feedback

[personal profile] ahappylie 2012-09-26 09:49 am (UTC)(link)
You guys are awesome. ;A;