From 044d43c18cf21a26697da4014b8cf5093256b945 Mon Sep 17 00:00:00 2001
From: jordanjay29 <jordanschnaidt@gmail.com>
Date: Sun, 15 May 2016 21:48:56 -0500
Subject: [PATCH] Change fa-times to fa-cancel

Mitigates confusion over button's immediate action, whether it will dismiss flag or delete post. In isolation, fa-times looks ambiguous enough to delete the post, which may delay a moderator's action when choosing how to respond. By replacing it with fa-ban, it should be immediately obvious that the action taken by the button will dismiss the flag, and not modify the post at all.
---
 extensions/flags/js/forum/src/addFlagsToPosts.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/extensions/flags/js/forum/src/addFlagsToPosts.js b/extensions/flags/js/forum/src/addFlagsToPosts.js
index a9613c5e8..e968c44f4 100644
--- a/extensions/flags/js/forum/src/addFlagsToPosts.js
+++ b/extensions/flags/js/forum/src/addFlagsToPosts.js
@@ -65,7 +65,7 @@ export default function() {
 
     items.merge(controls);
 
-    items.add('dismiss', <Button className="Button Button--icon Button--link" icon="times" onclick={this.dismissFlag.bind(this)} title={app.translator.trans('flarum-flags.forum.post.dismiss_flag_tooltip')}/>, -100);
+    items.add('dismiss', <Button className="Button Button--icon Button--link" icon="ban" onclick={this.dismissFlag.bind(this)} title={app.translator.trans('flarum-flags.forum.post.dismiss_flag_tooltip')}/>, -100);
 
     return items;
   };