<table><tr><td style="">ahmadosama 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/D10932">View Revision</a></tr></table><br /><div><div><p>Regarding the undo/redo stacks, the undo/redo command will not undo the whole reset, it will undo widgets somehow randomly.<br />
RadioButtons in the same page will be in the same undo command, as for TextLineEdit each single widget will be added separately to the undo/redo stack.</p>

<p>So triggering the reset action will result in adding several commands to the undo stack not just one, I am currently thinking of how to make the whole undo in a single command in the stack.</p>

<p>Also, the "Reset Forms" and "Show Forms/Hide Forms" buttons are stacked beside each other, the user can easily mistake by clicking the "Reset" button instead of the "Show Forms" Button.<br />
I am currently thinking about fixing this too.</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/D10932">https://phabricator.kde.org/D10932</a></div></div><br /><div><strong>To: </strong>ahmadosama, Okular, aacid<br /><strong>Cc: </strong>ngraham, aacid, Okular, michaelweghorn<br /></div>