Similar logic as 2a90a473; there's no
point in taunting the user by exposing UI elements to them that won't
work.

This also lets the ViewerWindow benefit from it.

The windows where never hiddden in these cases. The opening/closing
animations that could be seen was spectacle closing and opening a
window after the screenshot was taken.
BUG:376865

The setting name was changed 9d4ffdf7

This happened because the actions were given a parent in a previous
commit. They worked more than once before the commit because the actions
were being leaked. `action->setProperty("isConfigurationAction", true)`
fixes the single use action problem.

BUG: 464706

Don't dim as much with no selection and don't dim at all when annotating
with no selection.

Center without selection, put along the bottom of the inner area of the
selection when the selection is large enough that the toolbar has to fit
inside.

Allow dragging main toolbar and annotation tools toolbar around when
there's no selection and remember positions unless the selection
changes. I think this is needed because it's impossible to know which
part of the screen the user will want to annotate when there's no
selection. Not allowing dragging with a selection because the toolbars
for 2 screens have to act like they're one toolbar when a selection
spans 2 screens. That's hard to deal with when allowing them to be dragged.

also, use the new properties to keep AnnotationEditor and TextTool from
breaking after the change.

Aleix Pol Gonzalez authored Jan 19, 2023 and Aleix Pol Gonzalez committed Jan 20, 2023

What we need after recording a video is different from an image, so let
the UI adapt to it.

Adds a dependency to VLC-Qt's qml component to embed de video.

Aleix Pol Gonzalez authored Jan 04, 2023 and Aleix Pol Gonzalez committed Jan 20, 2023

Includes the wayland implementation

This way memory leak tooling doesn't report them as leaked.

BUG: 464465

When this feature isn't supported, currently its checkbox gets disabled.
Because there is no way to make it enabled without rebooting into an
X11 session, its permanently disabled presence is frustrating to the
user and consumes visual space.

To fix this, hide the checkbox instead of disabling it when its feature
is not available.

David Redondo authored Jan 17, 2023 and Nate Graham committed Jan 17, 2023

It's not used anymore

David Redondo authored Jan 17, 2023 and Nate Graham committed Jan 17, 2023

GIT_SILENT

The parentheses are needed otherwise the comparison is done first.
BUG:432011

Noah Davis authored Jan 12, 2023 and Fushan Wen committed Jan 13, 2023

Ensure the spinbox doesn't get cut off

We got some complaints about the "Always" default. That made me reconsider the
whole thing, and now I realize taking repeated rectangular screenshots of the
same area is a rather more niche usecase than I first imagined.

This should be a better balance, and hopefully less confusing for users.

cc @ngraham 

BUG: 463417