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

Image Attachments section redesign #18204

Open
artiomello opened this issue Dec 11, 2024 · 0 comments
Open

Image Attachments section redesign #18204

artiomello opened this issue Dec 11, 2024 · 0 comments
Labels
status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application

Comments

@artiomello
Copy link

NetBox version

v4.1.6

Feature type

Change to existing functionality

Triage priority

N/A

Proposed functionality

I believe the Image Attachment section needs to be revised to have it reflect data a bit more concisely, and precisely at that.

  1. Image size
image

Size needs to be shown in MB, not in bytes. I'm almost 40 now and even I can't recall a single time in my life where image size in bytes would make any sense to me. Let alone now.


  1. Name and Image columns
image

I think only one of them actually needs to stay. If you give a name to an image during upload - it's renamed accordingly, and the new name displayed in both columns. If you don't give a name, then the file name is taken and is displayed only in the Image column.

You can't have a situation where these two properties would be different, rather (and weirdly enough) omitted in some cases. So, they're pretty much the same. Hence having both of them doesn't make much sense.


  1. Referencing image attachments in other objects and views

So, if we open any object that has image attachments like this one, that has two:
image

We see that it likely filters that same original image table with itself, and we get the same columns as in the original table:
image
Only here this information is completely redundant:

  • We already know its name and that it's a site. Showing these properties here would make sense only in case it would show images for the site's child objects (which it should definitely not).
  • The image/name pair is redundant, as described above.
  • The size in bytes looks more like a phone number than anything else.
  • All in all, this redundant information just adds more clutter to the object's page.

What really should be displayed here are:

  • Name (serving as a pop-up image window).
  • Size (in MB, obviously).
  • Creation timestamp.

Use case

Updating this table, its object properties and how they are presented in other views/objects would make certain netbox pages less cluttery and more concise, showing only that which has value. This is especially true when viewing some more data rich objects, such as sites and racks.

Less is more, as they say.

Database changes

No response

External dependencies

No response

@artiomello artiomello added status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application labels Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: needs triage This issue is awaiting triage by a maintainer type: feature Introduction of new functionality to the application
Projects
None yet
Development

No branches or pull requests

1 participant