ravibpatel / CrashReporter.NET

Send crash reports of your classic desktop application developed using .NET Framework directly to your mail's inbox with full exception report, stack trace and screenshot.
MIT License
184 stars 74 forks source link

Screenshot not capturing correctly! #1

Closed StevenW1969 closed 3 years ago

StevenW1969 commented 7 years ago

Screenshot only captures a portion of the application on exception error. dsasdf

ravibpatel commented 7 years ago

did you set the CaptureScreen property to true? Like as shown in Readme.

https://github.com/ravibpatel/CrashReporter.NET#capture-whole-screen-instead-of-application-screen

StevenW1969 commented 7 years ago

Yes I did. I also noticed that it is only on my development workstation. I currently have [4] 28" Samsung 4K monitors at 3480 x 2160 resolution in an extended quadrant configuration so the total overall resolution is 6960 x 4320. I know this is not a standard configuration and is what is most likely the issue. I compiled my C# application and installed it on a regular workstation that has [4] 24" Asus HD monitors at 1920 x1200 resolution in an extended quadrant configuration so the total overall resolution is 3840 x 2400. The Screenshot was just fine on this machine. So I came to the conclusion that there must be a limit to the total resolution or that the 4K is an issue.

Our Engineering and development staff all have the HD resolution monitors, the 4K workstation is the new standard that we are changing all our workstations too.

The majority of our users for this application currently use the HD, but will soon be using 4K.

ravibpatel commented 7 years ago

Can you try the utility in below link? It saves a screenshot to the clipboard so you need to paste it into Paint or any other image manipulation program. If it works I can implement it in CrashReporter.NET.

https://www.dropbox.com/s/4kp50d8xk6rjxok/ScreenCapture_utility.zip?dl=0

StevenW1969 commented 7 years ago

Ravi, Thanks for getting back with me. I looked at your code a bit and seen that in the public class ReportCrash, line 126 try catch with if else you use the CaptureScreen before the CaptureActiveWindow. Doesn’t this 1st get a Screenshot of the whole screen before attempting to get the Active window? If so may I ask the reasoning for this, I’m curious. Not that it would probably makes much difference.

I downloaded the utility and tried it, but returned the same result.

[sig] Steve Williams (Product Design Engineer) 1725 Airpark Drive Grand Haven, MI 49417

O: 616.842.8760 x128 F: 800.842.0644 M: 616.335.4575 http://www.carterbearings.comhttp://www.carterbearings.com/ Engineering Hours are 6:00a-5:00p Mon-Fri EST

From: Ravi Patel [mailto:notifications@github.com] Sent: Wednesday, June 7, 2017 2:09 AM To: ravibpatel/CrashReporter.NET CrashReporter.NET@noreply.github.com Cc: Steve Williams stevew@carterbearings.com; Author author@noreply.github.com Subject: Re: [ravibpatel/CrashReporter.NET] Screenshot not capturing correctly! (#1)

Can you try the utility in below link? It saves a screenshot to the clipboard so you need to paste it into Paint or any other image manipulation program. If it works I can implement it in CrashReporter.NET.

https://www.dropbox.com/s/4kp50d8xk6rjxok/ScreenCapture_utility.zip?dl=0

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ravibpatel/CrashReporter.NET/issues/1#issuecomment-306696397, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZ4zEKSMFOgjUbtL9xoO3Rf41biXjXJWks5sBj6CgaJpZM4NwEwF.

ravibpatel commented 7 years ago

If you set CaptureScreen to true it will only call the CaptureScreen method of CaptureScreenshot class but if you don't set it then it will be false and it will skip the call to CaptureScreen and only call the CaptureActiveWindow which takes the screenshot and save it to the temp directory.

StevenW1969 commented 7 years ago

Ravi, Thanks for that, that is the information I was looking for. That helps! I only want a Active Window Shot not the whole screen.

[sig] Steve Williams (Product Design Engineer) 1725 Airpark Drive Grand Haven, MI 49417

O: 616.842.8760 x128 F: 800.842.0644 M: 616.335.4575 http://www.carterbearings.comhttp://www.carterbearings.com/ Engineering Hours are 6:00a-5:00p Mon-Fri EST

From: Ravi Patel [mailto:notifications@github.com] Sent: Wednesday, June 7, 2017 1:06 PM To: ravibpatel/CrashReporter.NET CrashReporter.NET@noreply.github.com Cc: Steve Williams stevew@carterbearings.com; Author author@noreply.github.com Subject: Re: [ravibpatel/CrashReporter.NET] Screenshot not capturing correctly! (#1)

If you set CaptureScreen to true it will only call the CaptureScreen method of CaptureScreenshot class but if you don't set it then it will be false and it will skip the call to CaptureScreen and only call the CaptureActiveWindow which takes the screenshot and save it to the temp directory.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ravibpatel/CrashReporter.NET/issues/1#issuecomment-306860479, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZ4zEJ0zZiCdHHheZUd8MRd7XyWswTpbks5sBthsgaJpZM4NwEwF.

ravibpatel commented 7 years ago

Just don't set the CaptureScreen to anything or set it to false and it will take only active window screenshot.

StevenW1969 commented 7 years ago

Lol, as I delete the this from my Program.cs file…… “reportCrash.CaptureScreen = false;”

[sig] Steve Williams (Product Design Engineer) 1725 Airpark Drive Grand Haven, MI 49417

O: 616.842.8760 x128 F: 800.842.0644 M: 616.335.4575 http://www.carterbearings.comhttp://www.carterbearings.com/ Engineering Hours are 6:00a-5:00p Mon-Fri EST

From: Ravi Patel [mailto:notifications@github.com] Sent: Wednesday, June 7, 2017 1:26 PM To: ravibpatel/CrashReporter.NET CrashReporter.NET@noreply.github.com Cc: Steve Williams stevew@carterbearings.com; Author author@noreply.github.com Subject: Re: [ravibpatel/CrashReporter.NET] Screenshot not capturing correctly! (#1)

Just don't set the CaptureScreen to anything or set it to false and it will take only active window screenshot.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHubhttps://github.com/ravibpatel/CrashReporter.NET/issues/1#issuecomment-306866295, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZ4zELlKroqfQ5jPyiFzWRCs2IVLFFOKks5sBt0mgaJpZM4NwEwF.

EddieDemon commented 4 years ago

Hi @StevenW1969, it's been quite some time since this issue was opened, but do you still encounter this problem? Or have you created a fix?

EddieDemon commented 3 years ago

@ravibpatel I suppose we could close this?

ravibpatel commented 3 years ago

@EddieDemon Yeah.