diff --git a/app/views/projects/update_default_context.js.rjs b/app/views/projects/update_default_context.js.rjs index 85f2513d..3a7d5dc9 100644 --- a/app/views/projects/update_default_context.js.rjs +++ b/app/views/projects/update_default_context.js.rjs @@ -1,3 +1,5 @@ +# TODO: is this DEAD code? + if @project.default_context.nil? page.notify :notice, t('projects.default_context_removed'), 5.0 else diff --git a/app/views/projects/update_default_tags.js.rjs b/app/views/projects/update_default_tags.js.rjs index 284c9c5d..2400a3a4 100644 --- a/app/views/projects/update_default_tags.js.rjs +++ b/app/views/projects/update_default_tags.js.rjs @@ -1,3 +1,4 @@ +# TODO: is this dead code? if @project.default_tags.nil? page.notify :notice, t('projects.default_tags_removed_notice'), 5.0 else diff --git a/app/views/projects/update_project_name.js.rjs b/app/views/projects/update_project_name.js.rjs index a9bf2d32..19738ee1 100644 --- a/app/views/projects/update_project_name.js.rjs +++ b/app/views/projects/update_project_name.js.rjs @@ -1,3 +1,4 @@ +#TODO: is this dead code? page['default_project_name_id'].value = @project.name page['todo_project_name'].value = @project.name diff --git a/app/views/projects/update_status.js.rjs b/app/views/projects/update_status.js.rjs index 28e04a1f..f2570dd4 100644 --- a/app/views/projects/update_status.js.rjs +++ b/app/views/projects/update_status.js.rjs @@ -1,3 +1,4 @@ +# TODO: is this dead code? page.select('#project_status .active span').each do |element| element.className = @project.current_state == :active ? 'active_state' : 'inactive_state' end diff --git a/app/views/recurring_todos/toggle_star.js.erb b/app/views/recurring_todos/toggle_star.js.erb index ff226546..85eb1d66 100644 --- a/app/views/recurring_todos/toggle_star.js.erb +++ b/app/views/recurring_todos/toggle_star.js.erb @@ -1,4 +1,4 @@ -<% if @saved -%> +<%- if @saved -%> $('div#recurring_todo_<%= @recurring_todo.id %> a.star_item img').toggleClass('starred_todo').toggleClass('unstarred_todo'); <%- else -%> TracksPages.page_notify('error', '<%= t('todos.error_starring_recurring', :description => @recurring_todo.description) %>', 8); diff --git a/app/views/todos/toggle_star.js.erb b/app/views/todos/toggle_star.js.erb index e760f8e2..9ec02520 100644 --- a/app/views/todos/toggle_star.js.erb +++ b/app/views/todos/toggle_star.js.erb @@ -1,5 +1,5 @@ -<% if @saved -%> +<%- if @saved -%> $('div#line_todo_<%= @todo.id %> a.star_item img').toggleClass('starred_todo').toggleClass('unstarred_todo'); -<% else -%> - TracksPages.page_notify('error', "Could not toggle the star of this todo", 5); -<% end -%> +<%- else -%> + TracksPages.page_notify('error', '<%= t('todos.error_starring', :description => @todo.description) %>', 8); +<%- end -%> diff --git a/config/locales/en.yml b/config/locales/en.yml index 28c1e8f6..7c0b38b8 100644 --- a/config/locales/en.yml +++ b/config/locales/en.yml @@ -394,6 +394,7 @@ en: action_saved_to_tickler: "Action saved to tickler" added_new_project: "Added new project" added_new_context: "Added new context" + error_starring: "Could not toggle the star of this todo \'%{description}\'" recurrence_completed: "There is no next action after the recurring action you just finished. The recurrence is completed" tagged_with: "tagged with ‘%{tag_name}’" no_actions_found: "No actions found" diff --git a/public/images/ui-anim_basic_16x16.gif b/public/images/ui-anim_basic_16x16.gif new file mode 100644 index 00000000..084ecb87 Binary files /dev/null and b/public/images/ui-anim_basic_16x16.gif differ diff --git a/public/javascripts/application.js b/public/javascripts/application.js index 827944e0..0b32a7e6 100644 --- a/public/javascripts/application.js +++ b/public/javascripts/application.js @@ -1,9 +1,9 @@ var TracksForm = { toggle: function(toggleLinkId, formContainerId, formId, hideLinkText, hideLinkTitle, showLinkText, showLinkTitle) { - form=$('#'+formContainerId) + var form=$('#'+formContainerId) form.toggle(); - toggleLink = $('#'+toggleLinkId); + var toggleLink = $('#'+toggleLinkId); if (!form.is(':visible')) { toggleLink.text(showLinkText).attr('title', showLinkTitle); } @@ -74,41 +74,6 @@ var TracksForm = { $(this).prev().val(''); }); - /* behavior for delete icon. DEAD CODE??? */ - $('.item-container a.delete_icon').live('click', function(evt){ - evt.preventDefault(); - params = {}; - if(typeof(TAG_NAME) !== 'undefined'){ - params._tag_name = TAG_NAME; - } - confirm_message = $(this).attr("x_confirm_message") - if(confirm(confirm_message)){ - itemContainer = $(this).parents(".item-container"); - itemContainer.block({ - message: null - }); - params._method = 'delete'; - $.post(this.href, params, function(){ - itemContainer.unblock(); - }, 'script'); - } - }); - - /* behavior for edit icon DEAD CODE???*/ - $('.item-container a.edit_icon').live('click', function(evt){ - evt.preventDefault(); - params = {}; - if(typeof(TAG_NAME) !== 'undefined'){ - params._tag_name = TAG_NAME; - } - itemContainer = $(this).parents(".item-container"); - $(this).effect('pulsate', { - times: 1 - }, 800); - $.get(this.href, params, function(){ - }, 'script'); - }); - /* submit todo form after entering new todo */ $("button#todo_new_action_submit").live('click', function (ev) { if (TodoItems.askIfNewContextProvided('', this)) @@ -156,7 +121,7 @@ var TracksPages = { Nifty("div#project_new_project_container","normal"); }, page_notify: function(type, message, fade_duration_in_sec) { - flash = $('h4#flash'); + var flash = $('h4#flash'); flash.html("

"+message+"

"); flash = $('h4#flash'); flash.show(); @@ -205,7 +170,7 @@ var TracksPages = { $(".alert").fadeOut(8000); /* for edit project form and edit todo form - * TODO: refactor to separate calls from project and todo */ +* TODO: refactor to separate calls from project and todo */ $('.edit-form a.negative').live('click', function(){ $(this).parents('.edit-form').fadeOut(200, function () { $(this).parents('.list').find('.project').fadeIn(500); @@ -253,10 +218,10 @@ var TodoItemsContainer = { setup_container_toggles: function(){ // bind handlers $('.container_toggle').click(function(evt){ - toggle_target = $(this.parentNode.parentNode).find('.toggle_target'); + var toggle_target = $(this.parentNode.parentNode).find('.toggle_target'); if(toggle_target.is(':visible')){ // hide it - imgSrc = $(this).find('img').attr('src'); + var imgSrc = $(this).find('img').attr('src'); $(this).find('img').attr('src', imgSrc.replace('collapse', 'expand')); $.cookie(TodoItemsContainer.buildCookieName(this.parentNode.parentNode), true); toggle_target.slideUp(500); @@ -272,7 +237,7 @@ var TodoItemsContainer = { // set to cookied state $('.container.context').each(function(){ if($.cookie(TodoItemsContainer.buildCookieName(this))=="true"){ - imgSrc = $(this).find('.container_toggle img').attr('src'); + var imgSrc = $(this).find('.container_toggle img').attr('src'); if (imgSrc) { $(this).find('.container_toggle img').attr('src', imgSrc.replace('collapse', 'expand')); $(this).find('.toggle_target').hide(); @@ -283,15 +248,15 @@ var TodoItemsContainer = { // private buildCookieName: function(containerElem) { - tracks_login = $.cookie('tracks_login'); + var tracks_login = $.cookie('tracks_login'); return 'tracks_'+tracks_login+'_context_' + containerElem.id + '_collapsed'; }, showContainer: function(containerElem) { - imgSrc = $(containerElem).find('.container_toggle img').attr('src'); + var imgSrc = $(containerElem).find('.container_toggle img').attr('src'); $(containerElem).find('.container_toggle img').attr('src', imgSrc.replace('expand', 'collapse')); }, hideContainer: function (containerElem) { - imgSrc = $(containerElem).find('.container_toggle img').attr('src'); + var imgSrc = $(containerElem).find('.container_toggle img').attr('src'); $(containerElem).find('.container_toggle img').attr('src', imgSrc.replace('collapse', 'expand')); } } @@ -299,7 +264,7 @@ var TodoItemsContainer = { var TodoItems = { getContextsForAutocomplete: function (term, element_to_block) { var allContexts = null; - params = default_ajax_options_for_scripts('GET', relative_to_root('contexts.autocomplete'), element_to_block); + var params = default_ajax_options_for_scripts('GET', relative_to_root('contexts.autocomplete'), element_to_block); params.data = "term="+term; params.dataType = "json"; params.async = false; @@ -313,10 +278,10 @@ var TodoItems = { var givenContextName = $('#'+source+'todo_context_name').val(); if (givenContextName.length == 0) return true; // do nothing and depend on rails validation error - contexts = TodoItems.getContextsForAutocomplete(givenContextName, element_to_block); + var contexts = TodoItems.getContextsForAutocomplete(givenContextName, element_to_block); if (contexts) { - for (i=0; i