simgunz (Simone Gaiarin)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Tuesday

  • Clear sailing ahead.

User Details

User Since
Mar 7 2016, 5:11 PM (184 w, 5 d)
Availability
Available

Recent Activity

Wed, Sep 18

simgunz added a comment to D15580: [WIP] New annotation toolbar.

Sorry for the long silence. I have been busy at work and then on vacation.

Wed, Sep 18, 4:45 PM · Okular

Aug 19 2019

simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Change typewriter and pin icons
Aug 19 2019, 6:38 AM · Okular

Aug 18 2019

simgunz added a comment to D15580: [WIP] New annotation toolbar.

Some thoughts:

  • Instead of using a horizontal toolbar below the main toolbar, instead I might experiment with a vertical toolbar like the one shown in Gwenview's View mode. We could make the toolbar live inside the Reviews tab, which already shows a list of all annotations (and then we should unify the terminology vis-a-vis "reviews" vs "annotations." REASON: cramming everything into a horizontal toolbar seems impossible for a feature this rich; using a vertical toolbar provides us enough horizontal space to show labels for everything, and enough vertical space to easily hold everything. Also we re-use an existing UI element that's currently rather bare.
Aug 18 2019, 6:33 AM · Okular
simgunz updated the summary of D15580: [WIP] New annotation toolbar.
Aug 18 2019, 6:28 AM · Okular

Aug 5 2019

simgunz added a comment to D15580: [WIP] New annotation toolbar.

For my own UI review, I must continue to push for putting the annotation tools on a second toolbar that appears below the main one when needed.

Aug 5 2019, 10:50 AM · Okular

Jul 28 2019

simgunz added a comment to D15580: [WIP] New annotation toolbar.

With the latest push all the features of the toolbar are implemented. There are some horrible things in the code, but I will clean and refactor it.

Jul 28 2019, 1:46 PM · Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Fix detach annotation when mouse mode unchecked
Jul 28 2019, 1:40 PM · Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jul 28 2019, 1:39 PM · VDG, Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Use ToggleActionMenu
  • Add stamp annotation
  • Fix swap straight line and arrow
  • Use Title style capitalization
  • Show real stamp symbols if squared
  • Better stamp icon
Jul 28 2019, 1:19 PM · Okular
simgunz added a comment to D21971: [DEMO] Enhance ToggleActionMenu with ImplicitDefaultAction mode..

I did test this today and it seems to be working correctly, at least in the use cases of Okular, i.e. for select annotation tools and for the annotation toolbar. Actually I am currently using in D15580 now (will push soon), given that ToolAction was not enough for what I needed ( I'll update my code accordingly if this revision is modified).

Jul 28 2019, 12:37 PM · Okular

Jul 25 2019

simgunz committed R223:f15e8568a533: General improvements to stamp annotation (authored by simgunz).
General improvements to stamp annotation
Jul 25 2019, 6:16 PM
simgunz closed T8074: Make custom stamp image support more discoverable as Resolved by committing R223:f15e8568a533: General improvements to stamp annotation.
Jul 25 2019, 6:16 PM · VDG, Okular
simgunz closed T8074: Make custom stamp image support more discoverable, a subtask of T8076: Fix design of annotation toolbar, as Resolved.
Jul 25 2019, 6:16 PM · VDG, Okular
simgunz closed D22064: General improvements to stamp annotation.
Jul 25 2019, 6:09 PM · Okular

Jul 24 2019

simgunz added a comment to D22064: General improvements to stamp annotation.

Final version of the new UI components:

Jul 24 2019, 7:54 PM · Okular
simgunz updated the diff for D22064: General improvements to stamp annotation.
  • Fix messages and i18nc
Jul 24 2019, 7:52 PM · Okular
simgunz added a comment to D22064: General improvements to stamp annotation.

Exactly what I was imagining!I might also add at the beginning something like: "Warning: feature is considered experimental."

Jul 24 2019, 4:29 PM · Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jul 24 2019, 4:04 PM · VDG, Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Add geometrical annotations to action collection
  • Move arrow before straight line
  • Allow setting action text
  • Set default shortcuts
  • Set favorite tools default shortcuts
  • Fix i18n argument
  • Auto-update conf file
  • Add semantic information to i18n
  • Refactor width an opacity
  • Remove space before percentage sign
  • Insert custom width and opacity in action list
Jul 24 2019, 3:57 PM · Okular
simgunz added a comment to D22064: General improvements to stamp annotation.

Jul 24 2019, 6:59 AM · Okular
simgunz updated the diff for D22064: General improvements to stamp annotation.
  • Add warning regarding broken stamp support in PDF
Jul 24 2019, 6:55 AM · Okular

Jul 22 2019

simgunz added a comment to T8076: Fix design of annotation toolbar.

@andreask Given that you gave your availabilty to work on the new Okular icons, I have opened this bug (408283) with a (messy) list of what icons are needed. I did not provide icon names yet.

Jul 22 2019, 6:23 PM · VDG, Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jul 22 2019, 6:14 PM · VDG, Okular
simgunz added a comment to D22064: General improvements to stamp annotation.

Small code improvements that can be ignored if you want.

Jul 22 2019, 6:02 PM · Okular
simgunz updated the diff for D22064: General improvements to stamp annotation.
  • Move to init list
  • Add translation context
  • Use new connect syntax
  • Add tooltip to custom stamp button
Jul 22 2019, 5:46 PM · Okular
simgunz added a comment to D15580: [WIP] New annotation toolbar.

Do you have to move this to gitlab anyway?

Jul 22 2019, 4:52 PM · Okular
simgunz updated the summary of D15580: [WIP] New annotation toolbar.
Jul 22 2019, 4:40 PM · Okular
simgunz added a comment to D15580: [WIP] New annotation toolbar.

@davidhurka Thanks for looking up the bug numbers

Jul 22 2019, 4:40 PM · Okular
simgunz updated the summary of D15580: [WIP] New annotation toolbar.
Jul 22 2019, 4:38 PM · Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.

Fix bad rebase

Jul 22 2019, 4:34 PM · Okular

Jul 14 2019

simgunz added a comment to D15580: [WIP] New annotation toolbar.

I messed up something... I see many files I have not modified that are marked as modified here. Probably because I changed the origin remote and my master was not in sync with upstream. I'll fix this with the next arc diff, now it does not let me arc diff again because it says that the diff is empty.

Jul 14 2019, 2:06 PM · Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jul 14 2019, 2:02 PM · VDG, Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Fix action deselected if favorite tool with the same type is activated
  • Fix geom shapes action disabled when okular reparses config
  • Disable text annotation when file does not support them
Jul 14 2019, 1:54 PM · Okular

Jul 5 2019

simgunz added a comment to T8076: Fix design of annotation toolbar.

How will you do what currently a double-click does? It should still be possible to do more than one annotation in one go.

Jul 5 2019, 7:20 AM · VDG, Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jul 5 2019, 7:06 AM · VDG, Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Add i18n
  • Move default value to declaration
  • Add tooltips
Jul 5 2019, 7:06 AM · Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jul 5 2019, 5:37 AM · VDG, Okular
simgunz updated subscribers of D15580: [WIP] New annotation toolbar.

I have managed to re-enable the action Tools > Review to show/hide the annotation toolbar.

Jul 5 2019, 5:35 AM · Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Clean parseTool
  • Initialize private members
  • Add action to toggle annotation toolbar, set default position below
Jul 5 2019, 5:05 AM · Okular

Jun 28 2019

simgunz retitled D15580: [WIP] New annotation toolbar from New annotation toolbar to [WIP] New annotation toolbar.
Jun 28 2019, 6:09 AM · Okular
simgunz updated subscribers of D22064: General improvements to stamp annotation.

@aacid Can you also have a look at this revision when you have the time?

Jun 28 2019, 6:08 AM · Okular

Jun 26 2019

simgunz added a comment to D21156: Fix browse mode not disabled after Ctrl is released.

Oh, I usually squash my changes into the original commit before uploading the change (i.e. via arc diff). I do not use an extra branch either.

Phew. Anyhow. Let me sum this all up with: Please make your life easier and rather start using KDE's GitLab instance: It's much easier to use since it shares the same workflow as all other major Git hosting platforms out there: https://invent.kde.org/kde/kdevelop

Phabricator is just horrible to adapt to and integrates badly with the usual Git workflow. For KDE development, Phabricator will likely be displaced by GitLab anyway in near future.

Jun 26 2019, 12:24 PM · Documentation, KDevelop
simgunz added a comment to D21156: Fix browse mode not disabled after Ctrl is released.

Huh? It's easier than that:

  • If you do not have the commit yet locally:

    ` arc patch --nobranch <ID> git push `

Or probably a simpler approach is to run these two commands irregarding the fact that I have the commits locally or not. So I just pull the 'clean' modification with the correct commit message and then push to a target branch. E.g: git push origin 5.3

Jun 26 2019, 12:01 PM · Documentation, KDevelop
simgunz added a comment to D21156: Fix browse mode not disabled after Ctrl is released.
  • If you do have the commit and it is the latest one in your history: ` arc amend git push `

Sorry but it is still not clear if this would work. In this diff I have three commits (modification_1, modification_2, revert modifications_2) in a feature branch forked from master. I need to push to 5.3. If I run those two commands I expect that three commits are pushed (not a single one with the total modification) and I would expect that those are pushed to origin/my-feature-branch not in 5.3. Isn't it?

Jun 26 2019, 11:55 AM · Documentation, KDevelop
simgunz added a comment to D21156: Fix browse mode not disabled after Ctrl is released.

Wouldn't make more sense to do arc land --onto 5.3 (and then merge 5.3 into master)? In this way the Differential Revision would be closed automatically.

I suggest to do a arc amend (to basically update the commit message with current reviewers, "Differentiatl Revision" line, etc.) and then git push your change manually to the right branch. Let's you use your normal git command-line to actually push changes, which to me is a much more thrust-worthy approach than to rely on arc to do that for me...

Jun 26 2019, 10:12 AM · Documentation, KDevelop
simgunz added a comment to D21156: Fix browse mode not disabled after Ctrl is released.

If you're not using arc you should have included the "Differential Revision" line in the commit so it would close automatically as stated in https://community.kde.org/Policies/Commit_Policy#Special_keywords_in_GIT_and_SVN_log_messages

I'll close this manually for you :)

Jun 26 2019, 6:16 AM · Documentation, KDevelop

Jun 25 2019

simgunz updated the summary of D22064: General improvements to stamp annotation.
Jun 25 2019, 6:09 PM · Okular
simgunz added a comment to D22064: General improvements to stamp annotation.

Added the enum, much better now.

Jun 25 2019, 6:08 PM · Okular
simgunz updated the diff for D22064: General improvements to stamp annotation.
  • Define enum for preview position
Jun 25 2019, 6:05 PM · Okular
simgunz added a comment to D21156: Fix browse mode not disabled after Ctrl is released.

I have pushed to 5.3 and then merged to master manually, as specified in the Phabricator page guide. How do I close the revision now?

Jun 25 2019, 5:35 PM · Documentation, KDevelop
simgunz added an edge to R32:5c5495929fcf: Fix browse mode not disabled after Ctrl is released: D21156: Fix browse mode not disabled after Ctrl is released.
Jun 25 2019, 5:34 PM
simgunz added 1 commit(s) for D21156: Fix browse mode not disabled after Ctrl is released: R32:5c5495929fcf: Fix browse mode not disabled after Ctrl is released.
Jun 25 2019, 5:34 PM · Documentation, KDevelop
simgunz committed R32:1100938098d6: Merge remote-tracking branch 'origin/5.3' (authored by simgunz).
Merge remote-tracking branch 'origin/5.3'
Jun 25 2019, 5:29 PM
simgunz committed R32:5c5495929fcf: Fix browse mode not disabled after Ctrl is released (authored by simgunz).
Fix browse mode not disabled after Ctrl is released
Jun 25 2019, 5:26 PM
simgunz updated subscribers of D22064: General improvements to stamp annotation.

@yurchor Are you willing to update the documentation regarding this modification after we merge this? (I saw you changed the doc regarding the refactor of the annotation tools config dialogs, and it seems you have experience with the documentation aspect).

Jun 25 2019, 7:30 AM · Okular

Jun 24 2019

simgunz added a comment to D22064: General improvements to stamp annotation.


Okular embedded stamp

Jun 24 2019, 6:23 AM · Okular
simgunz added a revision to T8074: Make custom stamp image support more discoverable: D22064: General improvements to stamp annotation.
Jun 24 2019, 6:10 AM · VDG, Okular
simgunz added a task to D22064: General improvements to stamp annotation: T8074: Make custom stamp image support more discoverable.
Jun 24 2019, 6:10 AM · Okular
simgunz updated the test plan for D22064: General improvements to stamp annotation.
Jun 24 2019, 6:09 AM · Okular
simgunz requested review of D22064: General improvements to stamp annotation.
Jun 24 2019, 6:05 AM · Okular
simgunz added a comment to D10859: Improve layout of annotation configuration dialogs.

@sander Thanks. It worked.

Jun 24 2019, 5:39 AM · Okular
simgunz committed R223:a33cb321df12: Improve layout of annotation configuration dialogs (authored by simgunz).
Improve layout of annotation configuration dialogs
Jun 24 2019, 5:37 AM
simgunz closed D10859: Improve layout of annotation configuration dialogs.
Jun 24 2019, 5:37 AM · Okular

Jun 23 2019

simgunz added a comment to D10859: Improve layout of annotation configuration dialogs.

Tried to land this, but I got this error message:

Jun 23 2019, 7:21 PM · Okular
simgunz updated the summary of D10859: Improve layout of annotation configuration dialogs.
Jun 23 2019, 6:35 PM · Okular

Jun 22 2019

simgunz added a comment to D10859: Improve layout of annotation configuration dialogs.

Perfect. I can land this tomorrow evening.

Jun 22 2019, 1:23 PM · Okular
simgunz added a comment to D10859: Improve layout of annotation configuration dialogs.

Just waiting for your OK.

Jun 22 2019, 12:52 PM · Okular

Jun 21 2019

simgunz added a comment to D21755: [RFC] Replace ToolAction by a more universal “ToggleActionMenu”.

I have tested this patch and added some inline comments. It seems to me that ToggleActionMenu requires way more external code to make it work, compared to ToolAction, which is quite automated. I think that some things could be made default in ToggleActionMenu, and some aspect hidden as well e.g. manage the QActionGroup internally without having the user have to manage it and set the action eveytime.

What would be a use case for an integrated QActionGroup? At least it couldn’t be used for the Selection Tools menu, because there are more mouse modes.

Jun 21 2019, 8:25 PM · Okular

Jun 20 2019

simgunz added a comment to D21755: [RFC] Replace ToolAction by a more universal “ToggleActionMenu”.

To be more clear, I think that the code below should provide a default working action menu (basically as it was before, but customizable if necessary):

Jun 20 2019, 7:55 PM · Okular
simgunz added a comment to D21755: [RFC] Replace ToolAction by a more universal “ToggleActionMenu”.

I have tested this patch and added some inline comments. It seems to me that ToggleActionMenu requires way more external code to make it work, compared to ToolAction, which is quite automated. I think that some things could be made default in ToggleActionMenu, and some aspect hidden as well e.g. manage the QActionGroup internally without having the user have to manage it and set the action eveytime.

Jun 20 2019, 7:50 PM · Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Remove unuseful annotation separators
  • Correctly set favorites action enabled
  • Properly trigger favorite action
  • Allow unchecking annotation actions
  • Minor refactor
  • Save state of continuous mode
Jun 20 2019, 7:07 PM · Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jun 20 2019, 7:07 PM · VDG, Okular
simgunz updated the diff for D21156: Fix browse mode not disabled after Ctrl is released.
  • Document feature
Jun 20 2019, 8:06 AM · Documentation, KDevelop
simgunz updated the diff for D21156: Fix browse mode not disabled after Ctrl is released.

I reverted the commit were the Ctrl feature was disabled, going back to the first fix I proposed

Jun 20 2019, 7:50 AM · Documentation, KDevelop

Jun 18 2019

simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jun 18 2019, 6:13 AM · VDG, Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jun 18 2019, 6:05 AM · VDG, Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jun 18 2019, 6:04 AM · VDG, Okular
simgunz added a comment to D15580: [WIP] New annotation toolbar.

Don’t have UI feedback that asks for action already in this patch. :)

  • Maybe the 4 left buttons should indicate that they require further action (drawing). Currently they look like buttons in a word processor, where you have to select the text first. *1) Using the existing dynamic annotation icons might look better, as soon as someone made them more low-resolution friently.
Jun 18 2019, 6:03 AM · Okular

Jun 17 2019

simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Fix and simplify width action
  • Fix and simplify opacity action
  • Fix annotation tools actions
  • Rename color action
  • Formatting
Jun 17 2019, 6:51 PM · Okular
simgunz added a comment to D21755: [RFC] Replace ToolAction by a more universal “ToggleActionMenu”.

Currently I am using ToolAction in the new annotation toolbar to selected among different geometrical annotation. [...]
For this purpose I need that the action are checkable, and that the ToggleActionMenu is checkable displaying the selected action (exaclty as ToolAction).

[...] For my use case, I would probably need to be able to display tooltips for each action in the ToggleActionMenu, to describe what they are.

[...] I need to use it for Geometrical annotations and for the Stamp annotation. For this last one I would need to display the different available stamps, so each action in the ToggleActionMenu should just be a checkable action with a full width image and no text. [...]

Maybe it’s better to use a KSelectAction directly? Not sure whether it shows the current selection on the toolbar button, but probably it does so in combobox mode.

For the stamps, you probably need a combobox with a custom item view.

Jun 17 2019, 5:43 PM · Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jun 17 2019, 3:10 PM · VDG, Okular
simgunz added a comment to D15580: [WIP] New annotation toolbar.

Favorite annotations:
The Star adds the currently selected tool to the favorite list, the bookmark symbol displays the list of the favorite tools:

Jun 17 2019, 2:58 PM · Okular
simgunz added a comment to D21755: [RFC] Replace ToolAction by a more universal “ToggleActionMenu”.

From my point of view, this is complete now. 3 TODOs left (see revision description at the top).

ToolAction used its own tooltip for the toolbar buttons. Tell me if that is still needed.

@simgunz you just told that you will use ToolAction? If you show me your implementation, it’s ok for me to use that instead. Otherwise you can tell me what you need (tooltips?). :)

Jun 17 2019, 2:50 PM · Okular

Jun 16 2019

simgunz updated the diff for D15580: [WIP] New annotation toolbar.

Added the favorite tools.

Jun 16 2019, 7:27 PM · Okular
simgunz added inline comments to D15580: [WIP] New annotation toolbar.
Jun 16 2019, 6:37 PM · Okular

Jun 14 2019

simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jun 14 2019, 10:23 AM · VDG, Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Added opacity action
  • Cleaned width action
  • Added action to access advanced settings for all annotations
  • Big code refactor:
    • m_toolDefinition is now a QDomDocument, so that the toolElement are not randomly deleted when the associated QDomDocument goes out of scope.
    • AnnotationActionHandler acts directly on PageViewAnnotator instead of emitting signals
  • Further code cleaning in pageview.cpp
  • Annotations work only in Normal mouse mode, and switch to it when they are selected
Jun 14 2019, 10:22 AM · Okular
simgunz added inline comments to D15580: [WIP] New annotation toolbar.
Jun 14 2019, 5:49 AM · Okular

Jun 12 2019

simgunz added inline comments to D15580: [WIP] New annotation toolbar.
Jun 12 2019, 6:34 AM · Okular

Jun 10 2019

simgunz updated the diff for D10859: Improve layout of annotation configuration dialogs.
  • Fix code formatting
Jun 10 2019, 5:07 PM · Okular
simgunz updated the diff for D10859: Improve layout of annotation configuration dialogs.
  • Properly cast QObject
Jun 10 2019, 5:00 PM · Okular
simgunz abandoned D21730: Pass gridLayout pointer to createStyleWidget.
Jun 10 2019, 4:54 PM · Okular
simgunz updated the summary of D21730: Pass gridLayout pointer to createStyleWidget.
Jun 10 2019, 4:53 PM · Okular
simgunz updated the diff for D10859: Improve layout of annotation configuration dialogs.

Fix author accelerator, remove 'for pdf only' tooltips

Jun 10 2019, 4:51 PM · Okular
simgunz requested review of D21730: Pass gridLayout pointer to createStyleWidget.
Jun 10 2019, 4:50 PM · Okular

Jun 5 2019

simgunz added a comment to D15580: [WIP] New annotation toolbar.

I renamed PageViewToolBar and moved to a file by itself. I took inspiration from Dolphin for the name.

Jun 5 2019, 4:10 PM · Okular
simgunz updated the diff for D15580: [WIP] New annotation toolbar.
  • Use draw-polyline icon for Polygon
  • Increase version of part.rc
  • Change icon of strikethrough (remove symbolic suffix)
  • Add TODO in code
  • Rename PageViewToolBar to AnnotationActionHandler
  • No need to forceHide mechanism
  • Use KToggleAnnotationToolBar to show/hide toolbar (not working)
Jun 5 2019, 4:02 PM · Okular
simgunz added a comment to D15580: [WIP] New annotation toolbar.

Or: Why is this still PageViewToolBar? It is not anymore in the PageView?

I'll move it, added to TODO

Jun 5 2019, 7:54 AM · Okular
simgunz added a comment to D15580: [WIP] New annotation toolbar.

Or: Why is this still PageViewToolBar? It is not anymore in the PageView?

Jun 5 2019, 7:31 AM · Okular
simgunz updated the task description for T8076: Fix design of annotation toolbar.
Jun 5 2019, 7:29 AM · VDG, Okular