KSnapshot: add setPixmap DCOP call #77
Merged
blu.256
merged 1 commits from feat/ksnapshot-dcop-set-pixmap into master 5 months ago
MicheleC
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'feat/ksnapshot-dcop-set-pixmap'
Deleting a branch is permanent. It CANNOT be undone. Continue?
This PR adds a new DCOP call to the KSnapshot interface,
setPixmap(TQPixmap &newImage).This can be useful to integrate KSnapshot with applications that can take their own screenshots better.
For example, a media player can use a single frame from a video instead of a lower-quality real screen grab. (I plan to implement this feature with the Kaffeine MPV backend). So the media player would launch an instance of KSnapshot and pass the frame via DCOP to the newly-launched instance. The user can then do with the shot whatever they would do with an ordinary screenshot, which is more convenient to the end user than just showing a save dialog, or -even worse- taking the liberty of saving the file wherever the player decides.
Nice idea Philippe, it works very well!
Small suggestion for improvement: the function argument should be
const TQPixmap &newImage, since the image passed to the function is not modified.20d9b1e7e2toea96ef12945 months agoDone.
ea96ef1294tod7130aacf95 months agoLooks good, thanks.
d7130aacf9into master 5 months agoReviewers
d7130aacf9.