Summary: | Dolphin is not able to show 2 decimal places in file sizes | ||
---|---|---|---|
Product: | [Applications] dolphin | Reporter: | cameronpoe |
Component: | general | Assignee: | Dolphin Bug Assignee <dolphin-bugs-null> |
Status: | CONFIRMED --- | ||
Severity: | normal | CC: | bugseforuns, frank78ac, justin.zobel, mail, meven29 |
Priority: | NOR | ||
Version: | 4.12.3 | ||
Target Milestone: | --- | ||
Platform: | openSUSE | ||
OS: | Linux | ||
Latest Commit: | Version Fixed In: |
Description
cameronpoe
2015-04-10 09:53:05 UTC
Dolphin and Konqueror seem to ignore settings in System Settings -> Locale -> Country/Region & Language -> Numbers But why? Could someone fix it, please. Thanks for the bug report. Dolphin 4.12.3 uses this to determine the text which should be displayed in the "Size" column: text = KGlobal::locale()->formatByteSize(size); So it seems that the problem is in kdelibs. I hope that "klocale" is the right component. But John Layt said in bug 312646 : "It looks like Dolphin uses the default formatByteSize() method which defaults to 1 decimal place and the default algorithm described below, rather than newer api which allows you to set the required significant digits and units to use (http://api.kde.org/4.x-api/kdelibs-apidocs/kdecore/html/classKLocale.html#a566a3d868abfca050e1191b16e85ed73). Perhaps a quick initial fix will be for Dolphin to define 2 decimal places?" Thanks for the hint, I completely forgot about the old report. I'll reassign to Dolphin then, but I'm not really sure why the formatByteSize function which does not take a 'precision' argument just ignores the setting in System Settings. Forcing each application to retrieve the setting and then pass it to the formatByteSize overload which takes a 'precision' argument does not look like the most maintainable and future-proof solution to me :-/ I can still reproduce this bug with Dolphin 17.12.3. I'll set the status to CONFIRMED, but I fear that a good fix won't be very easy to implement. Can confirm still an issue with Dolphin on git master. Please correct me if I am mistaken, I believe this setting for decimal place for file size does not exist anymore, since Plasma 5 I would guess. I could not find a trace of it in code and although I remember using it, it is not in the format kcm. So I suggest to close this as "NOT A BUG". If people miss the setting they should open a different bug. |