Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VS2012 project with 3 new features #44

Open
wants to merge 31 commits into
base: master
Choose a base branch
from

Conversation

andrasfuchs
Copy link

Hi!

I've made a fork from your project to help you with the work.

I needed to convert the project to the Visual Studio 2012, and I also needed to change some include and lib paths to make my copy complilable.

As you can see those changes are separated to different commits, I hope it will help you to separate them from the new feature implementations.

  • I updated the ffmpeg binaries to a much newer one.
  • I changed the codec to the x264rbg and I made a small modification to the configuration screen to make the user able to select a lossless compression if that's needed.
  • I also changed the source to support the following of the foreground window. So now you are free to change the window during playback, it always will record the foreground window if that is selected.

What do you think?

Update 2014-08-19:
I've made some new features today:

  • There are two new hotkeys:
    • Ctrl+Alt+W will freeze or release the current window for capturing. The idea is that you set the foreground window capture mode, hit start, go to the window you want to record, hit Ctrl+Alt+W, and from that on only that window will be recorded. If you want to switch back to "follow to foreground window" mode, hit Ctrl+Alt+W again.
    • Ctrl+Alt+B will display the position and size of the capture area (bounding box) during the recording.
  • The mouse was not visible in window capture mode, it's fixed
  • There was a misalignment between the bounding box and the window, so when you recorded a window not the proper part of it was recorded. It's fixed.

Solution and projects files were modified to make them compilabe without
installing the Windows SDKs's DirectShow sample. Lib and Includes paths
were modified too.
Modifications regarding the default Git settings.
ffmpeg update to 2014-06-09
Codec is replaced with x264rgb and there is an option to switch crf
value between 0 (lossless) and 1 (very good quality).
x64 solution modification
Add a new Windows event hook to capture the foreground window changed
event, in order to be able to capture the window which is currently in
the foreground at any given time.
Paths were fixed to reflect the new locations.
You can use the Ctrl+Alt+W keystroke to fix and release the window
handle of the recorded window. When you start recording, it is in the
released state, so you can switch between the windows and the SCR will
capture the foreground window. When you click Ctrl+Alt+W it will store
the window handle, so if you switch the foreground window to another
one, it will still record the previous window.
Refactored and reworked the rectangle handling of desktop, window and
bounding box objects. It also made the capture of the small windows
faster and resolved the "trailing" issue when a single window was
recorded which was smaller than the video resolution and it was moved
around.
In window capture mode the mouse disappeared. There was an error in the
AddMouse function.
New feature: when you hit Ctrl+Alt+W the active window will have a green
(released) or blue (freezed) border around it.
When you hit Ctrl+Alt+B, you will see where the bounding box (capture
area of the screen) is.
In window mode there was a small misalignment between the bounding box
and the real capture area. Now it is correctly show where the recording
will take place.
Removed some user-specific files
Remove batch files from Release and Debug folders
Merge x86 and x64 solutions into one.
Delete screen-capture-recorder.dll
Update to the gitignore file
Modified the .gitignore file to include some files which are needed for
compilation and testing.
Updated readme and installer files to 0.13.0
Replaced x64 ffmpeg (2014-06-09) with x86 (2014-08-24), because it
caused problems on 32-bit systems.
ChangeLog update
@rdp
Copy link
Owner

rdp commented Aug 25, 2014

I will definitely look at this once I get back into "open source" mode here...possibly 2 months or so, sorry about that!

The position of the mouse cursor was off in certain recording modes
Added new options for foreground window only recording (enable/disable,
capture mouse cursor and some description of the keystrokes)
@andrasfuchs
Copy link
Author

I fixed a problem with the mouse capture and extended the configuration utility with the new options.

@rdp
Copy link
Owner

rdp commented Oct 31, 2014

OK do you know exactly which commit did those two thing?

On Fri, Oct 31, 2014 at 1:24 PM, Andras Fuchs notifications@github.com
wrote:

I fixed a problem with the mouse capture and extended the configuration
utility with the new options.


Reply to this email directly or view it on GitHub
#44 (comment)
.

@andrasfuchs
Copy link
Author

Yes, you can see them next to the list of commits just above my previous comment.

Mouse cursor:
andrasfuchs@7a0d27b

New options in the config utility:
andrasfuchs@e263d7c

In the configuration utility we have the resizable window to select the
area to be recorded. This window now shows the resolution of itself in
the middle.
@andrasfuchs
Copy link
Author

And added a small extra function to the configuration utility to show the size of the area selection transparent window.

Commit:
andrasfuchs@76928b5

It looks like this: (the ninja is not included :))
screenshot 2014-11-01 10 14 55

@rdp
Copy link
Owner

rdp commented Nov 1, 2014

nice...now to find the time to integrate them...

On Sat, Nov 1, 2014 at 2:19 AM, Andras Fuchs notifications@github.com
wrote:

Yes, you can see them next to the list of commint just above my previous
comment.

Mouse cursor: (7a0d27b
7a0d27b
)
andrasfuchs/screen-capture-recorder-to-video-windows-free@7a0d27b
andrasfuchs@7a0d27b

New options in the config utility: (e263d7c
e263d7c
)
andrasfuchs/screen-capture-recorder-to-video-windows-free@e263d7c
andrasfuchs@e263d7c


Reply to this email directly or view it on GitHub
#44 (comment)
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants