Bug 468778 - Add screen recording options to "no screenshot taken on launch" window
Summary: Add screen recording options to "no screenshot taken on launch" window
Status: RESOLVED FIXED
Alias: None
Product: Spectacle
Classification: Applications
Component: General (show other bugs)
Version: 23.04.0
Platform: Other Linux
: NOR wishlist
Target Milestone: ---
Assignee: Noah Davis
URL:
Keywords:
: 477664 481363 (view as bug list)
Depends on:
Blocks:
 
Reported: 2023-04-21 22:23 UTC by Nate Graham
Modified: 2024-03-28 07:34 UTC (History)
9 users (show)

See Also:
Latest Commit: https://invent.kde.org/graphics/spectacle/-/commit/df97b89ec03513e2525d0648c3bacdfd793c49ac
Version Fixed In: 24.05
Sentry Crash Report:

Attachments
Screenshot of a start window with missing recording options (138.42 KB, image/png)
2023-08-09 16:39 UTC, Andrew Shark 		Details
View All Add an attachment

Note You need to log in before you can comment on or make changes to this bug.
Description Nate Graham 2023-04-21 22:23:01 UTC 
When using Spectacle in the mode where it doesn't take a screenshot at launch, it instead shows a window offering to take various types of screenshots. But this window doesn't have any options for screen recording. It would be nice to do that too, to make the feature accessible when using Spectacle in this mode.
Comment 1 Andrew Shark 2023-08-09 16:39:48 UTC 
Created attachment 160871 [details]
Screenshot of a start window with missing recording options

Steps to reproduce:
1. Open Spectacle settings.
2. Go to General tab.
3. In "At Spectacle startup" set "Do not make screenshot automatically".
4. Close Spectacle settings and main window.
5. Launch Spectacle.
6. Try to find options for screen recording.

Observed:
Only screenshot options are available (see screenshot).

Expected:
Recording options should be also available.

Workaround:
Make any screenshot to allow the main window be shown, then switch to Recording tab and use recording options from there.
Comment 2 Nate Graham 2023-11-28 20:47:10 UTC 
*** Bug 477664 has been marked as a duplicate of this bug. ***
Comment 3 Henning 2024-01-01 21:23:02 UTC 
Can confirm that the interface is really confusing.

Meta+Print takes a screenshot and then you also have the ability to record. Launching Spectacle from App menu does not allow this.

Also on Plasma 6 it seems the "print" button alone does nothing.
Comment 4 fanzhuyifan 2024-01-19 01:19:28 UTC 
Would it be appropriate to bump the importance since otherwise users may never find the screen recording functionality?
Comment 5 Noah Davis 2024-01-19 04:40:48 UTC 
(In reply to fanzhuyifan from comment #4)
> Would it be appropriate to bump the importance since otherwise users may
> never find the screen recording functionality?

I'm not sure if that warrants bumping up the importance or not. With that said, I will work on this now that the translated string freeze from beta is over.
Comment 6 fanzhuyifan 2024-01-20 22:47:28 UTC 
(In reply to Noah Davis from comment #5)
> (In reply to fanzhuyifan from comment #4)
> > Would it be appropriate to bump the importance since otherwise users may
> > never find the screen recording functionality?
> 
> I'm not sure if that warrants bumping up the importance or not. With that
> said, I will work on this now that the translated string freeze from beta is
> over.

Thanks! Should this be marked as assigned then?
Comment 7 Noah Davis 2024-01-20 22:52:18 UTC 
(In reply to fanzhuyifan from comment #6)
> Thanks! Should this be marked as assigned then?

No, we have a bot that auto-assigns when someone posts a relevant merge request if they put "BUG: 468778" in the description.
Comment 8 Nate Graham 2024-02-15 16:45:17 UTC 
*** Bug 481363 has been marked as a duplicate of this bug. ***
Comment 9 Bug Janitor Service 2024-03-03 21:04:29 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/graphics/spectacle/-/merge_requests/336
Comment 10 Noah Davis 2024-03-04 15:37:30 UTC 
Git commit df97b89ec03513e2525d0648c3bacdfd793c49ac by Noah Davis.
Committed on 04/03/2024 at 15:35.
Pushed by ndavis into branch 'master'.

Add recording options to the no screenshot dialog

I also removed the "Take a new screenshot" header because it was redundant and there's a decent amount of padding all around without it. Not using tabs because they don't look good in this context and there's no segmented control yet.

Depends on https://invent.kde.org/frameworks/qqc2-desktop-style/-/merge_requests/375

M  +2    -0    src/CMakeLists.txt
M  +169  -77   src/Gui/DialogPage.qml
M  +3    -19   src/Gui/RecordOptions.qml
A  +27   -0    src/Gui/RecordingModeButtonsColumn.qml     [License: LGPL(v2.0+)]
A  +23   -0    src/Gui/RecordingSettingsColumn.qml     [License: LGPL(v2.0+)]

https://invent.kde.org/graphics/spectacle/-/commit/df97b89ec03513e2525d0648c3bacdfd793c49ac