aprcela (Antonio Prcela)
User

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Monday

  • Clear sailing ahead.

User Details

User Since
Jul 12 2019, 10:06 PM (5 w, 16 h)
Availability
Available

Recent Activity

Today

aprcela retitled D23212: [kdevelop] Remove 'Git Stashes' text above separator from Remove 'Git Stashes' text above separator to [kdevelop] Remove 'Git Stashes' text above separator.
Sat, Aug 17, 10:32 AM · KDevelop
aprcela updated the test plan for D23212: [kdevelop] Remove 'Git Stashes' text above separator.
Sat, Aug 17, 10:29 AM · KDevelop
aprcela requested review of D23212: [kdevelop] Remove 'Git Stashes' text above separator.
Sat, Aug 17, 10:28 AM · KDevelop
aprcela updated the diff for D23210: Add autosave feature.
  • Indicate that it quits only after user's copy/save interaction
Sat, Aug 17, 9:07 AM · Spectacle

Yesterday

aprcela updated the test plan for D23210: Add autosave feature.
Fri, Aug 16, 10:22 PM · Spectacle
aprcela added a comment to D23210: Add autosave feature.

Has merge conflicts if D23162 gets approved.

Fri, Aug 16, 9:57 PM · Spectacle
aprcela updated the test plan for D23210: Add autosave feature.
Fri, Aug 16, 9:54 PM · Spectacle
aprcela requested review of D23210: Add autosave feature.
Fri, Aug 16, 9:52 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

Fixed, sorry..

Fri, Aug 16, 5:34 PM · Spectacle
aprcela updated the diff for D23162: Add option to additionally save screenshot to clipboard.
  • Rename some 'file' variables to 'image'
Fri, Aug 16, 5:33 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

Did some variable renaming.
Mostly from variables containing CopyToClipboard to CopySaveLocationToClipboard. So it's more clear what is saved to the clipboard (image or location).
Also: from mCopyFileToClipboard to mCopyImageToClipboard

Fri, Aug 16, 8:56 AM · Spectacle
aprcela updated the diff for D23162: Add option to additionally save screenshot to clipboard.
  • Rename some 'file' variables to 'image'
Fri, Aug 16, 8:51 AM · Spectacle

Thu, Aug 15

aprcela updated the diff for D23162: Add option to additionally save screenshot to clipboard.
  • Use radiobuttons instead of checkboxes
Thu, Aug 15, 10:03 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

Instead of repeatedly accessing SpectacleConfig::instance()->copyFileToClipboard(), maybe create a new member variable to hold that information, in the same vein as mCopyToClipboard

Thu, Aug 15, 6:55 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

Very nice work. Functionality seems bang on target.

Thu, Aug 15, 5:24 PM · Spectacle
aprcela updated the diff for D23162: Add option to additionally save screenshot to clipboard.
  • Use radiobuttons instead of checkboxes
Thu, Aug 15, 5:24 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

Added and changed :)

Thu, Aug 15, 4:52 PM · Spectacle
aprcela updated the diff for D23162: Add option to additionally save screenshot to clipboard.
  • Use radiobuttons instead of checkboxes
Thu, Aug 15, 4:52 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

Changed to radiobuttons.
Now I need someone to help me test if all works as expected, especially via DBus

Thu, Aug 15, 4:40 PM · Spectacle
aprcela updated the diff for D23162: Add option to additionally save screenshot to clipboard.
  • Use radiobuttons instead of checkboxes
Thu, Aug 15, 4:35 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

1st put the screenshot in the clipboard, after it has been saved replace the clipboard content with the file location

This seems like a pretty esoteric use case. What would be the purpose? Can you envision why someone would want to do this. Also note that this isn't currently a feature that exists, so we're not debating taking it away from anyone, but rather proposing to add it in the first place. That makes the bar for adding it much higher. :)

Thu, Aug 15, 2:19 PM · Spectacle
aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

@ngraham
Sorry, it was silly of me to assume that the feature number would suffice. I've added a description with a video showing the feature.

Thu, Aug 15, 7:25 AM · Spectacle
aprcela updated the test plan for D23162: Add option to additionally save screenshot to clipboard.
Thu, Aug 15, 7:22 AM · Spectacle
aprcela updated the diff for D23162: Add option to additionally save screenshot to clipboard.
  • Remove duplicate include
  • Move oneliner and remove method.
  • Enable notification when the file is saved to clipboard
Thu, Aug 15, 7:09 AM · Spectacle

Wed, Aug 14

aprcela added a comment to D23162: Add option to additionally save screenshot to clipboard.

I'm still not sure whether to set the new option (let's call this one FC) alone under general (as is now) or move "Copy file location to clipboard after saving" (let's call it FL) to general.
Both options can be used as ON, but if FL is also on, then after the file has been saved the clipboard gets overwritten by the file location. Maybe add a notice/warning where the options can be set?

Wed, Aug 14, 10:59 PM · Spectacle
aprcela requested review of D23162: Add option to additionally save screenshot to clipboard.
Wed, Aug 14, 10:55 PM · Spectacle

Tue, Aug 13

aprcela updated the diff for D23129: Remove redundant code.

Added my full name in git config.

Tue, Aug 13, 9:08 PM · Dolphin
aprcela requested review of D23129: Remove redundant code.
Tue, Aug 13, 3:49 PM · Dolphin

Sat, Jul 20

aprcela added a comment to D20427: When renaming files, move to next file using tab key..

Thanks for the patch. I like the feature but there is something missing: if the next file is out of the visible view, the view doesn't scroll and the rename operation gets canceled.

Sat, Jul 20, 9:25 AM · Dolphin

Jul 15 2019

This is a test notification, sent at Mon, Jul 15, 18:47.
Jul 15 2019, 4:47 PM

Jul 12 2019

aprcela added a watcher for Dolphin: aprcela.
Jul 12 2019, 10:08 PM