Add format to Record button

[orschiro]

Here is my challenge.

My default format is GIF.

But sometimes my recordings become too long and GIF won't work.

Then I need to switch to webm or mp4.

But often I forget to switch back.

Annoying because I then have to redo the video.

It would help me a lot if the record button would say "Record to GIF". Or "Record to WebM".

What do you think?

[phw]

Yes, could be helpful. Maybe even a quick select. My only concern is that it will make the label longer. But since the label autohides anyway this does not matter much.

I am also no opposed to have some kind of quick switching, e.g. by having an arrow dropdown behind the button :)

[orschiro]

I am also no opposed to have some kind of quick switching, e.g. by having an arrow dropdown behind the button :)

Ooh, this sounds excitingly good! :-)

[gort818]

I have an idea for this, what about have a notify event for the record button and have a drop down menu with formats. I will test this out this weekend.

[gort818]

@phw @orschiro Here is a quick test, how does it look?
What do you guys think?

[orschiro]

@gort818 this looks perfect! How can I test? :-)

[gort818]

@orschiro It is in my test here https://github.com/gort818/peek/tree/format-popover.. But so far it is just graphical It won't do anything atm, give me some time and I will get it going and let you know when it is ready to test :)

[orschiro]

Sounds good. Thanks, Alessandro!

[gort818]

@orschiro ok check it out https://github.com/gort818/peek/tree/format-popover build that and let me know what you think.

[orschiro]

@gort818 followed the compile from source instructions [1].

But [2] produces an error:

orschiro@x230:/tmp/peek/build$ cmake -DCMAKE_INSTALL_PREFIX=/usr/local ..
-- Could NOT find Vala (missing: VALA_EXECUTABLE) 
CMake Error at /usr/share/cmake-3.9/Modules/FindPackageHandleStandardArgs.cmake:137 (message):
  Could NOT find Vala (missing: VALA_EXECUTABLE) (Required is at least
  version "0.22")
Call Stack (most recent call first):
  /usr/share/cmake-3.9/Modules/FindPackageHandleStandardArgs.cmake:377 (_FPHSA_FAILURE_MESSAGE)
  cmake/FindVala.cmake:65 (find_package_handle_standard_args)
  CMakeLists.txt:27 (find_package)


-- Configuring incomplete, errors occurred!
See also "/tmp/peek/build/CMakeFiles/CMakeOutput.log".

Can you help?

[1] https://github.com/gort818/peek/tree/format-popover#from-source
[2] cmake -DCMAKE_INSTALL_PREFIX=/usr/local ..

[gort818]

You need Vala installed.

[orschiro]

Thanks! It finished building peek but if I run it [1], I don't get to see your dropdown menu.

orschiro@x230:/tmp/peek/build$ ls -la
total 672
drwxr-xr-x  9 orschiro orschiro   4096 Okt 22 09:57 .
drwxr-xr-x 12 orschiro orschiro   4096 Okt 22 09:57 ..
-rw-r--r--  1 orschiro orschiro  39387 Okt 22 09:57 application.h
-rw-r--r--  1 orschiro orschiro  45297 Okt 22 09:57 application_internal.h
-rw-r--r--  1 orschiro orschiro  38235 Okt 22 09:57 CMakeCache.txt
drwxr-xr-x 36 orschiro orschiro   4096 Okt 22 09:57 CMakeFiles
-rw-r--r--  1 orschiro orschiro   2264 Okt 22 09:57 cmake_install.cmake
-rw-r--r--  1 orschiro orschiro   1196 Okt 22 09:57 cmake_uninstall.cmake
-rw-r--r--  1 orschiro orschiro   3592 Okt 22 09:57 CPackConfig.cmake
-rw-r--r--  1 orschiro orschiro   4026 Okt 22 09:57 CPackSourceConfig.cmake
-rw-r--r--  1 orschiro orschiro    284 Okt 22 09:57 CTestTestfile.cmake
-rw-r--r--  1 orschiro orschiro   2603 Okt 22 09:57 DartConfiguration.tcl
drwxr-xr-x  5 orschiro orschiro   4096 Okt 22 09:57 data
-rw-r--r--  1 orschiro orschiro  78498 Okt 22 09:57 Makefile
-rwxr-xr-x  1 orschiro orschiro 410800 Okt 22 09:57 peek
drwxr-xr-x  3 orschiro orschiro   4096 Okt 22 09:57 po
drwxr-xr-x  6 orschiro orschiro   4096 Okt 22 09:57 src
drwxr-xr-x  3 orschiro orschiro   4096 Okt 22 09:57 Testing
drwxr-xr-x  3 orschiro orschiro   4096 Okt 22 09:57 tests
drwxr-xr-x  2 orschiro orschiro   4096 Okt 22 09:57 ui

[1] orschiro@x230:/tmp/peek/build$ ./peek

[orschiro]

Hang on. I think I didn't check out the right branch...

Let me try again!

[orschiro]

@gort818 here is my feedback [1]. :-)

[1] https://www.useloom.com/share/ea24a461c6bf4eda887e1b2dd598bd91

[phw]

Hey, looking cool so far. This is going into the direction I had imagined.

Two notes:

1. I actually would like to have a way to still do a one-click record start. So we should somehow separate the start recording with default format, and format selection. I had thought of using a https://developer.gnome.org/gtk3/stable/GtkMenuToolButton.html for this.

2. Having the popover to be actual clickable is a bit tricky, due to the content area being set to clickthrough. Have a look at AppMenu.update_input_shape(), it already does this for the appmenu, if it is part of the window (https://github.com/phw/peek/blob/master/src/ui/application-window.vala#L404)

Having something like this in update_input_shape() should work:

if (pop_format.visible) {
  var pop_format_region = create_region_from_widget (pop_format);
  window_region.union (pop_format);
}