Skip to content

Remove remix-tree-button #8652

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
mxmou merged 6 commits into from
Oct 26, 2025
Merged

Remove remix-tree-button #8652

mxmou merged 6 commits into from
Oct 26, 2025
+0 −131

Conversation

@ampelectrecuted
Copy link
Contributor

@ampelectrecuted ampelectrecuted commented Oct 17, 2025 •
edited
Loading

Resolves #8651

Changes

Removed remix-tree-button.

Reason for changes

Scratch now redirects remixtree to the normal remixes page, so this extension is now useless. :(

Tests

Tested on Firefox 144

@ampelectrecuted
Remove remix-tree-button
0e951ce 
Scratch now redirects remixtree to the normal remixes page.
@ampelectrecuted
Update addons.json
887ff6b
@mxmou mxmou added the scope: addon Related to one or multiple addons label Oct 17, 2025
mxmou
mxmou requested changes Oct 17, 2025
View reviewed changes
Copy link
Member

@mxmou mxmou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few other things need to be removed as well:

  • addons-l10n/en/remix-tree-button.json
  • addons/scratchr2/remixtree.css and the reference to it in addons/scratchr2/addon.json
  • the reference to .remixtree-button in addons/scratch3to2/project.css

@tallypaws
Copy link
Contributor

tallypaws commented Oct 17, 2025

why did they remove remix tree :c

@CubesterYT
Copy link

CubesterYT commented Oct 17, 2025

death to remix tree

@ampelectrecuted ampelectrecuted requested a review from mxmou October 17, 2025 20:56
@DNin01
Copy link
Member

DNin01 commented Oct 18, 2025

why did they remove remix tree :c

I don't know, but maybe we could make our own.

@mxmou
Copy link
Member

mxmou commented Oct 18, 2025

@ampelectrecuted You still need to delete the addons/scratchr2/remixtree.css file. There's also a line in addons/scratch3to2/project.css that mentions .remixtree-button - it should be removed.

I don't know, but maybe we could make our own.

The endpoint that returned the raw data for the remix tree (/projects/<id>/remixtree/bare/) has also been removed. This means we'd need to use the API, which unfortunately can't sort projects by loves like the remix tree did.

@medlans
Copy link

medlans commented Oct 20, 2025 •
edited
Loading

why did they remove remix tree :c

I don't know, but maybe we could make our own.

They've now rejected the suggestion (the rejection post didn't really include a reason). I suspect it was due to inappropriate projects showing on the remixtree only (that's what seemed to be happening recently), but I honestly don't see why the problem couldn't have been fixed in this case.

They filtered remixtree links by filtering "/remixtree" and made them redirect to the remixes page too

@mxmou
Copy link
Member

mxmou commented Oct 21, 2025

The endpoint that returned the raw data for the remix tree (/projects/<id>/remixtree/bare/) has also been removed.

It seems to be available again - but I don't think we can rely on it continuing to work.

@DNin01
Copy link
Member

DNin01 commented Oct 21, 2025

Well, now /remixtree does not redirect but displays the same content as /remixes.

@ampelectrecuted
Copy link
Contributor Author

ampelectrecuted commented Oct 26, 2025

i've removed all those files, @mxmou can you re-review

@mxmou mxmou added the status: needs review PR needs 1 more approval label Oct 26, 2025
@ampelectrecuted
Copy link
Contributor Author

ampelectrecuted commented Oct 26, 2025

wow this is a battlefield we have here

@ampelectrecuted
Copy link
Contributor Author

ampelectrecuted commented Oct 26, 2025

reminds me of 2024-10-28 (except that was much bigger than this) lol

Samq64
Samq64 approved these changes Oct 26, 2025
View reviewed changes
Copy link
Member

@Samq64 Samq64 left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that there are only up 4 buttons shown at a time the changes from #8466 are no longer necessary unless another addon adds a button there in the future.

@Samq64 Samq64 removed the status: needs review PR needs 1 more approval label Oct 26, 2025
@mxmou mxmou merged commit 1815ca3 into ScratchAddons:master Oct 26, 2025
3 checks passed
@Samq64 Samq64 mentioned this pull request Dec 29, 2025
Closed
@medlans
Copy link

medlans commented Jan 11, 2026

why did they remove remix tree :c

I don't know, but maybe we could make our own.

People have made recreations of the remixtree, so maybe one of those could be used with the permission from the creator?

https://retree.quuq.dev/ (I asked the creator of this one if they could use it for this extension)

Well, now /remixtree does not redirect but displays the same content as /remixes.

Yeah that was because there was a bug where the user called that would redirect to remixes LOL

@redspacecat
Copy link

redspacecat commented Jan 11, 2026

https://retree.quuq.dev/ (I asked the creator of this one if they could use it for this extension)

So what should I do?
It looks like this PR has already been merged. Should I make something that re-adds the button?

@medlans
Copy link

medlans commented Jan 11, 2026

https://retree.quuq.dev/ (I asked the creator of this one if they could use it for this extension)

So what should I do? It looks like this PR has already been merged. Should I make something that re-adds the button?

Yea lol

@redspacecat
Copy link

redspacecat commented Jan 11, 2026

https://retree.quuq.dev/ (I asked the creator of this one if they could use it for this extension)

So what should I do? It looks like this PR has already been merged. Should I make something that re-adds the button?

Yea lol

Uh. How? I'm sort of new to this lol.

@redspacecat redspacecat mentioned this pull request Jan 16, 2026
Open
@redspacecat redspacecat mentioned this pull request Jan 17, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mxmou mxmou mxmou approved these changes

@Samq64 Samq64 Samq64 approved these changes

Assignees

No one assigned

Labels

scope: addon Related to one or multiple addons

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Remove remix-tree-button

8 participants

@ampelectrecuted @tallypaws @CubesterYT @DNin01 @mxmou @medlans @redspacecat @Samq64