ahmadosama added a comment.

  Regarding the undo/redo stacks, the undo/redo command will not undo the whole 
reset, it will undo widgets somehow randomly.
  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.
  
  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.
  
  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.
  I am currently thinking about fixing this too.

REPOSITORY
  R223 Okular

REVISION DETAIL
  https://phabricator.kde.org/D10932

To: ahmadosama, #okular, aacid
Cc: ngraham, aacid, #okular, michaelweghorn

Reply via email to