Skip to content

Localize the storage units (MiB, GiB, etc.), in particular for the StorageUsageStatus component#50

Merged
Banane9 merged 1 commit into
masterfrom
feature/localizable-storage-units
May 21, 2025
Merged

Localize the storage units (MiB, GiB, etc.), in particular for the StorageUsageStatus component#50
Banane9 merged 1 commit into
masterfrom
feature/localizable-storage-units

Conversation

@Banane9

@Banane9 Banane9 commented May 20, 2025

Copy link
Copy Markdown
Contributor

@Banane9 Banane9 requested a review from Nytra May 20, 2025 21:41
@Nytra

Nytra commented May 20, 2025

Copy link
Copy Markdown
Contributor

Why remove other names from locale authors?

@Banane9

Banane9 commented May 20, 2025

Copy link
Copy Markdown
Contributor Author

Gah, dammit - I copied the en.json and then accidentally edited that when I wanted to make the other one. Missed undoing that.

@Nytra Nytra left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good (aside from the locale thing)

@Banane9 Banane9 force-pushed the feature/localizable-storage-units branch from e1b5952 to 6b93089 Compare May 21, 2025 10:18
@Banane9 Banane9 merged commit b0bf334 into master May 21, 2025
2 checks passed
@Banane9 Banane9 deleted the feature/localizable-storage-units branch May 21, 2025 10:19
Comment on lines +5 to +10
"CommunityBugFixCollection.StorageUnits.B": "t",
"CommunityBugFixCollection.StorageUnits.kB": "Kit",
"CommunityBugFixCollection.StorageUnits.MB": "Mit",
"CommunityBugFixCollection.StorageUnits.GB": "Git",
"CommunityBugFixCollection.StorageUnits.TB": "Tit",
"CommunityBugFixCollection.StorageUnits.PB": "Pit"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wtf are these? Never heard anyone use anything like these …I guess the saving grace is that this monkey can be disabled…

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