Skip to content

🐞 Open Quickly Overlay - The vertical separator should be the same color as the horizontal separator #214

Closed

Description

@austincondiff

Describe the bug
The vertical separator in the open quickly overlay doesn't match the horizontal one.

Expected behavior
Change the vertical separator to match the horizontal one.

Activity

  1. changed the title [-][BUG] - Open Quickly Overlay - The vertical separator should be the same color as the horizontal separator[/-] [+]🐞 Open Quickly Overlay - The vertical separator should be the same color as the horizontal separator[/+] on Mar 31, 2022
  2. stale commented on May 30, 2022

    @stale

    This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

  3. thecoolwinter commented on Jun 5, 2022

    @thecoolwinter
    Collaborator

    I think this can be closed, these looks like the same color to me.

    Screen Shot 2022-06-04 at 8 04 58 PM

  4. removed
    wontfixThis will not be worked on
    on Jun 5, 2022
  5. nrudnyk commented on Jun 7, 2022

    @nrudnyk
    Contributor

    @thecoolwinter not really, looks that the problem still exist for the dark theme
    image

  6. nrudnyk commented on Jun 7, 2022

    @nrudnyk
    Contributor

    moreover, there's much more important problem here

    record.mov

    maybe for the simplicity there's no need in previewing files?
    @lukepistrol @wdg what do you think?

  7. 0xWDG commented on Jun 7, 2022

    @0xWDG
    Collaborator

    Xcode does only show a list iirc.
    for me it's ok to only show an list.

    @austincondiff what do you think about this, you are better with the design :)

  8. nrudnyk commented on Jun 7, 2022

    @nrudnyk
    Contributor

    while file preview is cool (mos likely it is, plus it's useful), there's some cons to it:

    • it's already a little laggy previewing those files (maybe will be better with tree-sitter)
    • there's color issues caused by split view vs divider (which could be omitted altogether)
    • resizing splitter is awkward
    • and the last one - default size won't fit all files and name will be cut

    just a few points for consideration. Sorry for bringing this up (maybe that's the wrong place)

  9. austincondiff commented on Jun 7, 2022

    @austincondiff
    CollaboratorAuthor

    I'd prefer it with the preview, but if we include it, it needs to be fast. I believe we can get there with the new editor view.

    It shouldn't be resizable and the vertical border color should match the input's bottom border.

  10. thecoolwinter commented on Jun 8, 2022

    @thecoolwinter
    Collaborator

    The preview is also loading the entire file. We could definitely speed it up by just reading the first few lines and displaying them for the preview. Unless ofc we want it to be scrollable.

  11. austincondiff commented on Jun 10, 2022

    @austincondiff
    CollaboratorAuthor

    It should be scrollable and should use the same editor view @lukepistrol is working on. This should speed up performance.

  12. austincondiff commented on Feb 17, 2023

    @austincondiff
    CollaboratorAuthor

    As in Xcode, there will be a toggle to turn on and off this preview at the top right of the overlay.

    Clicking this toggle, changes the symbols appearance and hides the preview as shown below.

    This is out of scope of this particular issue so I will go ahead and create another with the above adjustments, but the divider should be a simple separator and should not resize the split when dragged.

  13. moved this from 🆕 New to 📋 Todo in CodeEdit Projecton Feb 17, 2023
  14. moved this from 📋 Todo to 🏁 Complete in CodeEdit Projecton Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    UIThis is UI relatedbugSomething isn't working

    Type

    No type

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions