Add the possibility to disable Screenshot

Feb 20, 2014 at 3:24 PM
Edited Feb 20, 2014 at 3:24 PM
Hi,

Your tool is great !

But I do not want users to send me screenshots:
  • They are huge, sending the report become too long.
  • It's look like user did not know that a screenshot is taken.
I made a patch to introduce two parameters in "ReportCrash.cs":
        /// <summary>
        /// Gets or Sets if screenshot is enable.
        /// </summary>
        public bool ScreenshotEnable { get; set; }

        /// <summary>
        /// Gets or Sets "Include screenshot" start value.
        /// </summary>
        public bool IncludeScreenshot { get; set; }
Here the patch (against revision 27634):
see it:
http://pastealacon.com/34019
download it:
http://pastealacon.com/pastebin.php?dl=34019
Coordinator
Feb 21, 2014 at 5:15 AM
User can decide if he/she wants to send the screenshot or not on screenshot tab using Include Screenshot tab. You can disable that by unchecking Include screenshot check box as default.
Feb 21, 2014 at 9:00 AM
Edited Feb 21, 2014 at 9:11 AM
Hi Ravi,

I did not see how a developer using crashreporter library can unchecks "include screenshot" check box by default without editing the code of the library.

When a developer uses crashreporter library, he has only access to "ReportCrash" class.
Actually there is no parameters in this class to set the default status of the "include screenshot" check box.

My patch add this parameter. I named it "IncludeScreenshot" (I'm ok with any other possible name).

I hope you see my point in my explanation.

I like this tool and it will be very cool if I can use "as it" without edit the code by my side.

Regards,
Pierre.

PS: I do my best but my english is still bad. sorry for that.
Coordinator
Feb 23, 2014 at 5:57 AM
Yeah you are right. There is no way to do that without touching the source code. So i will implement the feature in next version. Thanks for the feature suggestion.