Review Request 125136: Add an interface which allow plugin to show custom overlay icons

Olivier Goffart ogoffart at kde.org
Mon Oct 19 12:13:16 BST 2015


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

(Updated Okt. 19, 2015, 11:13 vorm.)


Status
------

This change has been discarded.


Review request for Dolphin.


Repository: dolphin


Description
-------

Add an interface which allow plugin to show custom overlay icons


My use case is an owncloud plugin.

I could not simply use KVersionControlPlugin because it is lacking some state (the shared state).
Also, that plugin does not let me update only one file at the time. (there is no signal to tell dolphin that only one icon has changed.)  KVersionControlPlugin::versionState is supposed to be blocking because it is called from a thread but then it calls it for all the files sequencially which is slow if there are many files.

Instead, this new KOverlayPlugin::getOverlays is supposed to return fast. What I do is that if I don't have the thng cached, i return nothing, but then i start the query, and then call the overlaysChanged changed signal.


I currently put this class in the version control library.  Should it go in KIOWidgets instead? (next to KAbstractFileItemActionPlugin)


Diffs
-----

  src/CMakeLists.txt e0f91de 
  src/kitemviews/kfileitemmodelrolesupdater.h 216b0a5 
  src/kitemviews/kfileitemmodelrolesupdater.cpp b03fd9c 
  src/views/versioncontrol/koverlayiconplugin.cpp PRE-CREATION 
  src/views/versioncontrol/koverlayiconplugin.desktop PRE-CREATION 
  src/views/versioncontrol/koverlayiconplugin.h PRE-CREATION 

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


Testing
-------


Thanks,

Olivier Goffart

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20151019/2e44436c/attachment.htm>


More information about the kfm-devel mailing list