-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124589/
-----------------------------------------------------------

(Updated Aug. 2, 2015, 12:17 p.m.)


Review request for Plasma, Kai Uwe Broulik and Sebastian Kügler.


Changes
-------

Good catch with respect to the asynchronous calll. This is now implemented, see 
updated review.


Repository: kdeplasma-addons


Description
-------

The disk quota is usually used in enterprise installations where network shares 
are mounted locally. Typically, sysadmins want to avoid that users copy lots of 
data into their folders, and therefor set quotas (the quota limit has nothing 
to do with the physical size of a partition). Typically, once a user gets over 
the hard limit of the quota, the account is blocked and the user cannot login 
anymore. This happens from time to time, since the users are not really aware 
of the current quota limit and the already used disk space.

Here is where the "Disk Quota" plasmoid helps: It continusouly monitors the 
disk quota and warns the quota apprpriately.

A detailed description including screenshots can be found in this blog: 
http://kate-editor.org/?p=3591

(I had a KDE4 hack of this plasmoid running at university, and it proved very 
usable over the years, so it is probably a good idea to have it by default in 
plasma)

Issues:
- the panel icon is larger than the others (some wrong margin?)
- an icon for the metadata.desktop is missing (the shipped quota.svg file is 
not available here, it seems).
- the grid units probably need some more tuning


Diffs (updated)
-----

  applets/CMakeLists.txt c60c350 
  applets/diskquota/CMakeLists.txt PRE-CREATION 
  applets/diskquota/Messages.sh PRE-CREATION 
  applets/diskquota/icons/quota.svg PRE-CREATION 
  applets/diskquota/package/contents/config/main.xml PRE-CREATION 
  applets/diskquota/package/contents/ui/ListDelegateItem.qml PRE-CREATION 
  applets/diskquota/package/contents/ui/main.qml PRE-CREATION 
  applets/diskquota/package/metadata.desktop PRE-CREATION 
  applets/diskquota/plugin/DiskQuota.h PRE-CREATION 
  applets/diskquota/plugin/DiskQuota.cpp PRE-CREATION 
  applets/diskquota/plugin/QuotaItem.h PRE-CREATION 
  applets/diskquota/plugin/QuotaItem.cpp PRE-CREATION 
  applets/diskquota/plugin/QuotaListModel.h PRE-CREATION 
  applets/diskquota/plugin/QuotaListModel.cpp PRE-CREATION 
  applets/diskquota/plugin/plugin.h PRE-CREATION 
  applets/diskquota/plugin/plugin.cpp PRE-CREATION 
  applets/diskquota/plugin/qmldir PRE-CREATION 

Diff: https://git.reviewboard.kde.org/r/124589/diff/


Testing
-------

Tested combinations:
- no quota installed: A nice message is displayed telling the user that 'quota' 
is missing.
- quota installed, but no quota restrictions set: The applet says "No quota 
restrictions found"
- quota installed, quotas active: The applet continuously shows the data. The 
quota entries are in a QAbstractItemModel derived class, so inserting/removing 
quotas all works (tested).
- filelight installed: the item under mouse gets highlighted. If clicked, 
filelight starts with the correct location.


Thanks,

Dominik Haumann

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to