Skip to content

full screen now opens in front of vscode #3741

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.

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

Merged
merged 3 commits into from
Feb 5, 2025
Merged

Conversation

HD787
Copy link
Contributor

@HD787 HD787 commented Jan 15, 2025

Description

full screen opens behind vscode making it so the initial perceived effect of the toggle fullscreen is just closing the sidebar, this fixes that, it will now open in front.

Checklist

  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screenshots

[ For visual changes, include screenshots. ]

Testing instructions

Copy link

netlify bot commented Jan 15, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit cb07f64
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/67a27c32e6f8760008d734e7

@fbricon
Copy link
Contributor

fbricon commented Jan 27, 2025

Could we stop calling it full screen and rename that feature "Detach window" or anything more accurate than "full screen"?

@HD787
Copy link
Contributor Author

HD787 commented Jan 27, 2025

@fbricon I'm not opposed but I did not name the feature, I would just make a PR or convince one of the maintainers.

@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 05:59 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:00 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 06:02 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj had a problem deploying to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Failure
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj temporarily deployed to tests-requiring-secrets February 1, 2025 09:46 — with GitHub Actions Inactive
@sestinj sestinj merged commit bdec7e6 into continuedev:main Feb 5, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants