merge ... and + buttons for more tags in sidebar (publiclab#8386)

* merge ... and + buttons for more tags in sidebar * Update _tags.html.erb * revert test changes from publiclab#7715
manchere · Feb 13, 2021 · 75d98b2 · 75d98b2
1 parent 0ac135d
commit 75d98b2
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/app/views/tag/_tagging.html.erb b/app/views/tag/_tagging.html.erb
@@ -60,6 +60,8 @@ $(function () {
   $("#tags-open").click(function(){
     $("#tagform").show();
     $("#tags-open").hide();
+    $('.more-tags').show();
+    $('.show-more-tags').hide();
     $(".tag-input").focus();
   });
 </script>

diff --git a/app/views/tag/_tags.html.erb b/app/views/tag/_tags.html.erb
@@ -32,6 +32,9 @@
         $('.show-more-tags').click(function showMoreTags() {
           $('.more-tags').show();
           $('.show-more-tags').hide();
+          $("#tagform").show();
+          $("#tags-open").hide();
+          $(".tag-input").focus();
         });
       })();
     </script>

diff --git a/test/system/post_test.rb b/test/system/post_test.rb
@@ -50,11 +50,11 @@ def setup
 
     # There should be 1 tag that shows up as a badge and 2 as a card
     page.assert_selector('.tags-list .card-body', :count => 2)
-    # page.assert_selector('.tags-list p.badge', :count => 1)
+    page.assert_selector('.tags-list p.badge', :count => 1)
 
-    # accept_alert do
-    #   find('.tags-list p.badge .tag-delete').click()
-    # end
+    accept_alert do
+      find('.tags-list p.badge .tag-delete').click()
+    end
 
     # Make sure that 1 of the 3 tags is removed
     page.assert_selector('.tags-list p.badge', :count => 0)