Beta Toolbar

JohnK · 2011-11-30 11:23:44

Just a small bug but if you take a look at the thread http://www.boards.ie/vbulletin/showthread.php?t=2056465735 - in the breadcrumbs on the toolbar its finishing in &q... in place of the actual double quote; I know the text would be considered too long and need to be shorted but since those 12 characters from the two "…

30-11-2011 11:23AM

#1

JohnK

Registered Users, Registered Users 2 Posts: 4,775 ✭✭✭

Join Date: December 2002

Posts: 4574

Just a small bug but if you take a look at the thread http://www.boards.ie/vbulletin/showthread.php?t=2056465735 - in the breadcrumbs on the toolbar its finishing in &q... in place of the actual double quote; I know the text would be considered too long and need to be shorted but since those 12 characters from the two " would have displayed as only 2 its cut off extra characters from the actual title which in this case would flag there are spoilers in the thread.

So maybe instead of snipping [US] S06E09 "Get Gellar" [SPOILERS WITHIN] snip [US] S06E09 "Get Gellar" [SPOILERS WITHIN] then encode it.

Like I said its really only a small thing but would make it look a bit neater.

0

Comments

#2 30-11-2011 01:06PM

Boards.ie: Danny

Closed Accounts Posts: 3,609 ✭✭✭

Join Date: October 2011

Posts: 3538

I'm not sure I agree with your use case - the actual vBulletin breadcrumb, in much bigger fontsize and providing the full thread title is a much better source of determining if a thread has spoilers as opposed to a snipped breadcrumb in the beta menu; What about long episode names? Then the spoiler tag doesn't appear even if we decoded the thread title before snipping

Nonetheless, you were correct in that the thread title should be decoded first before snipped to length and then re-encoded. The fix for this is in, it should go live soon.

Danny

0

Beta Toolbar

Comments

Leave a Comment