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

[Windows] Disks widget shows \\?\Volume{drive ID} instead of drive names #1075

Closed
1 task done
Scripter17 opened this issue Mar 22, 2023 · 5 comments · Fixed by #1064
Closed
1 task done

[Windows] Disks widget shows \\?\Volume{drive ID} instead of drive names #1075

Scripter17 opened this issue Mar 22, 2023 · 5 comments · Fixed by #1064
Labels
feature Requests for a new feature.

Comments

@Scripter17
Copy link

Checklist

Describe the issue

Basically just what's in the title

What operating system and version are you using?

Windows 10 22H2

What architecture are you using?

x86

What terminal(s) are you running bottom on?

Regular command prompt (cmd.exe)

What version of bottom are you running?

0.8.0

How did you install bottom?

Through cargo

How can we reproduce this?

  1. Open bottom on windows
  2. Look at the disks widget

Additional information

No response

@Scripter17 Scripter17 added the bug Something isn't working the way that is expected. label Mar 22, 2023
@Scripter17 Scripter17 changed the title [Windows] Disks widget shows \\?\{drive ID} instead of drive names [Windows] Disks widget shows \\?\Volume{drive ID} instead of drive names Mar 22, 2023
@ClementTsang
Copy link
Owner

ClementTsang commented Mar 22, 2023

That's not really a bug, it's currently intentional behaviour, though I can look to changing it if possible. Windows reports weird-looking drive names with the library I was using.

@Scripter17
Copy link
Author

Scripter17 commented Mar 22, 2023

...So the intended behaviour is to... sort alphabetically by drive ID instead of what the drive is named...

I can understand this being a "I can't be bothered to fix this" problem because it's the exact kind of obvious feature windows would make hard (at least without spawning a wmic process each time you want to get the disk info) but this does not at all look like intended behaviour.

I'm really am sorry if I'm coming off as rude but I am just baffled

@ClementTsang
Copy link
Owner

ClementTsang commented Mar 22, 2023

The original and intended behaviour was sorting by whatever was in the disk name column, yes, especially prior to being able to sort by any column. And the returned name for a drive on Windows has always been this weird name.

I'm fine with changing it to be nicer (if reasonably possible) but it isn't exactly a bug how it currently is, hence why I closed it. If you would prefer, I can reopen this as a feature request.

@ClementTsang ClementTsang reopened this Mar 22, 2023
@ClementTsang ClementTsang added feature Requests for a new feature. and removed bug Something isn't working the way that is expected. labels Mar 22, 2023
@ClementTsang
Copy link
Owner

ClementTsang commented Mar 22, 2023

Looks like #1064 will end up changing this anyway, though currently the behaviour for some drives is that it'll just return a blank name now.

@ClementTsang
Copy link
Owner

New behaviour is that it'll return whatever sysinfo returns, which generally seems to match whatever it is named in File Explorer. For drives with no name, for now, I instead return No Name, as I'm not sure if leaving it blank would cause confusion.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Requests for a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants