From 1e0f1bd611d0ce2e356d9f975d3a09ef95e17ec6 Mon Sep 17 00:00:00 2001 From: Jyri-Petteri Paloposki Date: Wed, 22 May 2019 01:28:50 +0300 Subject: [PATCH] Change context element IDs to use underscore instead of just the c prefix --- app/assets/javascripts/tracks.js.erb | 2 +- app/helpers/todos_helper.rb | 2 +- app/views/contexts/_context.html.erb | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/assets/javascripts/tracks.js.erb b/app/assets/javascripts/tracks.js.erb index 80813d5f..02da5979 100644 --- a/app/assets/javascripts/tracks.js.erb +++ b/app/assets/javascripts/tracks.js.erb @@ -644,7 +644,7 @@ var ContextListPage = { } }, save_context_name: function(value, settings) { - var context_id = $(this).parents('.container.context').get(0).id.split('c')[1]; + var context_id = $(this).parents('.container.context').get(0).id.split('_')[1]; var highlight = function(){ $('div.context span#context_name').effect('highlight', {}, 500); }; diff --git a/app/helpers/todos_helper.rb b/app/helpers/todos_helper.rb index 6f08c9f7..a2291bdb 100644 --- a/app/helpers/todos_helper.rb +++ b/app/helpers/todos_helper.rb @@ -118,7 +118,7 @@ module TodosHelper toggle = settings[:collapsible] ? container_toggle("toggle_#{settings[:id]}") : "" "#{toggle} #{settings[:title]} #{settings[:append_descriptor]}".html_safe end - header += content_tag(:div, nil, :id =>"c_"+settings[:id]+"_target", :class=>"context_target drop_target") + header += content_tag(:div, nil, :id => settings[:id] + "_target", :class => "context_target drop_target") header.html_safe end diff --git a/app/views/contexts/_context.html.erb b/app/views/contexts/_context.html.erb index 1263d671..a0921254 100644 --- a/app/views/contexts/_context.html.erb +++ b/app/views/contexts/_context.html.erb @@ -8,7 +8,7 @@ cache [context, source_view_key, current_user.date.strftime("%Y%m%d"), @tag_name render :partial => 'todos/collection', :object => @not_done, :locals => { :settings => { - :id => "c#{context.id}", + :id => "context_#{context.id}", :collapsible => settings[:collapsible], :container_name => 'context', :title => show_context_name(context),