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

Display file name and file size in status bar upon hovering over a treemap tile #30

Closed
Smile4ever opened this Issue Oct 23, 2016 · 3 comments

Comments

Projects
None yet
2 participants
@Smile4ever
Contributor

Smile4ever commented Oct 23, 2016

It would be nice to see the file name and size in the status bar if you hover above a file in the colored view, just like WinDirStat does. #28 was the wrong feature request in fact :)

@shundhammer

This comment has been minimized.

Show comment
Hide comment
@shundhammer

shundhammer Oct 23, 2016

Owner

I admit I never used WinDirStat (I don't use Windows all that much). How does it handle an item in the tree view being selected at the same time? That behaviour would be an inconsistency if an item in the tree view or in the treemap is selected; it would be unclear what item the status bar refers to.

Anyway, using the status bar rather than a tool tip is a completely different matter, and that sounds much more intuitive to me - except for that issue of hover vs. currently selected item (and there is almost always something selected in the tree view and thus in the treemap).

Owner

shundhammer commented Oct 23, 2016

I admit I never used WinDirStat (I don't use Windows all that much). How does it handle an item in the tree view being selected at the same time? That behaviour would be an inconsistency if an item in the tree view or in the treemap is selected; it would be unclear what item the status bar refers to.

Anyway, using the status bar rather than a tool tip is a completely different matter, and that sounds much more intuitive to me - except for that issue of hover vs. currently selected item (and there is almost always something selected in the tree view and thus in the treemap).

@Smile4ever

This comment has been minimized.

Show comment
Hide comment
@Smile4ever

Smile4ever Oct 23, 2016

Contributor

WinDirStat can run very well on Wine, so maybe you should try it :)

WinDirStat shows the selected file in the status bar until you hover another file in the color map. If you go outside the color map area, it displays the selected file again.

I think this is actually more intuitive. This gives you the chance to inspect file sizes and file names without actually changing the tree view, which may be undesirable.

Please note that WinDirStat does not show the file size upon hovering, this is where QDirStat could be made better than WinDirStat :)

If you think this is inconsistent behaviour, you can introduce a preference to make both behaviours possible :)

Thanks a lot for considering this.

Contributor

Smile4ever commented Oct 23, 2016

WinDirStat can run very well on Wine, so maybe you should try it :)

WinDirStat shows the selected file in the status bar until you hover another file in the color map. If you go outside the color map area, it displays the selected file again.

I think this is actually more intuitive. This gives you the chance to inspect file sizes and file names without actually changing the tree view, which may be undesirable.

Please note that WinDirStat does not show the file size upon hovering, this is where QDirStat could be made better than WinDirStat :)

If you think this is inconsistent behaviour, you can introduce a preference to make both behaviours possible :)

Thanks a lot for considering this.

@shundhammer shundhammer changed the title from Display file name and file size upon hovering to Display file name and file size in status bar upon hovering over a treemap tile Oct 23, 2016

@shundhammer

This comment has been minimized.

Show comment
Hide comment
@shundhammer

shundhammer Feb 27, 2017

Owner

Implemented.

I am not yet sure if this isn't a big too busily flickering the way it is now; maybe it should get a timeout.

Owner

shundhammer commented Feb 27, 2017

Implemented.

I am not yet sure if this isn't a big too busily flickering the way it is now; maybe it should get a timeout.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment