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

Change icons for storage devices #170

Open
1 task done
CiscoDeTours opened this issue Feb 27, 2024 · 2 comments
Open
1 task done

Change icons for storage devices #170

CiscoDeTours opened this issue Feb 27, 2024 · 2 comments
Labels
enhancement New feature or request

Comments

@CiscoDeTours
Copy link

CiscoDeTours commented Feb 27, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Is your feature request related to a problem? Please describe.

Drives

Version 1.3.0 displays an internal drive icon that doesn't match the representation of these kind of devices like the external one does.

Describe the solution you'd like

NEWdrives

  • Replace the internal drive icon with the current external one.
  • Use a generic usb icon for external drives.

Describe alternatives you've considered

None.

Additional context

Nothing to add.

@CiscoDeTours CiscoDeTours added the enhancement New feature or request label Feb 27, 2024
@nokyan
Copy link
Owner

nokyan commented Feb 28, 2024

Hi, thanks for the issue.
When deciding which icon to use for which drive, Resources only considers whether a drive is external (i. e. removable) as a last resort and tries to find other clues first.
In short, the algorithm for it works like that:

  1. Try to match the block device name to a device, e. g. sr0 is always going to be a CD/DVD/Blu-ray drive
  2. Is the drive "rotational"? Then it's an HDD, use the HDD icon (the icon currently used for your 8 GB drive)
  3. If not, is it removable? Then use the flash drive icon, otherwise it's probably an SSD, use the SSD icon (the icon currently used for your other drives)

Your 8 GB flash drive has the same issue as my 32 GB one, in that they're recognized by the kernel as "rotational", which Resources uses as a clue to assume they are HDDs, hence the wrong icon.
The icons for your 1 TB and 4 TB drives are used for (presumed internal) SSDs. They are working as intended (I'm assuming that your drives are SSDs, if not, tell me).

@CiscoDeTours
Copy link
Author

I understand that icon's selection works automatically thanks to your explanation. Although SSD assignment is correct, my suggestion aims to change its icon to an aesthetically better one like HDD which I confirm is wrongly assigned to external drives as you mention.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants