simgunz created this revision.
simgunz added a reviewer: Okular.
Herald added a project: Okular.
Herald added a subscriber: okular-devel.
simgunz requested review of this revision.

REVISION SUMMARY
  Configuration:
  
  - Add push button to select custom stamp image
  - Check if loaded image is usable as stamp or throw error
  - Keep image proportions in previewer
  - Move previewer below the combobox to display larger preview
  
  Annotation tool:
  
  - Keep stamp image proportion in annotation preview (while left mouse button 
is down)
  - Adding the annotation with one-click (without holding the left mouse button 
and dragging) adds the stamp with original proportions
  
  TODO:
  
  [ ] Check if filters in file chooser make sense / propose better alternative
  [ ] Update doc

TEST PLAN
  From stamp annotation configuration dialog:
  
  - Click push button next to combo box opens a file selector
  - Selecting a corrupted image file should throw an error
  - Selecting a good image file shows the preview of the image
  - Select a horizontal image shows a large clear preview
  - Select a vertical image file shows a smaller preview without messing up the 
visual of the config dialog
  
  From page view, select the stamp annotation with horizontal image file (not 
squared):
  
  - Click and hold. The preview maintains proportions
  - Single click. The stamp image in the pdf maintains proportions and has the 
same size of the click and hold preview.
  - Add an annotation of the Okular custom stamps (internal SVG so treated 
slightly differently) do not create problems

REPOSITORY
  R223 Okular

BRANCH
  improve-stamp-annotation

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

AFFECTED FILES
  ui/annotationwidgets.cpp
  ui/annotationwidgets.h
  ui/guiutils.cpp
  ui/guiutils.h
  ui/pagepainter.cpp
  ui/pageviewannotator.cpp

To: simgunz, #okular
Cc: okular-devel, fbampaloukas, joaonetto, tfella, ngraham, darcyshen, aacid

Reply via email to