Skip to content

We can keep the stage the exact same size - #3147

Merged
apple502j merged 3 commits into
ScratchAddons:masterfrom
mxmou:we-can-keep-the-stage-the-exact-same-size
Jul 29, 2021
Merged

We can keep the stage the exact same size#3147
apple502j merged 3 commits into
ScratchAddons:masterfrom
mxmou:we-can-keep-the-stage-the-exact-same-size

Conversation

@mxmou

@mxmou mxmou commented Jul 28, 2021

Copy link
Copy Markdown
Member

See #2682 (comment)

Changes

Makes the TurboWarp stage the same size as Scratch's stage and removes the now unnecessary background and border from the iframe.

@mxmou
mxmou requested a review from WorldLanguages as a code owner July 28, 2021 20:06
@WorldLanguages

Copy link
Copy Markdown
Member

Negative margin, I knew, but I'm a CSS noob 😂

@WorldLanguages

Copy link
Copy Markdown
Member

I added the background/border to have some UI indication that this is not the vanilla Scratch player, do you think we should remove these entirely or try to remind the user in some other way?

Comment thread addons/turbowarp-player/style.css
@mxmou

mxmou commented Jul 28, 2021

Copy link
Copy Markdown
Member Author

I added the background/border to have some UI indication that this is not the vanilla Scratch player, do you think we should remove these entirely or try to remind the user in some other way?

I don't think reminding the user is necessarily - there isn't a way to always automatically replace the Scratch player with the TurboWarp one, so a button needs to be clicked for the replacement to happen anyway.

@WorldLanguages

Copy link
Copy Markdown
Member

Do I merge or will you add CSS comments?

@apple502j apple502j added priority: 3 Medium priority. Includes bugs and useful features scope: addon Related to one or multiple addons labels Jul 29, 2021
@apple502j
apple502j merged commit 4a49444 into ScratchAddons:master Jul 29, 2021
@mxmou
mxmou deleted the we-can-keep-the-stage-the-exact-same-size branch July 29, 2021 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: 3 Medium priority. Includes bugs and useful features scope: addon Related to one or multiple addons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants