loh.tar added a comment.
I suggest: - Add a new action "Select Page" similar to "Select All" - Call first that new action/function, then the copy function, instead of coding explicit in mouse event - Shorten your comments, don't say what is obviously, explain things which may indistinct e.g."select the word which was clicked on" "select the entire page if click event is in empty portion" REPOSITORY R223 Okular REVISION DETAIL https://phabricator.kde.org/D18744 To: shubham, aacid, #vdg Cc: loh.tar, alexde, ngraham, okular-devel, tfella, darcyshen, aacid
