From 1738a85bdc8187a93f1aaca5d499d1436b2cb62f Mon Sep 17 00:00:00 2001 From: ckeboss Date: Thu, 18 Jan 2018 10:53:05 -0800 Subject: [PATCH 1/2] Update post-menu.js.es6 The bookmark ID's were not very clear and not using the same names as the classes. This addresses that. --- app/assets/javascripts/discourse/widgets/post-menu.js.es6 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/widgets/post-menu.js.es6 b/app/assets/javascripts/discourse/widgets/post-menu.js.es6 index 6186ae4e72..88bb08be26 100644 --- a/app/assets/javascripts/discourse/widgets/post-menu.js.es6 +++ b/app/assets/javascripts/discourse/widgets/post-menu.js.es6 @@ -194,7 +194,7 @@ registerButton('bookmark', attrs => { } return { - id: attrs.bookmarked ? 'bookmark' : 'unbookmark', + id: attrs.bookmarked ? 'bookmarked' : 'unbookmarked', action: 'toggleBookmark', title: attrs.bookmarked ? "bookmarks.created" : "bookmarks.not_bookmarked", className, From beee69e566eb15b1c184772db62a66f11cf393ba Mon Sep 17 00:00:00 2001 From: ckeboss Date: Thu, 18 Jan 2018 11:05:25 -0800 Subject: [PATCH 2/2] Update tense and change actions around. --- app/assets/javascripts/discourse/widgets/post-menu.js.es6 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/assets/javascripts/discourse/widgets/post-menu.js.es6 b/app/assets/javascripts/discourse/widgets/post-menu.js.es6 index 88bb08be26..46b9819457 100644 --- a/app/assets/javascripts/discourse/widgets/post-menu.js.es6 +++ b/app/assets/javascripts/discourse/widgets/post-menu.js.es6 @@ -194,7 +194,7 @@ registerButton('bookmark', attrs => { } return { - id: attrs.bookmarked ? 'bookmarked' : 'unbookmarked', + id: attrs.bookmarked ? 'unbookmark' : 'bookmark', action: 'toggleBookmark', title: attrs.bookmarked ? "bookmarks.created" : "bookmarks.not_bookmarked", className,