Remove data-turbolinks-action from user name tag

Author: huacnlee

app/assets/stylesheets/application.scss

@@ -44,7 +44,7 @@ textarea,
 body {
   background: #e9eaed;
   color: $black;
-  font-family: Helvetica,Arial,"PingFang SC","Hiragino Sans GB","Source Han Sans CN",Roboto,"Heiti SC","Microsoft Yahei",sans-serif;
+  font-family: Helvetica, Arial,"PingFang SC","Hiragino Sans GB","Source Han Sans CN",Roboto,"Heiti SC","Microsoft Yahei",sans-serif;
   padding-top: 65px;
 }
 

app/helpers/users_helper.rb

@@ -19,7 +19,6 @@ def user_name_tag(user, options = {})
     name ||= login
     options[:class] ||= "#{user_type}-name"
     options['data-name'.freeze] = name
-    options['data-turbolinks-action'] = 'replace'
 
     link_to(login, main_app.user_path(user), options)
   end
@@ -52,7 +51,6 @@ def user_avatar_tag(user, version = :md, opts = {})
       end
 
     options = {}
-    options['data-turbolinks-action'] = 'replace'
 
     if opts[:link] != false
       link_to(raw(img), user_path(user), options)

spec/helpers/users_helper_spec.rb

@@ -14,17 +14,17 @@
   describe 'user_name_tag' do
     it 'should result right html in normal' do
       user = build(:user)
-      expect(helper.user_name_tag(user)).to eq(link_to(user.login, user_path(user.login), class: 'user-name', 'data-name' => user.name, 'data-turbolinks-action' => 'replace'))
+      expect(helper.user_name_tag(user)).to eq(link_to(user.login, user_path(user.login), class: 'user-name', 'data-name' => user.name))
     end
 
     it 'should result right html with string param and downcase url' do
       login = 'Monster'
-      expect(helper.user_name_tag(login)).to eq(link_to(login, user_path(login), class: 'user-name', 'data-name' => login, 'data-turbolinks-action' => 'replace'))
+      expect(helper.user_name_tag(login)).to eq(link_to(login, user_path(login), class: 'user-name', 'data-name' => login))
     end
 
     it 'should out name with Team' do
       user = build(:team)
-      expect(helper.user_name_tag(user)).to eq(link_to(user.name, user_path(user.login), class: 'team-name', 'data-name' => user.name, 'data-turbolinks-action' => 'replace'))
+      expect(helper.user_name_tag(user)).to eq(link_to(user.name, user_path(user.login), class: 'team-name', 'data-name' => user.name))
     end
 
     it 'should result empty with nil param' do
@@ -40,14 +40,14 @@
     it 'should work if user exists' do
       user = create(:user)
       img = image_tag(user.letter_avatar_url(96), class: 'media-object avatar-48')
-      expect(user_avatar_tag(user)).to eq link_to(raw(img), user_path(user), 'data-turbolinks-action' => 'replace')
+      expect(user_avatar_tag(user)).to eq link_to(raw(img), user_path(user))
     end
 
     it 'should work if avatar exist' do
       user = create(:avatar_user)
       image_url = user.avatar.url(:md)
       img = image_tag(image_url, class: 'media-object avatar-48')
-      expect(user_avatar_tag(user)).to eq link_to(raw(img), user_path(user), 'data-turbolinks-action' => 'replace')
+      expect(user_avatar_tag(user)).to eq link_to(raw(img), user_path(user))
     end
 
     it 'should work with different size' do
@@ -58,7 +58,7 @@
       user = create(:avatar_user)
       image_url = user.avatar.url(:md) + "?t=#{user.updated_at.to_i}"
       img = image_tag(image_url, class: 'media-object avatar-48')
-      expect(user_avatar_tag(user, :md, timestamp: true)).to eq link_to(raw(img), user_path(user), 'data-turbolinks-action' => 'replace')
+      expect(user_avatar_tag(user, :md, timestamp: true)).to eq link_to(raw(img), user_path(user))
     end
 
     it 'should work if link is false' do