SwordPhish icon indicating copy to clipboard operation
SwordPhish copied to clipboard

A confirmation dialog for reports

Open znb opened this issue 8 years ago • 8 comments

Just some feedback from our user base.

We have had a few false positives being reported because people are clicking the wrong button. It would be handy to have a confirmation dialog pop up after clicking the submit button.

It's a nice to have for me, but users would probably celebrate.

znb avatar Jan 31 '18 17:01 znb

Sounds like a sensible suggestion. I’ll see what I can do.

Thanks for the feedback.

eth0izzle avatar Jan 31 '18 17:01 eth0izzle

Hi , I try the button and recived a few false positives also . I think that a confirmation dialog pop up it's good idea . And also maybe pin the button in the end of ribbon , because it's close to delete button . Thank you .

oded86w avatar Feb 18 '18 09:02 oded86w

@oded86w thanks for the feedback. Are you finding they are hitting the button accidentally or are reporting the wrong e-mails? i.e spam is being reported a phishing.

eth0izzle avatar Feb 19 '18 11:02 eth0izzle

I think from both cases . But the most is accidentally .

oded86w avatar Feb 19 '18 14:02 oded86w

I think it be nice to move the button to end . and maybe the logo is black and look similar to delete (X) .

oded86w avatar Feb 19 '18 14:02 oded86w

You can change the position of the button under "Options > Customize Ribbon", but I'm not sure how you deploy this. I'll add the option for a confirmation dialog.

eth0izzle avatar Feb 19 '18 14:02 eth0izzle

I know how to change in my outlook but change in few computer it's not comfortable . you can change in code in ribbon xml : insertBeforeMso="GroupMailDelete" and change it ?

"groupMailSwordPhish" insertAfterMso="GroupMailSearch" getLabel="GetGroupLabel"

"groupReadSwordPhish" insertAfterMso="GroupMailEdit" getLabel="GetGroupLabel"

I think it be nice :)

oded86w avatar Feb 19 '18 14:02 oded86w

We had the same need so we added the following code to Ribbon.cs

        public void ReportMailItem(IRibbonControl control)
        {
            //added message box confirmation
            DialogResult confirmation = MessageBox.Show("Are you sure you would like to report this email as phishing?", "Phishing Report Confirmation", MessageBoxButtons.YesNo);
            if (confirmation == DialogResult.Yes)
            {
                Globals.ThisAddIn.SendReport(GetMailItemBasedOnControl(control));
                ribbon.InvalidateControl(control.Id);
            }
        }

thomasbrewster avatar Oct 26 '18 20:10 thomasbrewster