+
Skip to content
This repository was archived by the owner on Aug 31, 2023. It is now read-only.

feat(playground): Use full height for right side panels #3072

Merged
merged 4 commits into from
Aug 17, 2022

Conversation

MichaReiser
Copy link
Contributor

@MichaReiser MichaReiser commented Aug 17, 2022

This PR improves our playground to make use of the full height for the right hand side panels rather than 70% (which may result in scroll bars on small screens)

Test Plan

Screenshot from 2022-08-17 10-07-17
Screenshot from 2022-08-17 10-07-27
Screenshot from 2022-08-17 10-07-35
Screenshot from 2022-08-17 10-07-45

@MichaReiser MichaReiser requested a review from a team August 17, 2022 07:06
@MichaReiser MichaReiser temporarily deployed to aws August 17, 2022 07:06 Inactive
This PR improves our playground to make use of the full height for the right hand side panels rather than 70% (which may result in scroll bars on small screens)
@MichaReiser MichaReiser force-pushed the feat/playground-heights branch from 963edfb to ffd9ace Compare August 17, 2022 07:08
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Aug 17, 2022

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 63e2d39
Status: ✅  Deploy successful!
Preview URL: https://42068f9d.tools-8rn.pages.dev
Branch Preview URL: https://feat-playground-heights.tools-8rn.pages.dev

View logs

@MichaReiser MichaReiser temporarily deployed to aws August 17, 2022 07:08 Inactive
@github-actions
Copy link

github-actions bot commented Aug 17, 2022

@ematipico
Copy link
Contributor

ematipico commented Aug 17, 2022

The issue now is that if we paste a long code, the scroll bar appears in a different way. Is it correct?

Now
Screenshot 2022-08-17 at 08 26 31

Before
Screenshot 2022-08-17 at 08 26 40

@MichaReiser MichaReiser temporarily deployed to aws August 17, 2022 08:06 Inactive
@MichaReiser MichaReiser temporarily deployed to aws August 17, 2022 08:08 Inactive
@MichaReiser
Copy link
Contributor Author

I noticed that too when debugging binary expression formatting. Should now be fixed.

@MichaReiser MichaReiser temporarily deployed to aws August 17, 2022 08:15 Inactive
@ematipico
Copy link
Contributor

I found a strange bug, not related to your PR. If I paste a big chunk of code, the prettier's output doesn't appear.

@MichaReiser
Copy link
Contributor Author

I found a strange bug, not related to your PR. If I paste a big chunk of code, the prettier's output doesn't appear.

Have you checked the console output for any errors?

@MichaReiser MichaReiser merged commit 1cd07a6 into main Aug 17, 2022
@MichaReiser MichaReiser deleted the feat/playground-heights branch August 17, 2022 11:01
@ematipico
Copy link
Contributor

I found a strange bug, not related to your PR. If I paste a big chunk of code, the prettier's output doesn't appear.

Have you checked the console output for any errors?

Yes, and no errors

@MichaReiser
Copy link
Contributor Author

You'll have to do some debugging then ;)

IWANABETHATGUY pushed a commit to IWANABETHATGUY/tools that referenced this pull request Aug 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载