config option to disable grim warrning

[Monirzadeh]

This PR attempts to fix annoying notification about Grim
close #3186
@mmahmoudian Should I add anything else? or any opinions?

[Monirzadeh]

@mmahmoudian can you please review this one?

[mmahmoudian]

Thanks for the PR. As an internal policy, I generally refrain from merging code changes before it is also approved by other devs. I asked other devs to review this issue on our Matrix channel (which you are also welcome to join). I will also tag them here to increase visibility:

@veracioux @panpuchkov @jack9603301

[jack9603301]

Thanks to your PR, this PR doesn't seem to have any side effects. Instead, it will no longer output a grim warning message when the user wants to cancel the grim warning. Since this PR does not introduce any by-products, I will agree with this PR change

[b0ch3nski]

There is a typo across all the code: Warnning -> Warning

[Monirzadeh]

Thanks @b0ch3nski i fix that.

[jack9603301]

Well done, since this PR does not introduce any side effects, I will approve it

[mmahmoudian]

@Monirzadeh thanks for the PR. Would you please handle the clang format. I'll merge when that passes. 👍

clang-format -i $(git ls-files "*.cpp" "*.h")

Source: https://github.com/flameshot-org/flameshot/blob/master/docs/CONTRIBUTING.md#code

[mmahmoudian]

@Monirzadeh thanks for the PR and clean code.
@jack9603301 thanks for the code review.

[killertofus]

I still see this

dwl, arch flameshot-git

[Monirzadeh]

can you share flameshot.ini ?

[killertofus]

@Monirzadeh

[General]
contrastOpacity=188
drawColor=#800000
savePath=/home/killertofus/Pictures
uploadWithoutConfirmation=true 

[Monirzadeh]

@killertofus
please add this to the config.

disabledGrimWarning=true

dose it solve the problem?

[killertofus]

yes @Monirzadeh