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

Rename "File Size" column in file browser to "Size" #16734

Open
JasonWeill opened this issue Aug 30, 2024 · 12 comments · May be fixed by #16749
Open

Rename "File Size" column in file browser to "Size" #16734

JasonWeill opened this issue Aug 30, 2024 · 12 comments · May be fixed by #16749

Comments

@JasonWeill
Copy link
Contributor

JasonWeill commented Aug 30, 2024

Description

It is possible to read file sizes when the file size column in the filebrowser is very narrow, so much so that the word "Size" isn't visible in the column header:

image

Shorten this column name to "Size" — there is no ambiguity here.

For comparison, "Size" is used as a column header in macOS's Finder:

image

Context

  • Operating System and version: macOS Sonoma 14.6.1
  • Browser and version: Firefox ESR 129.0.1
  • JupyterLab version: 4.3.0beta1
@JasonWeill JasonWeill added bug status:Needs Triage Applied to new issues that need triage good first issue and removed status:Needs Triage Applied to new issues that need triage labels Aug 30, 2024
@ChairBorn
Copy link

hey, @JasonWeill, I'm looking to contribute, if it's available, can you assign this?

@asli-nobita
Copy link

Hi @JasonWeill I'm completely new to open-source contributions and I think this issue is a good one for me to start with. If available, can I work on this?

@krassowski
Copy link
Member

^ https://jupyterlab.readthedocs.io/en/latest/developer/contributing.html#contribute

As for the issue in question, some thoughts:

  • the "File Size" column currently does not show the size of the directory; it is fine as it is not a "Directory Size" column, but if we rename it to just "Size" then I think it is a reasonable ask to open a follow-up issue for showing size of the directory (in number of items or space usage)
  • we could make it responsive as is the "Last modified"/"Modified" column but ideally we would generalise the logic used for "Last modified" column

@jtpio
Copy link
Member

jtpio commented Sep 4, 2024

we could make it responsive as is the "Last modified"/"Modified" column but ideally we would generalise the logic used for "Last modified" column

Having the same behavior as the "Last Modified / Modified" column sounds good. Also noting that for Notebook 7 "File Size" is not an issue since the file browser takes the full width, so it could stay that way when there is enough space.

@JasonWeill
Copy link
Contributor Author

@krassowski I've never seen a file browser that displays a "Directory Size" column; instead, the lack of value in the "Size" column is another cue that a particular row represents something other than a file. We should discuss displaying directories' sizes in another issue.

I agree that making the column title names responsive would be good to have.

@JasonWeill
Copy link
Contributor Author

@ChairBorn Thank you for your interest! I assigned this issue to you. Please leave a comment here if you have any questions or if you'd like us to review a pull request.

@krassowski
Copy link
Member

Does "Folder size" count?

image

@krassowski
Copy link
Member

We should discuss displaying directories' sizes in another issue.

#16743

@JasonWeill
Copy link
Contributor Author

@krassowski It does, but that was an add-on to Windows XP, not a core feature of Windows Explorer in that era. Still useful.

@ChairBorn
Copy link

On it, will get back on that shortly

@ChairBorn
Copy link

ChairBorn commented Sep 5, 2024

Hey, @JasonWeill, I changed the file browser column name from "File Size" to "Size" as defined on line 2811 of packages/filebrowser/src/listing.ts. Now, it is also defined in overlay of Browser Menu (packages/filebrowser-extension/src/index.ts) as well as commandList docs. I'm not sure which way you would like it. Let me know
Screenshot 2024-09-05 110521

I have created a pull request #16749

@ChairBorn
Copy link

Ok, so I almost finished writing a helper function to adjust the text according to the space available and then realized that I can't get the Modified to change to Last Modified... It only changes when the layout for that tab is extremely wide
LastModified
Modified
I'll fix the spacing for both of those.

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

Successfully merging a pull request may close this issue.

5 participants