After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 622473 - Adding up the totals for all directories does not add up
Adding up the totals for all directories does not add up
Status: RESOLVED FIXED
Product: baobab
Classification: Core
Component: general
git master
Other Linux
: Normal normal
: ---
Assigned To: Baobab Maintainers
Baobab Maintainers
Depends on:
Blocks:
 
 
Reported: 2010-06-23 08:34 UTC by aapgorilla
Modified: 2012-10-13 15:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Show scan errors in the treeview using colors (10.42 KB, patch)
2012-09-29 15:23 UTC, Stefano Facchini
none Details | Review
Show scan errors in the treeview using colors (8.10 KB, patch)
2012-09-29 15:26 UTC, Stefano Facchini
needs-work Details | Review
Show scan errors in the treeview using colors (8.70 KB, patch)
2012-09-29 16:34 UTC, Stefano Facchini
committed Details | Review
Add theming for baobab error states (845 bytes, patch)
2012-09-29 16:35 UTC, Stefano Facchini
committed Details | Review

Description aapgorilla 2010-06-23 08:34:24 UTC
See the attached screenshot, baobab tells me 6.6GB is used on the disk but when you look at all the directories it only adds up to around 4.8GB. It turned out the remaining 1.8GB was located in another user's home but I found the totals not adding up very confusing 

http://img268.imageshack.us/f/screenshotfb.png/
Comment 1 Stefano Facchini 2012-09-29 15:23:28 UTC
Created attachment 225383 [details] [review]
Show scan errors in the treeview using colors
Comment 2 Stefano Facchini 2012-09-29 15:24:02 UTC
This should help to avoid the confusion
Comment 3 Stefano Facchini 2012-09-29 15:26:12 UTC
Created attachment 225384 [details] [review]
Show scan errors in the treeview using colors

Ops.. committed to much
Comment 4 Paolo Borelli 2012-09-29 15:31:34 UTC
Review of attachment 225384 [details] [review]:

::: src/baobab-cellrenderers.vala
@@ -26,1 +28,16 @@
-                text = "%.1f %%".printf (value);
+                text = (state != Scanner.State.ERROR ? "%.1f %%".printf (value) : "");
+            }
+        }
... 13 more ...

instead of hardcoding orange, push an error class on the style context and then remove it after rendering and then set the error color in css
Comment 5 Stefano Facchini 2012-09-29 16:34:30 UTC
Created attachment 225385 [details] [review]
Show scan errors in the treeview using colors

I couln't figure out how to use CSS to make the font bold... but maybe we don't even need to make it 
bold.
Comment 6 Stefano Facchini 2012-09-29 16:35:22 UTC
Created attachment 225386 [details] [review]
Add theming for baobab error states

Corresponding patch for gnome-themes-standard
Comment 7 Paolo Borelli 2012-09-30 10:28:53 UTC
Both patches look good!
Comment 8 Stefano Facchini 2012-10-13 15:07:23 UTC
Comment on attachment 225385 [details] [review]
Show scan errors in the treeview using colors

Attachment 225385 [details] pushed as 5374f02 - Show scan errors in the treeview using colors