Skip to content

🐞 Sidebar moved to far left #177

Closed

Description

@0xWDG

Describe the bug
The sidebar is moved out of view.

To Reproduce
Steps to reproduce the behavior:

  1. Resize your window to (e.g. 800px (wide) 600px (height)
  2. Open a project
  3. Open a file in the editor

Screenshots
27353C1B-4402-4E0E-9478-D72596BED07B

Desktop (please complete the following information):

  • OS: 12.3 (21E230)
  • Xcode: Version 13.3 (13E113)

Additional context
None.

Activity

  1. added a commit that references this issue on Mar 22, 2022
    c453081
  2. self-assigned this
    on Mar 22, 2022
  3. highjeans commented on Mar 22, 2022

    @highjeans
    Contributor

    We might want to have the sidebar locked to a certain size.

  4. thecoolwinter commented on Mar 23, 2022

    @thecoolwinter
    Collaborator

    Looks like it might have something to do with window sizing.

    Screen.Recording.2022-03-22.at.6.58.13.PM.mov
  5. lukepistrol commented on Mar 23, 2022

    @lukepistrol
    Member

    This has to do with the branch picker in the status bar which displays an error message when no git repo is available. Fixed in #193

  6. thecoolwinter commented on Mar 23, 2022

    @thecoolwinter
    Collaborator

    I had just come to the same conclusion! I was just about to make a pull request to fix it. Perfect!

  7. thecoolwinter commented on Mar 23, 2022

    @thecoolwinter
    Collaborator

    Actually, don't close this quite yet. The problem is being caused by the .fixedSize() modifier in StatusBarBranchPicker. Since that has a fixed size, SwiftUI can't compress it when it needs to shrink the window. Just in case someone has a long branch name that should be changed to .fixedSize(horizontal: false, vertical: true). I'll make a PR for that now.

  8. RayZhao1998 commented on Mar 23, 2022

    @RayZhao1998
    Collaborator

    @thecoolwinter I don't think changing from .fixedSize() to .fixedSize(horizontal:false, vertical: true) is a right solution. It'll make the StatusBarBranchPicker take up extra space.

    截屏2022-03-23 下午9 32 22

  9. RayZhao1998 commented on Mar 23, 2022

    @RayZhao1998
    Collaborator

    open another issue #209

  10. thecoolwinter commented on Mar 23, 2022

    @thecoolwinter
    Collaborator

    I agree but we could also add a Spacer with min width 0 and it would fix this problem. Having the false value for the horizontal makes it so we can know that SwiftUI will shrink this specific component rather than another one or not shrink it at all.

  11. changed the title [-][BUG] - Sidebar moved to far left[/-] [+]🐞 Sidebar moved to far left[/+] on Mar 31, 2022
  12. moved this from 🆕 New to 🏁 Complete in CodeEdit Projecton Feb 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions