1/21 Notice - The remaining sCode issues will be smoothed out asap! It's not gonna be forgotten. I've just been unusually busy with offline stuff this weekend.
Hello everyone! This post is to let you know that we've begun to gradually roll out the new sCode processor. Please read the FAQ below as it touches upon what you will notice, and how to report any problems. Please make sure you read this FAQ thoroughly before asking a question or making a report. Please make reports in this topic.
Known problems
As problems are reported, they will be listed here, so you know what's already on our radar.
[spoiler=things that have been fixed]
Troubleshooting broken code
The new filter is a lot stricter about enforcing the rules of both sCode and HTML. It also has a few quirks of its own. Here are some common issues to look for if your code has stopped working. For more help go to this topic, which has the same list there as well: Profile Issues following sCode Updates
[spoiler=Spoiler titles]
When using the spoiler= sCode, you can no longer include sCode within the title; it must be plaintext. If you need a fancier title, you can use this workaround for now:
[spoilertitle]
Which will look like this:
[spoilertitle]
<style type="text/css">
<style>
Examples of things that won't work:
<style type=text/css>
<style type='text/css'>
Hellowill result in an error. The correct order would be:
Hello
hello
world
Will give you two line breaks. But this will give you just one line break in between the lines:
hello
world
This will also only give you one line break:
hello
world
div[style^='width:50%'] { display:none; }
This is because of the square brackets within the code.
This is unintentional, and will be fixed, but is mentioned here because it could be causing your code to break.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
...no, it's not meant to imply anything. The main goal of the rewrite was to modernize the code so we could upgrade to PHP7. The purpose of this announcement was to make people aware that changes are happening, so no one panics if they see really wonky sCode somewhere on the site, they know why it's happening all of a sudden and where to report it :)
It's such a big rewrite that I don't expect a 100% smooth transition, I expect at least a few glitches to pop up. But by being transparent about it, and rolling things out very gradually, we hope to make the transition to the new code as smooth as possible.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Fix incoming for the broken spoilers on forums :)
[edit] and fixed!
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Some personal replies..
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Thank you so much for the report! Looking into it right now.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Okay, the armory and scrollery should be showing up correctly for both those with new sCode on and those still on the old sCode :)
Please continue to ping me for broken stuff like that :) thank you so much!
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
@ Laurey Ooh, interesting, thank you. On it :)
[edit] Fix pushed and will be live in the next 10-15 minutes or so.
Once the fix is live, you may want to refresh the cache on your forum post by editing it.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Nice catch, fixing now.
[edit] Fix pushed, update in progress :)
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
@ Woven Hmm... I'll see what I can do about that preference.
[edit] It's difficult to make the filter respect the preference, and it's impossible (and always was, afaik) to make it respect the preference on the forums. As a compromise, I could do something like...
[url target=blank]www.subeta.net[/url]
...opens in a new tab, but
www.subeta.net
...opens in the same tab.
So that would at least give you some control over it, if we can't outright pass it that preference somehow. What do you think?
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Yes, you understand me :)
What do you mean by "problem", can you describe what you're encountering? The more descriptive, the better :) Can you provide a screenshot of what you're seeing, perhaps? The spoiler in that post works correctly for me. What do you mean by "disappears completely", like the spoiler just completely disappears? And only if you refresh the page? What's it look like right now?
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
I think your spoiler code is a bit broken, you have an equals sign without anything after it.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
I figured out it is because of your unclosed "size" tag. I'll see about making that more forgiving. For now you can fix it by only putting the "size" tag around text.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
For now they won't. I'm going to add it to the first post though, I will try to modify the code to automatically correct it.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Nesting things in the spoiler title like that won't work anymore, I'm afraid. The new sCode is much stricter than the old one, so putting something as a tag "attribute" like that doesn't allow for nested code.
There is a workaround you can use. The CSS on this doesn't currently match, but that's something I can fix. I'll see about cleaning this up and putting it in the sCode reference as an official alternative, if you want to do fancy stuff with it.
[spoilertitle]
[spoilertitle]
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Thanks for the reports! All of the issues you reported have now been added to the first post, so they're on the radar :)
I'm thinking of heading to bed soon ish, will pick back up with these tomorrow.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Wow, that's really weird.
Could you view the source code of the page and try to figure out where that link is happening? (Like there must be a stray a href near the top of the page, but.. where exactly? I find it intriguing that it's not just the sMails but the entire page turns into the link...)
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
I figured it out and have pushed a fix! This one was baffling to pin down.
It happened under very specific circumstances: when you get a reply to an sMail you sent, you'll get a sidebar event saying something like "You have a reply from X in your inbox!" which links to your mail page. That's what caused the page to break. The event link had an unclosed link tag. Usually your browser is smart and can automatically "fix" broken HTML like this but on the mail.php page (which has a lot of other malformed HTML) it wasn't able to do so.
The giant page-link can be fixed therefore by simply deleting the event.
The fix I pushed should cause all events from here on out to use a correctly closed link :) However any notifications you had from before will still have the malformed link since they were created before the fix was out.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Thank you! Added to my to-do list (first post)
It looks like I just forgot the avatar scode completely ;-; I knew I'd forget something. I'll have that corrected asap. My bad.
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
The font-weight is the same as ever, so I'm not sure what you could be seeing.
Could you share a screenshot to show what you mean by extra spacing on comments? And is this on all the comments pages, or just P2P comments, your comments, or one of the other specific pages?
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights
Hey thanks for all your help finding bugs here and there.
Things that have been removed and won't be fixed at the time:
As for the Harvest sCode, I'm still double-checking on whether to add that back in since it's an old function. I'll have an update on it soon.
Let me know if I missed anything, I went back through the last 3 pages of forum posts but there was a lot to skim.
Fix for item scramble has been pushed :) Thank you!
I'm planning to update the sCode reference after this is more fully merged. You can remind me if I forget to do so, I think I must've forgot last time cuz I swear I added the egg and tp scodes, like I seriously thought I did xD
🐝 ☕ bug (he/him) | your friendly neighborhood code wrangler. stay in the loop! join and check out the latest admin post highlights