From c7d188147210c8b09bf8e6b785054be41e0e8043 Mon Sep 17 00:00:00 2001 From: Luke Melia Date: Tue, 6 Jan 2009 00:49:40 -0500 Subject: [PATCH 1/2] Restore automatic focus of first todo form field after clicking edit icon to edit. --- app/views/todos/edit.js.rjs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/views/todos/edit.js.rjs b/app/views/todos/edit.js.rjs index 9f6da060..9e57e85c 100644 --- a/app/views/todos/edit.js.rjs +++ b/app/views/todos/edit.js.rjs @@ -1,4 +1,4 @@ page[dom_id(@todo, 'form')].down('.placeholder').replace_html :partial => 'todos/edit_form' page[dom_id(@todo, 'line')].hide page[dom_id(@todo, 'edit')].show -page[dom_id(@todo, 'form')].down('table').down('input').focus \ No newline at end of file +page[dom_id(@todo, 'form')].down('input#todo_description').focus \ No newline at end of file From 00eff7e8778406d6fb2d9f2af7a5a472c57b3cd6 Mon Sep 17 00:00:00 2001 From: Luke Melia Date: Tue, 6 Jan 2009 01:00:06 -0500 Subject: [PATCH 2/2] Fixed bug where adding tag to Todo via Ajax inline edit didn't reflect tag changes immediately. --- app/controllers/todos_controller.rb | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/app/controllers/todos_controller.rb b/app/controllers/todos_controller.rb index ff5b767f..cf36bfaa 100644 --- a/app/controllers/todos_controller.rb +++ b/app/controllers/todos_controller.rb @@ -175,7 +175,10 @@ class TodosController < ApplicationController def update @source_view = params['_source_view'] || 'todo' init_data_for_sidebar unless mobile? - @todo.tag_with(params[:tag_list]) if params[:tag_list] + if params[:tag_list] + @todo.tag_with(params[:tag_list]) + @todo.tags(true) #force a reload for proper rendering + end @original_item_context_id = @todo.context_id @original_item_project_id = @todo.project_id @original_item_was_deferred = @todo.deferred?