dporobic (Damir Porobic)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Jan 26 2017, 10:32 AM (68 w, 6 d)
Availability
Available

Recent Activity

Mon, May 21

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@rkflx do you have a CI that builds on every commit? xD
That was another refactoring, should be working again.

Mon, May 21, 7:49 AM · Spectacle

Thu, May 10

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@rkflx the slow test build should be fixed now. The initial approach has build all the source for every source file, now we build the source as a library once and link it to every test.

Thu, May 10, 10:10 AM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@rkflx are you using latest version? I have fixed those error yesterday commit

Thu, May 10, 8:19 AM · Spectacle

Apr 22 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

It's not obvious enough. You could have just changed the colours of those items to a lighter shade.

It's not the same, with this approach you see selected items that are hidden behind other items and the color could be any color of course.

Apr 22 2018, 11:03 AM · Spectacle

Apr 20 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@huoni how about something like this for selected items:

Apr 20 2018, 5:03 PM · Spectacle

Apr 17 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Thanks for the feedback :)

Apr 17 2018, 8:00 PM · Spectacle

Apr 14 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@rkflx @huoni
I've added few more things, like multiple item selection/delete/move, bring selected items to front/back, different cursors for hover over items and resize, snap-to for creating items (hold ctrl), toggle selection (hold ctrl), preciser hit detection (we count now only hit on shape, if item has no file, the inner space is not accounted as item).

Apr 14 2018, 8:35 PM · Spectacle

Apr 3 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Yeap, that looks nice, that is the kind of control I was hopping to implement. I'll look later into implementing something like this (or the exact think) in KWidgetsAddons, when we get to the fine-tuning stuff.

Apr 3 2018, 6:20 AM · Spectacle

Mar 15 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Yeah, saving configuration and last screenshot option is saved by default though this can be disabled via settings. Same works in KImageAnnotator now, I'm using more or less the same approach.
The color grid works like: Load a predefined list of colors, after loaded, load last or default color for current tool, if the color doesn't exists in current grid, add the color to the grid. So basically all but the last selected custom color is cleared on startup. If you have selected one of the default colors, all custom colors are deleted after startup.

Mar 15 2018, 7:26 PM · Spectacle

Mar 14 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

…now this also looks kinda neat. I guess that's because compared to your initial screenshot here the downwards pointing arrows are missing, making me wonder about usability for the user, though…

The initial screenshot is a QToolButton with menu. What I've added here is only a QToolButton that opens QColorDialog, which, in case we pick this approach, I would replace with a popup menu, that shows a grid with colors, similar to the one I have used in KSnip. We can show with or without arrow. KColorCombo is not very flexible as I can't set and custom icons so probably wouldn't use it. With the other option (one that looks and works like a combobox), I would make my own color combobox where I can set custom icons.

Mar 14 2018, 8:39 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

And here the button version, which for me personally fits/looks best (image different icons for the two pickers and some nicer ordering):

Mar 14 2018, 7:03 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

I would definitely go with two row tools when we put the properties stuff down there, I do not like the space there too.

Mar 14 2018, 6:34 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

I've been playing around with the color picker, tried out KColorCombo and some custom stuff. Does this look anything useful? With the icon left from the Combobox it seems a bit too wide. Below for the size picker another option, custom adding icon into the combobox, something like they have in LibreOffice Draw, we could make the same for the color picker.

Mar 14 2018, 6:16 PM · Spectacle

Mar 12 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

I agree that three color settings might be an overkill for simply annotating a screenshots but I think that we need two, at least for the text and numbering tool as @huoni explained.

Mar 12 2018, 7:27 PM · Spectacle

Mar 11 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

How about something like this instead of a new UI for editing screenshots with Spectacle: https://github.com/lupoDharkael/flameshot/blob/master/img/appPreview/animatedUsage.gif

Mar 11 2018, 3:34 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Update:
I was not happy with the KColourPaint style so I went more in direction of LibreOffice Draw, I'm trying out not this approach, here is how it looks like:

Mar 11 2018, 1:06 PM · Spectacle

Mar 7 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Can we in Phabricator somehow track open tasks? I would like to have a list of stuff that is planned and will come eventually in future, we could use the to prioritize and track progress. Other folks would be able to see what is planned and what note and eventually add wishes. Something like sub-tasks. How do you handle stuff like that?

Mar 7 2018, 10:16 AM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Instead of PNGs, we can probably find some SVG icons in Breeze or try to request missing ones.

Agreed. Current icons don't adapt to dark themes :)

Mar 7 2018, 10:11 AM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@rkflx Thanks for the detailed feedback :)

Mar 7 2018, 9:52 AM · Spectacle

Feb 25 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

I've made a prototype for the Image Annotator, nothing fancy, four annotation items, selection, resizing and deleting. I wanted to test how I could achieve the item resizing and I think that works fine now, so I will slowly turn that into a working application/widget.

Feb 25 2018, 6:44 PM · Spectacle

Feb 7 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Nice, I'll start work on this, hopefully before weekend. I believe we can quickly have some basic functionality for first thought.

Feb 7 2018, 12:22 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

I came to the conclusion though, that this component/framework probably won't be so generic to be called "KImageEditor" but rather "KScreenshotEditor" or "KAnnotateImageEditor"

Feb 7 2018, 7:41 AM · Spectacle

Feb 5 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Assume we want to annotate something in Gwenview or Skanlite: Certainly we would not want to replace Gwenview's advanced image display component completely

Feb 5 2018, 9:49 AM · Spectacle

Feb 4 2018

dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

I think I'm a fan of the annotation widget approach. Later on we could replace the preview window with the annotation widget.
Someone mentioned KParts earlier, I haven't worked with it yet but that could be something suitable as fart as I can tell from my quick googeling. Would be nice to have someone with experience with KParts to give kick start or at least point me in the right direction.

Feb 4 2018, 3:35 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

Ok, that would go in direction of being an annotation "widget" that you could use basically anywhere, interesting idea.

Feb 4 2018, 2:53 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@rkflx
Ok, so we go for now with a separate window, it's not the perfect solution but it might be our best bet for now and we could change stuff in the future when we have some foundation.

Feb 4 2018, 2:17 PM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

@rkflx Thank you for your detailed feedback!

Feb 4 2018, 11:32 AM · Spectacle
dporobic added a comment to T6321: [WISH] Create a quick image editor for Spectacle.

If no one is working on this task I would take it and see what I can do.

Feb 4 2018, 8:18 AM · Spectacle

Feb 13 2017

dporobic added a comment to D4483: Keyboard shortcuts for Uses in Code Browser.

Changed formatting as suggested. If it's ok now could you please commit the patch, I don't have commit rights?

Feb 13 2017, 8:07 AM · KDevelop

Feb 12 2017

dporobic added a comment to D4483: Keyboard shortcuts for Uses in Code Browser.

Changed formatting as suggested. If it's ok now could you please commit the patch, I don't have commit rights?

Feb 12 2017, 1:15 PM · KDevelop
dporobic updated the diff for D4483: Keyboard shortcuts for Uses in Code Browser.

More formatting and style cleanup as suggested by mwolff

Feb 12 2017, 1:13 PM · KDevelop

Feb 11 2017

dporobic added inline comments to D4483: Keyboard shortcuts for Uses in Code Browser.
Feb 11 2017, 7:32 PM · KDevelop
dporobic updated the diff for D4483: Keyboard shortcuts for Uses in Code Browser.

Moved must of the functionality to single free function in separate namespace. Some additional cleanup as suggested by mwolff

Feb 11 2017, 7:28 PM · KDevelop

Feb 7 2017

dporobic retitled D4483: Keyboard shortcuts for Uses in Code Browser from to Keyboard shortcuts for Uses in Code Browser.
Feb 7 2017, 6:29 PM · KDevelop

Feb 1 2017

dporobic added a comment to D4334: Pressing "Continue" starts "Debug" if program is not running.

Sure, damir_porobic(at)live(dot)com would be the address and name is Damir Porobic.

Feb 1 2017, 6:18 PM · KDevelop

Jan 31 2017

dporobic added a comment to D4334: Pressing "Continue" starts "Debug" if program is not running.
In D4334#81627, @mwolff wrote:

one minor nitpick, otherwise lgtm

do you have commit rights? if so, feel free to amend and push to master

Jan 31 2017, 4:02 PM · KDevelop
dporobic updated the diff for D4334: Pressing "Continue" starts "Debug" if program is not running.

Changed function name that changes the continue action properties from continueStartsDebug to setContinueStartsDebug

Jan 31 2017, 3:57 PM · KDevelop

Jan 30 2017

dporobic added a comment to D4334: Pressing "Continue" starts "Debug" if program is not running.
In D4334#81123, @mwolff wrote:

I am missing some things here, besides the nitpicks below:

The action's tooltip, and possibly even its icon, should now imo update depending on the state to indicate what action will be triggered when it is executed.

Can you implement that as well please?

Thanks

Jan 30 2017, 6:29 PM · KDevelop
dporobic updated the diff for D4334: Pressing "Continue" starts "Debug" if program is not running.

+ Added a function that changes the icon and text of the continue_debug action to look like launch_debug action depending on the debugger state.
+ Changed formatting as suggested by mwolff.

Jan 30 2017, 6:21 PM · KDevelop

Jan 29 2017

dporobic added reviewers for D4334: Pressing "Continue" starts "Debug" if program is not running: apol, kfunk.
Jan 29 2017, 1:50 PM · KDevelop
dporobic retitled D4334: Pressing "Continue" starts "Debug" if program is not running from to Pressing "Continue" starts "Debug" if program is not running.
Jan 29 2017, 1:49 PM · KDevelop

Jan 26 2017

dporobic added a comment to D4126: Add a "clear" button and related code for the OutputView.

https://bugs.kde.org/show_bug.cgi?id=361420

Jan 26 2017, 10:15 PM