[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2022-02-10 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=447517

--- Comment #8 from Patrick Silva  ---
(In reply to Antonio Prcela from comment #5)
> Can confirm now. Seems that when both options for "after taking a
> screenshot" are set:
> "save file to default folder" - enabled
> and
> "copy image to clipboard"
> 
> Will take a look into it. Have a feeling where this is :)

two notifications bug is still reproducible on neon unstable.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2022-01-11 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447517

Nate Graham  changed:

   What|Removed |Added

  Latest Commit||https://invent.kde.org/grap
   ||hics/spectacle/commit/08e82
   ||e2e0783c77b60b77aaa7018f517
   ||a52d1efd
 Resolution|--- |FIXED
 Status|ASSIGNED|RESOLVED
   Version Fixed In||22.04

--- Comment #7 from Nate Graham  ---
Git commit 08e82e2e0783c77b60b77aaa7018f517a52d1efd by Nate Graham, on behalf
of Antonio Prcela.
Committed on 11/01/2022 at 15:37.
Pushed by ngraham into branch 'master'.

Simplify branching for the notify text and fix messages containing dots

Simplify the logic by removing the bool isBackgroundMode. Fix double
notifications
where one has dots instead of the filename and save path.

Add missing notification text if mCopyLocationToClipboard is set. Uses 2in1
principle: Considering that when mCopyLocationToClipboard is true it means that
the image also gets saved as a file, so this makes the notification text show
both the path where it's saved & the name of it _and_ the information that the
file path has been added to the clipboard.
FIXED-IN: 22.04

M  +7-9src/SpectacleCore.cpp

https://invent.kde.org/graphics/spectacle/commit/08e82e2e0783c77b60b77aaa7018f517a52d1efd

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2021-12-30 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=447517

Nate Graham  changed:

   What|Removed |Added

 CC||n...@kde.org

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2021-12-29 Thread Bug Janitor Service
https://bugs.kde.org/show_bug.cgi?id=447517

Bug Janitor Service  changed:

   What|Removed |Added

 Status|CONFIRMED   |ASSIGNED

--- Comment #6 from Bug Janitor Service  ---
A possibly relevant merge request was started @
https://invent.kde.org/graphics/spectacle/-/merge_requests/116

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2021-12-29 Thread Antonio Prcela
https://bugs.kde.org/show_bug.cgi?id=447517

Antonio Prcela  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|REPORTED|CONFIRMED

--- Comment #5 from Antonio Prcela  ---
Can confirm now. Seems that when both options for "after taking a screenshot"
are set:
"save file to default folder" - enabled
and
"copy image to clipboard"

Will take a look into it. Have a feeling where this is :)

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2021-12-29 Thread medin
https://bugs.kde.org/show_bug.cgi?id=447517

--- Comment #4 from medin  ---
Created attachment 144926
  --> https://bugs.kde.org/attachment.cgi?id=144926=edit
Spectacle settings

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2021-12-29 Thread medin
https://bugs.kde.org/show_bug.cgi?id=447517

--- Comment #3 from medin  ---
(In reply to Antonio Prcela from comment #2)
> How did you take the screenshot (keyboard shortcut, call from terminal)?
> What settings do you have set in Spectacle?

With terminal and after trying all cases (region, full-screen, active window,
window under cursor) it shows only one notification, but with shortcuts and in
all cases it shows two notifications. Concerning spectacle settings see
attached image.

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2021-12-28 Thread Antonio Prcela
https://bugs.kde.org/show_bug.cgi?id=447517

Antonio Prcela  changed:

   What|Removed |Added

 CC||antonio.prc...@gmail.com

--- Comment #2 from Antonio Prcela  ---
How did you take the screenshot (keyboard shortcut, call from terminal)?
What settings do you have set in Spectacle?

-- 
You are receiving this mail because:
You are watching all bug changes.

[Spectacle] [Bug 447517] Two notifications are shown after each screen capture

2021-12-25 Thread Patrick Silva
https://bugs.kde.org/show_bug.cgi?id=447517

Patrick Silva  changed:

   What|Removed |Added

 CC||bugsefor...@gmx.com

--- Comment #1 from Patrick Silva  ---
Related to bug 422874

-- 
You are receiving this mail because:
You are watching all bug changes.