Open Bug 1723365 Opened 3 years ago Updated 1 year ago

about:processes memory usage rounding seems to coarse

Categories

(Toolkit :: Performance Monitoring, enhancement, P3)

enhancement

Tracking

()

People

(Reporter: JanH, Unassigned)

Details

Attachments

(4 obsolete files)

Once process memory usage gets into the GB range, using only integers for the memory usage seems to be a little too coarse, at least in my opinion. Would it make sense to show at least one decimal place for the GB level?

This is caused by maximumFractionDigits:0 at https://searchfox.org/mozilla-central/rev/9bed2623831804ac086bbb80cb666e5c3b00416a/toolkit/locales/en-US/toolkit/about/aboutProcesses.ftl#167

And the unit selection and localization is done on the JavaScript side by https://searchfox.org/mozilla-central/rev/9bed2623831804ac086bbb80cb666e5c3b00416a/toolkit/components/aboutprocesses/content/aboutProcesses.js#899-938

I wonder if there would instead be a way to let fluent pick the unit and display a reasonable string containing a localized number + unit. Greg, do you know if that's possible?

Flags: needinfo?(gtatum)
Priority: -- → P3

I'm not sure what exactly would be a reasonable string for this case, but that API is backed by by Intl.NumberFormat: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat

The other option would be to handle the Intl.NumberFormat manually in JS and pass the result in as a variable.

Flags: needinfo?(gtatum)
Comment on attachment 9327669 [details]
firefox Release 90.0.33

[redacted - mhoye]
Comment on attachment 9327710 [details]
Open attached PDF switches to unrelated open Tab instead (Account Settings)

[redacted - mhoye]
The content of attachment 9327669 [details] has been deleted for the following reason:

Spam
The content of attachment 9327670 [details] has been deleted for the following reason:

Spam
The content of attachment 9327710 [details] has been deleted for the following reason:

Spam
The content of attachment 9327712 [details] has been deleted for the following reason:

Spam
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: