zed-industries / zed

Code at the speed of thought – Zed is a high-performance, multiplayer code editor from the creators of Atom and Tree-sitter.
https://zed.dev
Other
49.57k stars 3.04k forks source link

Split divider missing when left pane is active #6844

Closed diocletiann closed 2 weeks ago

diocletiann commented 9 months ago

Check for existing issues

Describe the bug / provide steps to reproduce it

It shows up for a moment with the scroll bar but then disappears edit: that was the scrollbar area border, the split divider doesn't show up. When the right pane is active it looks normal.

Environment

Zed: v0.120.3 (Zed Preview) OS: macOS 14.3.0 Memory: 32 GiB Architecture: aarch64

If applicable, add mockups / screenshots to help explain present your vision of the feature

image image

If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.

If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.

No response

maxdeviant commented 9 months ago

I haven't been able to reproduce this yet in Zed Preview v0.120.3:

Screenshot 2024-01-28 at 11 17 41 AM
diocletiann commented 9 months ago

I haven't been able to reproduce this yet in Zed Preview v0.120.3:

Ah, I see why. It happens when using "active_pane_magnification".

maxdeviant commented 9 months ago

I haven't been able to reproduce this yet in Zed Preview v0.120.3:

Ah, I see why. It happens when using "active_pane_magnification".

What do you have your active_pane_magnification value set at?

diocletiann commented 9 months ago

I haven't been able to reproduce this yet in Zed Preview v0.120.3:

Ah, I see why. It happens when using "active_pane_magnification".

What do you have your active_pane_magnification value set at?

1.8

tugkanpilka commented 7 months ago

I've tracked down the bug and found that it is all about the code block:

for (ix, child_layout) in layout.children.iter_mut().enumerate() {
    if active_pane_magnification.is_none() {
        if ix < len - 1 {
            child_layout.handle =
                Some(Self::layout_handle(self.axis, child_layout.bounds, cx));
        }
    }
}

When i delete the if condition the bug disappeared.

for (ix, child_layout) in layout.children.iter_mut().enumerate() {
    if ix < len - 1 {
        child_layout.handle =
            Some(Self::layout_handle(self.axis, child_layout.bounds, cx));
    }
}

But i did not understand why there is a condition for if active_pane_magnification.is_none() and is it have any side effects when deleted. Any thoughts?

github-actions[bot] commented 1 month ago

Hi there! 👋 We're working to clean up our issue tracker by closing older issues that might not be relevant anymore. Are you able to reproduce this issue in the latest version of Zed? If so, please let us know by commenting on this issue and we will keep it open; otherwise, we'll close it in 7 days. Feel free to open a new issue if you're seeing this message after the issue has been closed. Thanks for your help!