Hans5958
Hans5958
I got the same bug when excluding the custom instruments. It works when I added it.
Here's a solution: For these cases, `pull_request_target` should be used instead of `pull_request`.
This will require changes on the docs, specifically on [the Design page](https://scratchaddons.com/docs/reference/design).
For the record: This suggestion is highly anticipated. Hundreds of (duplicate) website feedback, Discord suggestions, Scratch comments have already been made. This suggestion is amplified that it is already implemented...
> > Yes this would not be visible to non SA users > > Sounds useless. I mean that's what you get for not having SA. Doesn't mean it is...
To be honest, resizing it up to the image borders wouldn't be big enough to my liking. I would still need to use CSS' `scale`. I think this is fine...
> Really? After cropping the whitespace from [icon-transparent.svg](https://github.com/ScratchAddons/ScratchAddons/images/icon-transparent.svg) with Inkscape on master it looks like this: Huh. I thought you were thinking to not scale it altogether, > It doesn't...
> The extension UI doesn't contain large text or images which makes the larger logo look unbalanced - it feels like the logo wants to be the most important thing...
> It doesn't fit with the current text size (right now the scratch S is about the same size as the S in settings). Again, IMO it shouldn't be the...
@mxmou There is an issue for that on #3396, so this is a duplicate.