XKit-Rewritten icon indicating copy to clipboard operation
XKit-Rewritten copied to clipboard

No Recommended: hide "more like this" after liking posts

Open marcustyphoon opened this issue 3 years ago • 6 comments

my brain is fried, I don't think this is good UI copy (should it use "more like this" maybe?), and I can't remember if the "more like this" string is already used elsewhere

User-facing changes

  • Adds an option to No Recommended to hide the "more like this" recommended posts that appear after you like a post

Technical explanation

In a departure from the other carousel hiding scripts, this targets the header text instead of the DOM structure of the recommendation bit; I don't know if chicletRow is already used elsewhere/will get used elsewhere.

If Staff adds RNG to the title, then idk, I give up :P

Issues this closes

resolves #637

marcustyphoon avatar Jun 18 '22 18:06 marcustyphoon

There's a "More like this" title for recommendations in the photo lightbox. Do the chiclet posts themselves have no useful react data?

AprilSylph avatar Jun 18 '22 19:06 AprilSylph

Hm. I assume not a lot of unrelated components would subscribe to relatedPostsContext. Seems reasonable.

marcustyphoon avatar Jun 18 '22 20:06 marcustyphoon

I assume(?) that the test feature this PR is for is not currently active; feel free to close if so, and this can be reopened if it resurfaces.

marcustyphoon avatar Jul 22 '22 20:07 marcustyphoon

It's 5 hours too late for me to check, so I'll have to try to remember to look on Monday.

AprilSylph avatar Jul 22 '22 20:07 AprilSylph

It's still active. My main account has it.

AprilSylph avatar Jul 25 '22 17:07 AprilSylph

Gotcha. Darn, my test account appears to have lost it.

marcustyphoon avatar Jul 25 '22 18:07 marcustyphoon

Closing this because whether or not the feature it's supposed to hide still exits, iirc the code doesn't work.

marcustyphoon avatar Feb 13 '23 18:02 marcustyphoon