Add --select-and-resize option and feature

[eepp]

See commit message.

[Roger]

awesome! i wanted to implement something like this, i'll take a look later

[sethdusek]

Do you think that .webm recording could be added on later too? it'd be pretty awesome

[eepp]

Do you think that .webm recording could be added on later too? it'd be pretty awesome

Everything is possible, but recordMyDesktop is already close to this.

[sethdusek]

unfortunately recordmydesktop does not let you select a portion of the screen using your mouse or save as .webm (needed for uploading to gfycat)

[eepp]

Which is why I wrote close: you may use -x, -y, --width, and --height to select a rectangle, then use FFmpeg to convert:

ffmpeg -i out.ogv out.webm

One thing I'm thinking about is to split the rectangle selection code from escrotum, and create a simple app. which returns the selection's position and dimension. Then we could add the -x, -y, --width, and --height options to escrotum, and this rectangle selection tool could be used with recordMyDesktop as well. Something like this:

#!/usr/bin/bash

rect="$(selectrect)"

if [ $? -ne 0 ]; then
    exit
fi

x=$(echo $rect | cut -f1 -d' ')
y=$(echo $rect | cut -f2 -d' ')
w=$(echo $rect | cut -f3 -d' ')
h=$(echo $rect | cut -f4 -d' ')

recordmydesktop -x $x -y $y --width $w --height $h --no-sound -o /tmp/out.ogv --overwrite

if [ $? -ne 0 ]; then
    exit
fi

ffmpeg -i /tmp/out.ogv /tmp/out.webm
# do whatever you like with /tmp/out.webm here

WDYT?

[Zod-]

I actually added webm recording using ffmpeg in my fork some time ago but separating it seems to make more sense. I'm using a wrapper script anyway that calls escrotum and then directly uploads the screenshot/video or other files.

[Roger]

About video recording, i created the issue here #24

[dequis]

There are lots of functions in here that are essentially copypasted versions of others with very minor variations for each side, and for each corner. You really should find some other way to do this.

There are

• 8 variants of pos_to_rect_is_resizing_*
• 4 variants of rect_resize_*,
• 4 variants of handle_ss_rz_*

It's way too much code duplication.

[Roger]

@eepp hey, will you try to reduce the code duplication? i'll like to have this feature merged :)

[eepp]

What do you propose?

There are four pos_to_rect_is_resizing_*() checking for top/bottom/left/right conditions, and the four others reuse those four.

All the handle_ss_rz_*() reuse the equivalent rect_resize_*(), which are pretty much unique in what they do.

I guess top/left and bottom/right functions could be merged somehow.