Move project player addons to a top-level category - #3955
Conversation
|
@mxmou Do you like this? I'm not convinced. (asking you since you made the issue that suggested this) |
|
There's only one theme in the project player themes, so maybe it would be better to get rid of the project player themes category completely. |
|
Honestly I don't really like this. IMO the project player is part of the editor. |
It's also part of the website. |
|
Manifest: ScratchAddons/manifest-schema#56 |
|
In my opinion, I agree with this being a separate category. The player which can be found in 3 places (projects/*/, projects/*/editor, projects/*/embed). I don't think that to change something in an embed, you have to look under the editor section. And @mxmou argued that this could technically also be a website addon since you can find it there too. I think that when most people think of an "editor" addon, they think of scripts that can run on projects/*/editor. However, all of these scripts that run, run on "projects", which are all 3, project page, project editor, and project embed. This is why I think they should have their own category. |
|
I'm not convinced this should be merged, but I was more skeptical a few months ago. Some issues I see:
|
Clones and mouse-pos are editor addons. I don't think this is an issue - there aren't many popup addons either.
Why is that important? More subcategories can be added later if there are enough addons for the same part of the editor.
The category buttons are very big - reducing all of them to the size of "small" ones (Credits, Help Translate, Send Feedback) would add enough space for adding another button. The header could be slightly smaller too. |
|
Sorry, I forgot to reply.
I think I created the popup category because having "message count in extension icon" as a website addon was too weird. It did involve the Scratch website in a way, because the message count is taken from there, but in no way it modifies the Scratch website and it doesn't even link to it.
Okay, good point. We have tags such as
That could work. |
|
Am I missing any addon that could fit into this category? I previously said clones and mouse-pos are targeted for developing a project, not for watching projects, so those don't really fit. Also, please see what I say at the end about remove-curved-stage-border.
|
|
Done. The project player themes sub-category already existed, but I removed it. |
|
Added "pending" because the spacing of the category selector needs to be reduced first. |
|
This is still on the v1.36 milestone but Vue 3 is on v1.38. Is that the case or is this one also being pushed back to 1.38? I don't mind fixing the merge conflicts. |
|
Is this still planned for 1.37? |
|
Hmm, I'll be looking at this after the MV3 release. Sorry. |
|
The |
|
@Hans5958 For the manifest schema does it need to be a new version or do I edit the existing one? |
I'm pretty sure we should bump the schema to a new version. The last schema hasn't changed for a few releases. |
Resolves #3014
Changes
Moves project player addons to a new top-level category under the Scratch Editor Features category. Since other the other top-level category tags are short, the
projectPlayertag has been renamed to justplayer.Reason for changes
It's confusing for project player addons to be a subcategory of editor addons since those addons also affect the project page and embeds.
Tests
Tested on Edge 96.