Skip to content

Conversation

@filipchristiansen
Copy link
Contributor

What it does

  • Updates the file-size slider label in git_form.jinja and the formatSize helper in utils.js to show kB / MB instead of the misleading kb / mb.
  • No behaviour or API changes.

Why

  • Lower-case “b” denotes bits. Our UI currently claims kilobits/megabits while the backend logic actually works in bytes. This patch fixes that mismatch.

Discussion

  • We’re standardising on correct decimal prefixes for now. I’m still weighing a full switch to IEC binary prefixes (KiB/MiB) and have asked @cyclotruc for input.

Thanks

@filipchristiansen filipchristiansen merged commit 327958e into main Jun 22, 2025
18 checks passed
@filipchristiansen filipchristiansen deleted the fix/ui-decimal-prefixes branch June 22, 2025 07:56
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.

2 participants