D15580: New annotation toolbar

[Simone Gaiarin]

simgunz added a comment.


  In D15580#605913 <https://phabricator.kde.org/D15580#605913>, @davidhurka wrote:
  
  > In D15580#605611 <https://phabricator.kde.org/D15580#605611>, @simgunz wrote:
  >
  > > aacid requested to write some autotest for the ToggleActionMenu before merging this. I'll merge master in this review and work on the autotests soon.
  >
  >
  > It’s okay to me that you write the autotests. If you wish, I could do that instead.
  >
  > I am new to autotests, so I will have to learn how to do that. I am also unsure about what should be tested. The ToggleActionMenu itself, or its behaviour in Okular?
  
  
  I'm more than happy if you do that :-)
  
  I am also not an expert on UI autotests, my suggestion is to have a look at the other Okular autotests and take inspiration from there. Probably it is good to test the component on its own (all the possible combination of parameters) and also the behavior in Okular.
  
  I'll try to move this to invent soon.

REPOSITORY
  R223 Okular

BRANCH
  new-annotation-toolbar_ToggleActionMenu

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

To: simgunz, #okular, #vdg, ngraham
Cc: kde-doc-english, andreashurka, yurchor, trickyricky26, simgunz, ltoscano, cfeck, aacid, davidhurka, knambiar, ngraham, tobiasdeiminger, okular-devel, johnzh, andisa, siddharthmanthan, maguirre, gennad, fbampaloukas, joaonetto, kezik, tfella, skadinna, darcyshen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/okular-devel/attachments/20200204/926aaa90/attachment.html>