<table><tr><td style="">aacid 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/D16285">View Revision</a></tr></table><br /><div><div><p>Ok, understood.</p>

<p>Personally i do not like this feature, the fact that i printed page 2 out of 400 of a document four months ago really means nothing regarding what i want to print now, in fact, most probably page 2 is exactly what i do not want to print.<br />
Also it is not a behaviour the rest of KDE Applications has so it creates confusion amonsgt users why Okular seems to remember some settings but the rest of applications that seemingly have "the same printing dialog" behave different.</p>

<p>Besides that, i think the function names/comments could be improved</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">/**
 * Sets the printing range of the current document
 */
void setPrintRange(int from, int to);</pre></div>

<p>doesn't really have to do anything with setting the printing rahge, all it does is save some value in an xml file.</p>

<p>So probably something like</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">/**
 * Sets the last page range used when printing the current document
 */
void setLastPrintRange(int from, int to);</pre></div>

<p>makes more sense.</p></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/D16285">https://phabricator.kde.org/D16285</a></div></div><br /><div><strong>To: </strong>ahmadosama, Okular<br /><strong>Cc: </strong>aacid, yurchor, broulik, ngraham, okular-devel<br /></div>