Skip to content
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

🐞 Drag-collapsing Inspector sidebar in full-screen breaks layout #1224

Closed
Moshyfawn opened this issue Apr 9, 2023 · 2 comments · Fixed by #1776
Closed

🐞 Drag-collapsing Inspector sidebar in full-screen breaks layout #1224

Moshyfawn opened this issue Apr 9, 2023 · 2 comments · Fixed by #1776
Labels
bug Something isn't working

Comments

@Moshyfawn
Copy link

Description

Dragging the Inspectors sidebar to collapse causes the main layout to break: an empty space appears in place of the collapsed sidebar. Trying to open the sidebar again causes a further layout shift, inducing the sidebar anchor placement.

To Reproduce

  1. Open CodeEdit
  2. Click on "Hide or show the Inspectors" sidebar toggle icon in the top right corner
  3. Drag the sidebar by the left edge (to the right) to collapse
  4. See the main file content area now has an empty black area on the right

Expected Behavior

Dragging the Inspectors sidebar to collapse - collapses the sidebar without breaking the main layout

Version Information

CodeEdit: 0.0.3-alpha (32)
macOS: 13.3.1
Xcode: 14.3 (14E222b)

Additional Context

No response

Screenshots

codeedit-inspectors-sidebar-drag-collapse-issue-screencast.mp4
@Moshyfawn Moshyfawn added the bug Something isn't working label Apr 9, 2023
@austincondiff austincondiff changed the title 🐞 Drag-collapsing Inspectors sidebar breaks layout 🐞 Drag-collapsing Inspector sidebar in full-screen breaks layout Jun 11, 2023
@austincondiff
Copy link
Collaborator

This also removes width from the window when drag-collapsing not in full-screen.

@armartinez
Copy link
Contributor

This issue is caused by the current STTextView implementation, so hopefully it gets resolved once https://github.com/CodeEditApp/CodeEditTextView/pull/211 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🏁 Complete
Development

Successfully merging a pull request may close this issue.

3 participants