<table><tr><td style="">loh.tar added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D18744">View Revision</a></tr></table><br /><div><div><p>I suggest:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">Add a new action "Select Page" similar to "Select All"</li>
<li class="remarkup-list-item">Call first that new action/function, then the copy function, instead of coding explicit in mouse event</li>
<li class="remarkup-list-item">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"</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R223 Okular</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18744">https://phabricator.kde.org/D18744</a></div></div><br /><div><strong>To: </strong>shubham, aacid, VDG<br /><strong>Cc: </strong>loh.tar, alexde, ngraham, okular-devel, tfella, darcyshen, aacid<br /></div>