Awesome.
- Queries
- All Stories
- Search
- Advanced Search
Advanced Search
Mar 18 2022
I am continuing work on this feature here: https://invent.kde.org/graphics/spectacle/-/merge_requests/129 (My code is loosely based on Leon's. I wasn't able to help with pushing this MR over the finish line back then but I always wanted to see this actually get merged.)
Jan 23 2022
perfect.
This is in progress with https://invent.kde.org/graphics/spectacle/-/merge_requests/119
Jan 21 2022
Hi @acrouthamel ! Do you have any further information about the kwin functionality? was it in any way used? Or is anybody working on this?
Nov 27 2021
Oct 19 2021
Resolved by adding an option in https://invent.kde.org/graphics/spectacle/-/merge_requests/71.
May 14 2021
Done in Spectacle 20.08 IIRC.
Dec 18 2020
Sweet!
Awesome feature!
Oct 14 2020
Thanks for the feedback. Totally understand that the rectangular region capture mode is a desirable feature and that it is not an option to drop it.
Oct 13 2020
It doesn't work when trying to capture open menus. At least on X11, menus close when doing the capture.
In D29282#676586, @czeidler wrote:I am the author of D29550 which adds supports for a post capture selection option for already taken screenshots. D29550 is currently blocked by this patch.
After giving it a good thought, I see the following general problems with the existing rectangular region capture mode:
- It doesn't work when trying to capture open menus. At least on X11, menus close when doing the capture.
- By supporting different screen scales, the QuickEditor code becomes very complex. I wouldn't feel comfortable applying a patch while keeping all the screen scaling in mind. I would have to invest quite some time to get a deep understanding of the scaling problem and even then I doubt I could make changes without introducing any new issues, which brings me to the next point.
- Testing, how do we test any changes? With X11, Wayland, scaling factors, monitor positions and X QuickEditor features there seem to be too many combinations to test properly. Especially, I don't see a reliable and easy way to automate these kind of tests.
- All the hard bits of translating real screen position to window positions which is already done by the display server needs to be redone in QuickEditor (more or less). Not really a problem but also not a nice thing to do.
For this reason I suggest the following approach: a) Compose the screen capture(s) into a single pixmap as it is done already. b) Drop support for rectangular region capture mode in favour of the post selection mode in D29550 which operates on this pixmap.
This has the advantage that a) and b) are completely decoupled. How the pixmap is composed can easily be changed and screen scales can be applied as desired. QuickEditor stays relatively simple and straightforward to modify.
Sep 2 2020
Yaayy!
Merging this as discussed in the VDG chat
Can this be moved over to invent.kde.org and merged?
Jul 27 2020
For now this patch is blocked until we can get the screen scale factor from somewhere else than libkscreen.
In D29282#676011, @meven wrote:In D29282#676008, @ngraham wrote:Tried this out and Rectangular Region mode disappeared from the combobox.
I think this is because there is a now check in spectacle that plasma is > 5.20 for the shuttermode.
Should check for > 5.19.80 then
Jul 25 2020
In D29282#676008, @ngraham wrote:Tried this out and Rectangular Region mode disappeared from the combobox.
I think this is because there is a now check in spectacle that plasma is > 5.20 for the shuttermode.
You can edit the PlatformKWinWayland::supportedShutterModes to work locally.
The patch needs adjustment.
Jul 24 2020
Would be really nice to finally get this in.
Jul 23 2020
Tried this out and Rectangular Region mode disappeared from the combobox.
Jun 23 2020
Rebased on master
Jun 19 2020
Ah, that sounds like a sensible idea.
Jun 17 2020
In D29282#675339, @ngraham wrote:Needs a rebase.
Jun 16 2020
Needs a rebase.
Jun 13 2020
Not really needed now that the notification has a hamburger menu on it.
Jun 11 2020
Needs https://invent.kde.org/graphics/spectacle/-/merge_requests/5 after latest D29408 changes
Check Plasma major version, use splitRef
Jun 10 2020
Make raw DBus call
In D29408#674966, @meven wrote:In D29408#674962, @davidre wrote:I think we shouldn't use QDBUsInterface but given that we already do this is probably ok
The alternative is just less documented AFAICT.
IRC the alternative is about adding xml files that generate code.
In D29408#674962, @davidre wrote:I think we shouldn't use QDBUsInterface but given that we already do this is probably ok
I think we shouldn't use QDBUsInterface but given that we already do this is probably ok
Very short nice!
Jun 9 2020
I am abandonning https://invent.kde.org/frameworks/kdbusaddons/-/merge_requests/1/diffs.
Check only once the plasma version when needed
Avoid adding the dependency to spectacle
Jun 8 2020
This adds a dependency on https://invent.kde.org/frameworks/kdbusaddons/-/merge_requests/1/diffs
Use KDBusAddon' PlasmaVersion
Removed .arcconfig, rebased
Jun 6 2020
Jun 5 2020
This is ready to land except I need either to :
- wait after 20.08 is released since this fix is dependent on KWin D29407 that won't be available before Plasma 5.20.
- or add a runtime plasma version check
Clean up delayed shutterMode
Fix build
Jun 4 2020
Maybe that https://phabricator.kde.org/D23932 would come in handy?
Is there a way we can make it a runtime dependency rather than a build-time dependency? I'm just worried about people trying to install Spectacle on a non-Plasma platform and then getting half of Plasma dragged in, which would also complicate packaging for Snap, Flatpak, and AppImage.
Jun 3 2020
All right, maybe it was a local config issue on my side. I'll make noise if it stays broken. :)
In D29487#673809, @meven wrote:In D29487#673803, @davidre wrote:Ping. Did you have time to look into
I noticed a flaw however in the current code if the first instance of spectacle is started with --new-instance a second one will always be started when the shortcut is pressed regardless of the setting.
Well it is not a flaw, it is by design.
--new-instance launches an instance without dbus connection and unique instance.
The first one without --new-instance will be the dbus unique instance.
Alright I guess. Should we maybe even hide it from the user visible options then?
Jun 2 2020
In D29487#674322, @ngraham wrote:Hmm, for me this breaks the option to open a new instance when the PrintScreen key is pressed and an existing instance is already running. It only works if I launch Spectacle manually using the --new-instance flag. The feature should work regardless of how the app is launched.
In D29282#674390, @ngraham wrote:Hmm, that's unfortunate. Is there really no cross-compositor screenshot API in Wayland? Every compositor needs to re-invent the wheel?
Jun 1 2020
Hmm, that's unfortunate. Is there really no cross-compositor screenshot API in Wayland? Every compositor needs to re-invent the wheel?
In D29282#674318, @ngraham wrote:I don't think making KScreen a mandatory dependency is going to work. If we do that, then installing Spectacle on a platform other than Plasma is going to try to pull in all of KScreen and its dependencies. It would be half of the Plasma stack.
Hmm, for me this breaks the option to open a new instance when the PrintScreen key is pressed and an existing instance is already running. It only works if I launch Spectacle manually using the --new-instance flag. The feature should work regardless of how the app is launched.
I don't think making KScreen a mandatory dependency is going to work. If we do that, then installing Spectacle on a platform other than Plasma is going to try to pull in all of KScreen and its dependencies. It would be half of the Plasma stack.