Bug 468778

Summary: Add screen recording options to "no screenshot taken on launch" window
Product: [Applications] Spectacle Reporter: Nate Graham <nate>
Component: GeneralAssignee: Noah Davis <noahadvs>
Status: RESOLVED FIXED    
Severity: wishlist CC: ales.astone, ashark, boredsquirrel, fanzhuyifan, justin, kde, luis.buechi, michelnospam, mikko.mensonen
Priority: NOR    
Version: 23.04.0   
Target Milestone: ---   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=468780
https://bugs.kde.org/show_bug.cgi?id=468899
Latest Commit: Version Fixed In: 24.05
Sentry Crash Report:
Attachments: Screenshot of a start window with missing recording options

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