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

Bug: checkboxes misbehave, a lot. #4846

Open
TheYarin opened this issue Sep 6, 2024 · 4 comments
Open

Bug: checkboxes misbehave, a lot. #4846

TheYarin opened this issue Sep 6, 2024 · 4 comments
Assignees
Labels
bug Something isn't working UI Any broken visuals like layouts, colors etc.

Comments

@TheYarin
Copy link

TheYarin commented Sep 6, 2024

Version

0.12.24

Operating System

Windows

Distribution Method

msi (Windows)

Describe the issue

A few checkboxes issues:

  1. They don't accurately represent the state of their children:
    1. deselecting a hunk from a fully selected file changes the checkmark to an empty box instead of a "partial" sign
    2. manually selecting all of the hunks in a file does not change its checkbox to checkmark. But the "all files" checkbox is checked? awkward
  2. There's an unintuitive grayed-out checkmark sign, no idea what it's supposed to represent
  3. when any files are changed, all the files are auto selected, which is REALLY annoying when I'm in the middle of organizing my commits and staging specific hunks
  4. If I manually uncheck a file and then click the "all files" checkbox, it no longer checks the file I unchecked. But the "all files" checkbox is still marked as "checked", despite some files not being checked.

How to reproduce

No response

Expected behavior

We can all imagine the intuitive behavior of a checkbox tree, right?

Relevant log output

No response

@TheYarin TheYarin added the bug Something isn't working label Sep 6, 2024
@Byron Byron added the UI Any broken visuals like layouts, colors etc. label Sep 7, 2024
@Byron
Copy link
Collaborator

Byron commented Sep 7, 2024

Thanks a lot for reporting! It sounds like the logic behind the checkboxes can indeed be improved, and I appreciate to have these bundled in a single issue.

Maybe @estib-vega finds this topic interesting as well.

@TheYarin
Copy link
Author

Amazing @estib-vega, I'll test the new version and report back

@TheYarin
Copy link
Author

I'm glad to report the commit-composing experience is significantly less unpredictable thanks to the checkboxes behaviours update.

However, I still see this checkmark from time to time and I don't know what to make of it:
image

@estib-vega
Copy link
Contributor

However, I still see this checkmark from time to time and I don't know what to make of it:

@TheYarin hey there, happy to hear that!

If you're referring to the little gray checkmark that would be its hover-state look.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working UI Any broken visuals like layouts, colors etc.
Projects
None yet
Development

No branches or pull requests

3 participants