-
-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
Comments
hey, @JasonWeill, I'm looking to contribute, if it's available, can you assign this? |
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? |
^ https://jupyterlab.readthedocs.io/en/latest/developer/contributing.html#contribute As for the issue in question, some thoughts:
|
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. |
@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. |
@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 It does, but that was an add-on to Windows XP, not a core feature of Windows Explorer in that era. Still useful. |
On it, will get back on that shortly |
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 I have created a pull request #16749 |
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:
Shorten this column name to "Size" — there is no ambiguity here.
For comparison, "Size" is used as a column header in macOS's Finder:
Context
The text was updated successfully, but these errors were encountered: