Skip to content

Show file size depend on the OS#1810

Closed
Gao-Jun wants to merge 1 commit intoinstructure:masterfrom
Fudan-University:more_friendly_bytes
Closed

Show file size depend on the OS#1810
Gao-Jun wants to merge 1 commit intoinstructure:masterfrom
Fudan-University:more_friendly_bytes

Conversation

@Gao-Jun
Copy link
Contributor

@Gao-Jun Gao-Jun commented Feb 8, 2021

One day, a Windows user asked, why I got a smaller file downloaded (806K) compared with the size shown in 'Files'(825K) ? Is the file downloaded completely?

In Mac OS, 1 KB = 1000 Bytes. In Windows and Linux, 1 MB = 1024 Bytes

This patch will display the file size depend on the OS.

In detail, 1 MB = 1000 Bytes for Mac user, and 1MiB = 1024 Bytes for others

1 MB = 1000 Bytes for Mac user, and 1MiB = 1024 Bytes for others
@dustin-cowles dustin-cowles self-assigned this Aug 30, 2022
@dustin-cowles
Copy link
Contributor

Hi @Gao-Jun Sorry this has not been addressed in some time. We are working on getting to things much quicker now. This file has been moved and thus this PR is no longer valid. If you wanted to update it I could work on getting it reviewed.

@Gao-Jun
Copy link
Contributor Author

Gao-Jun commented Jun 12, 2024

Hi @dustin-cowles An updated version of this PR is here #2367

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants