creesch

Results 223 comments of creesch

@agentlame @TheEnigmaBlade @Dakta @JordanMilne @LowSociety I could use some input here. We changed all unescapes() in 3.0.4 to decodeURIComponent. Reasoning being that it prepares people for when reasons starts to...

Well, I didn't really go with a smart schema solution since I didn't think we would need a new schema for it. So basically I went for the bruteforce method...

Oh btw, places where it is used https://github.com/creesch/reddit-moderator-toolbox/blob/master/modules/removalreasons.js#L226 https://github.com/creesch/reddit-moderator-toolbox/blob/master/modules/config.js#L117 https://github.com/creesch/reddit-moderator-toolbox/blob/master/modules/config.js#L384

moving this to 3.2, clearly this needs a bit more thought.

Sigh... If we ever want to do this I fear we need to do a new version schema...

Still relevant but does require a schema change if we ever tackle this with a two stage rollout. I tagged the modules that make use of versioned json schemas, I...

> Update: Yea, nevermind. Modules are a lot more complex than I thought. Toolbox modules are somewhat complex at first glance but there is a structure to it. Also, looking...

Oh heh, I just realized that unhide all at least works so that is good. Still like to have a way to do it for just the most recent post.

> Adding a button for this feels like it'll be a bit clunky to me, but I don't really like this feature to begin I didn't either until we started...

Seems like it has been like this for a long long time... At least the selectors in toolbox as it looks for `.remove-button` which the spam button also has as...