From 5812bbd1f13c99f4f316cc446e6c2781497dcaa2 Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Thu, 9 Sep 2010 09:35:32 -0700 Subject: [PATCH 01/11] removed group name edit madness from javascript. --- public/javascripts/view.js | 46 -------------------------------------- 1 file changed, 46 deletions(-) diff --git a/public/javascripts/view.js b/public/javascripts/view.js index 7d096e52b..fbe84aac8 100644 --- a/public/javascripts/view.js +++ b/public/javascripts/view.js @@ -24,54 +24,8 @@ $(document).ready(function(){ $(".image_cycle img").load( function() { $(this).fadeIn("slow"); }); - - - $(".edit_group_button").click(function() { - - var element = $(this).closest("li").children(".group_name").children("a"); - var oldHTML = element.html(); - - var link = element.attr("href"); - - element.toggleClass("editing"); - - if( element.hasClass("editing") ) { - element.attr("contentEditable", true); - element.focus(); - - //remove newline action - $(element).keypress(function(e) { - if (e.which == 13) { - e.preventDefault(); - element.attr("contentEditable", false); - element.toggleClass("editing"); - element.blur(); - - //save changes - $.ajax({ - type: "PUT", - url: link, - data: {"group" : {"name" : element.text() }} - }); - } - }); - - //update all other group links - $(element).keyup(function(e) { - $("a[href='"+link+"']").not(element).text(element.text()); - }); - - } else { - element.attr("contentEditable", false); - } - - }); - - });//end document ready - - function pane_toggler_button( name ) { $("#add_" + name + "_button").toggle( From f70ade67054a3eab31f1ed60bd3d118e22811cd6 Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 13:48:35 -0700 Subject: [PATCH 02/11] DG IZ; moved info on the person show page --- app/views/albums/index.html.haml | 3 +++ app/views/albums/show.html.haml | 3 +++ app/views/groups/index.html.haml | 3 +++ app/views/groups/show.html.haml | 3 +++ app/views/layouts/application.html.haml | 3 ++- app/views/people/show.html.haml | 31 +++++++++---------------- app/views/photos/show.html.haml | 3 +++ 7 files changed, 28 insertions(+), 21 deletions(-) diff --git a/app/views/albums/index.html.haml b/app/views/albums/index.html.haml index 87902a523..2be4bace9 100644 --- a/app/views/albums/index.html.haml +++ b/app/views/albums/index.html.haml @@ -7,6 +7,9 @@ = content_for :page_title do = link_to "Albums", albums_path +- content_for :left_pane do + = render "shared/group_friends" + %h1.big_text Albums .right diff --git a/app/views/albums/show.html.haml b/app/views/albums/show.html.haml index 5e6e64817..3d24b1c09 100644 --- a/app/views/albums/show.html.haml +++ b/app/views/albums/show.html.haml @@ -8,6 +8,9 @@ = content_for :page_title do = link_to @album.name, @album +- content_for :left_pane do + = render "shared/group_friends" + .album_id{:id => @album.id, :style => "display:hidden;"} diff --git a/app/views/groups/index.html.haml b/app/views/groups/index.html.haml index e4e18829e..d41f596e1 100644 --- a/app/views/groups/index.html.haml +++ b/app/views/groups/index.html.haml @@ -1,6 +1,9 @@ - content_for :page_title do Home +- content_for :left_pane do + = render "shared/group_friends" + %ul#stream - for post in @posts = render type_partial(post), :post => post unless post.class == Album diff --git a/app/views/groups/show.html.haml b/app/views/groups/show.html.haml index 822a33ef4..4163d118d 100644 --- a/app/views/groups/show.html.haml +++ b/app/views/groups/show.html.haml @@ -1,6 +1,9 @@ - content_for :page_title do Home +- content_for :left_pane do + = render "shared/group_friends" + %ul#stream - for post in @posts = render type_partial(post), :post => post diff --git a/app/views/layouts/application.html.haml b/app/views/layouts/application.html.haml index 3c70d3a8a..dc687861f 100644 --- a/app/views/layouts/application.html.haml +++ b/app/views/layouts/application.html.haml @@ -73,7 +73,8 @@ .container .span-5.last - = render "shared/group_friends" + + = yield :left_pane .span-19.last = yield diff --git a/app/views/people/show.html.haml b/app/views/people/show.html.haml index 4d75df57e..acd52eef3 100644 --- a/app/views/people/show.html.haml +++ b/app/views/people/show.html.haml @@ -1,14 +1,10 @@ -.span-20.last - #profile.span-19.last +- content_for :page_title do + = @person.real_name + +- content_for :left_pane do + #profile .profile_photo = person_image_link(@person) - - %h1 - = @person.real_name - - if @person != current_user.person && current_user.friends.include?(@person) - .right - = link_to 'remove friend', @person, :confirm => 'Are you sure?', :method => :delete, :class => "button" - %ul -unless @posts.first.nil? %li @@ -22,19 +18,14 @@ = hidden_field_tag :from, :from, :value => @groups_with_person.first.id = hidden_field_tag :friend_id, :friend_id, :value => @person.id = submit_tag "save" - %li - url: - = @person.url - - - unless @latest_status_message.nil? - %h1.pinched - %span - = "\"#{@latest_status_message.message}\"" - %span="posted: #{how_long_ago(@latest_status_message)}" + + - if @person != current_user.person && current_user.friends.include?(@person) + = link_to 'remove friend', @person, :confirm => 'Are you sure?', :method => :delete, :class => "button" + +.span-20.last - .span-20.last + .span-19.last - if @posts - %h3= "stream - #{@post_count} item(s)" %ul#stream - for post in @posts = render type_partial(post), :post => post unless post.class == Album diff --git a/app/views/photos/show.html.haml b/app/views/photos/show.html.haml index 51c7234f5..d582bc095 100644 --- a/app/views/photos/show.html.haml +++ b/app/views/photos/show.html.haml @@ -13,6 +13,9 @@ = content_for :page_title do = link_to "Photo", @photo +- content_for :left_pane do + = render "shared/group_friends" + %h1.big_text = @photo.image From 136eb61b023110e8a57144019a196503132ce84d Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 14:52:11 -0700 Subject: [PATCH 03/11] DG IZ; groups = relations. now you can rename groups in the manage relation page --- app/views/groups/edit.html.haml | 12 +++++++---- app/views/layouts/application.html.haml | 3 +-- app/views/shared/_group_nav.haml | 6 +++--- public/javascripts/view.js | 27 ++++++++++++++++++++++++ public/stylesheets/application.css | 23 +++++++++++++++----- public/stylesheets/sass/application.sass | 26 +++++++++++++++++++++-- 6 files changed, 81 insertions(+), 16 deletions(-) diff --git a/app/views/groups/edit.html.haml b/app/views/groups/edit.html.haml index 3b69a93fa..c59157f48 100644 --- a/app/views/groups/edit.html.haml +++ b/app/views/groups/edit.html.haml @@ -2,15 +2,18 @@ = javascript_include_tag 'jquery-ui-1.8.4.custom.min.js' = javascript_include_tag 'group-edit.js' +- content_for :left_pane do + requests + %h1{:class => 'big_text', :id => 'group_title'} - = "Editing Groups" + = "Managing Relations" %ul#group_list - for group in @groups %li.group - %h3= group.name + %h3{:contenteditable => true}= group.name %ul{:id => group.id} -if group.people.size < 1 @@ -20,8 +23,9 @@ -for person in group.people %li.person{:id => person.id, :from_group_id => group.id} - = image_tag(person.profile.image_url(:thumb_small),:size => "30x30") unless person.profile.image_url.nil? - = person.real_name + = image_tag(person.profile.image_url(:thumb_small)) unless person.profile.image_url.nil? + .name + = person.real_name %p %br = link_to 'Update Groups', '#', :class => 'button', :id => "move_friends_link" diff --git a/app/views/layouts/application.html.haml b/app/views/layouts/application.html.haml index dc687861f..d3c7e06fa 100644 --- a/app/views/layouts/application.html.haml +++ b/app/views/layouts/application.html.haml @@ -60,7 +60,7 @@ .span-5.last - if @group == :all %h1 - = link_to "All Groups", root_path + = link_to "All Relations", root_path - else %h1 = link_to @group.name, @group @@ -73,7 +73,6 @@ .container .span-5.last - = yield :left_pane .span-19.last diff --git a/app/views/shared/_group_nav.haml b/app/views/shared/_group_nav.haml index 730672333..5c07b6e83 100644 --- a/app/views/shared/_group_nav.haml +++ b/app/views/shared/_group_nav.haml @@ -7,12 +7,12 @@ %li.new_group= link_to("+", "#add_group_pane", :id => "add_group_button") - #group_edit_button - = link_to "edit", edit_group_path(Group.first), :class => "edit_group_button", :title => "Manage your facets." + #group_manage_button + = link_to "manage", edit_group_path(Group.first), :class => "edit_group_button", :title => "Manage your facets." %ul{ :style => "position:absolute;right:0;bottom:0;"} %li{:class => ("selected" if @group == :all)} - = link_to "All Groups", root_url + = link_to "All Relations", root_url .yo{ :style => "display:none;"} #add_group_pane diff --git a/public/javascripts/view.js b/public/javascripts/view.js index fbe84aac8..8b4c44134 100644 --- a/public/javascripts/view.js +++ b/public/javascripts/view.js @@ -24,8 +24,35 @@ $(document).ready(function(){ $(".image_cycle img").load( function() { $(this).fadeIn("slow"); }); + });//end document ready +$(".group h3").live( 'click', function() { + + var $this = $(this); + var id = $this.closest("li").children("ul").attr("id"); + var link = "/groups/"+ id; + + $this.keypress(function(e) { + if (e.which == 13) { + e.preventDefault(); + $this.blur(); + + //save changes + $.ajax({ + type: "PUT", + url: link, + data: {"group" : {"name" : $this.text() }} + }); + } + //update all other group links + $this.keyup(function(e) { + $("a[href='"+link+"']").text($this.text()); + }); + }); +}); + + function pane_toggler_button( name ) { $("#add_" + name + "_button").toggle( diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css index 1bc9132e6..9e2777f40 100644 --- a/public/stylesheets/application.css +++ b/public/stylesheets/application.css @@ -453,11 +453,11 @@ h1.big_text { #group_nav { position: relative; color: black; } - #group_nav #group_edit_button { + #group_nav #group_manage_button { display: inline; margin-top: 1px; font-size: 12px; } - #group_nav #group_edit_button a { + #group_nav #group_manage_button a { color: #999999; } #group_nav ul { margin-bottom: 0; @@ -588,6 +588,8 @@ h1.big_text { .group { list-style: none; } + .group h3 { + display: inline-block; } .group ul { min-height: 20px; margin: 0; @@ -598,11 +600,22 @@ h1.big_text { border-radius: 3px; list-style: none; padding: 15px; } - .group ul li { + .group ul li.person { + display: inline-block; margin-left: 0; padding: 5px; - cursor: move; } - .group ul li img { + cursor: move; + margin: 5px; + background-color: #cccccc; + -webkit-border-radius: 3px; + text-align: center; + width: 75px; + height: 75px; + padding: 5px; + border: 1px solid #999999; } + .group ul li.person img { + height: 50px; + width: 50px; display: inline-block; } .group ul .grey { font-style: italic; diff --git a/public/stylesheets/sass/application.sass b/public/stylesheets/sass/application.sass index 4b23cecfc..2c6c4be5a 100644 --- a/public/stylesheets/sass/application.sass +++ b/public/stylesheets/sass/application.sass @@ -579,7 +579,7 @@ h1.big_text :color #000 - #group_edit_button + #group_manage_button :display inline :margin :top 1px @@ -770,6 +770,10 @@ h1.big_text .group :list :style none + + h3 + :display inline-block + ul :min-height 20px :margin 0 @@ -784,15 +788,33 @@ h1.big_text :style none :padding 15px - li + li.person + :display inline-block :margin :left 0 :padding 5px :cursor move + :margin 5px + + :background + :color #ccc + :-webkit-border-radius 3px + + :text + :align center + + :width 75px + :height 75px + :padding 5px + + :border 1px solid #999 img + :height 50px + :width 50px :display inline-block + .grey :font :style italic From 82a31228fa5876aab3ceecdb8c47cd18bce1b7cc Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:07:55 -0700 Subject: [PATCH 04/11] DG IZ; requests bar added to group/edit --- app/controllers/groups_controller.rb | 1 + app/views/groups/edit.html.haml | 16 +++++++- public/stylesheets/application.css | 36 +++++++++--------- public/stylesheets/sass/application.sass | 48 ++++++++++++------------ 4 files changed, 57 insertions(+), 44 deletions(-) diff --git a/app/controllers/groups_controller.rb b/app/controllers/groups_controller.rb index 0d1cecf6b..1e47b7a8b 100644 --- a/app/controllers/groups_controller.rb +++ b/app/controllers/groups_controller.rb @@ -34,6 +34,7 @@ class GroupsController < ApplicationController def edit @groups = current_user.groups + @remote_requests = Request.for_user current_user end def update diff --git a/app/views/groups/edit.html.haml b/app/views/groups/edit.html.haml index c59157f48..342e9786c 100644 --- a/app/views/groups/edit.html.haml +++ b/app/views/groups/edit.html.haml @@ -3,10 +3,22 @@ = javascript_include_tag 'group-edit.js' - content_for :left_pane do + %h1{:id => 'group_title'} + Requests + + - for request in @remote_requests + .person{:id => request.person.id} + = image_tag(request.person.profile.image_url(:thumb_small)) unless request.person.profile.image_url.nil? + .name + = request.person.real_name + + + + requests -%h1{:class => 'big_text', :id => 'group_title'} - = "Managing Relations" +%h1{:id => 'group_title'} + Relations %ul#group_list diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css index 9e2777f40..73d991f77 100644 --- a/public/stylesheets/application.css +++ b/public/stylesheets/application.css @@ -600,23 +600,23 @@ h1.big_text { border-radius: 3px; list-style: none; padding: 15px; } - .group ul li.person { - display: inline-block; - margin-left: 0; - padding: 5px; - cursor: move; - margin: 5px; - background-color: #cccccc; - -webkit-border-radius: 3px; - text-align: center; - width: 75px; - height: 75px; - padding: 5px; - border: 1px solid #999999; } - .group ul li.person img { - height: 50px; - width: 50px; - display: inline-block; } - .group ul .grey { + .group .person { + display: inline-block; + margin-left: 0; + padding: 5px; + cursor: move; + margin: 5px; + background-color: #cccccc; + -webkit-border-radius: 3px; + text-align: center; + width: 75px; + height: 75px; + padding: 5px; + border: 1px solid #999999; } + .group .person img { + height: 50px; + width: 50px; + display: inline-block; } + .group .person .grey { font-style: italic; color: #666666; } diff --git a/public/stylesheets/sass/application.sass b/public/stylesheets/sass/application.sass index 2c6c4be5a..912f42815 100644 --- a/public/stylesheets/sass/application.sass +++ b/public/stylesheets/sass/application.sass @@ -788,31 +788,31 @@ h1.big_text :style none :padding 15px - li.person + .person + :display inline-block + :margin + :left 0 + :padding 5px + :cursor move + :margin 5px + + :background + :color #ccc + :-webkit-border-radius 3px + + :text + :align center + + :width 75px + :height 75px + :padding 5px + + :border 1px solid #999 + + img + :height 50px + :width 50px :display inline-block - :margin - :left 0 - :padding 5px - :cursor move - :margin 5px - - :background - :color #ccc - :-webkit-border-radius 3px - - :text - :align center - - :width 75px - :height 75px - :padding 5px - - :border 1px solid #999 - - img - :height 50px - :width 50px - :display inline-block .grey From 4770616a2670af24868fb06f341fe7e1af9dc89f Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:19:04 -0700 Subject: [PATCH 05/11] DG IZ; small fix --- app/views/groups/edit.html.haml | 11 ++++++----- public/stylesheets/sass/application.sass | 2 +- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/app/views/groups/edit.html.haml b/app/views/groups/edit.html.haml index 342e9786c..9bfa2861b 100644 --- a/app/views/groups/edit.html.haml +++ b/app/views/groups/edit.html.haml @@ -6,11 +6,12 @@ %h1{:id => 'group_title'} Requests - - for request in @remote_requests - .person{:id => request.person.id} - = image_tag(request.person.profile.image_url(:thumb_small)) unless request.person.profile.image_url.nil? - .name - = request.person.real_name + .requests + - for request in @remote_requests + .person{:id => request.person.id} + = image_tag(request.person.profile.image_url(:thumb_small)) unless request.person.profile.image_url.nil? + .name + = request.person.real_name diff --git a/public/stylesheets/sass/application.sass b/public/stylesheets/sass/application.sass index 912f42815..c4bfc4760 100644 --- a/public/stylesheets/sass/application.sass +++ b/public/stylesheets/sass/application.sass @@ -767,7 +767,7 @@ h1.big_text :border 1px #666 solid -.group +.group, .requests :list :style none From 8afb590128f6e84117762534a17a55268b4792db Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:28:11 -0700 Subject: [PATCH 06/11] small fix --- app/views/groups/edit.html.haml | 7 +------ public/javascripts/group-edit.js | 9 +++++++-- public/stylesheets/sass/application.sass | 2 +- 3 files changed, 9 insertions(+), 9 deletions(-) diff --git a/app/views/groups/edit.html.haml b/app/views/groups/edit.html.haml index 9bfa2861b..9810c37fd 100644 --- a/app/views/groups/edit.html.haml +++ b/app/views/groups/edit.html.haml @@ -6,18 +6,13 @@ %h1{:id => 'group_title'} Requests - .requests + .group - for request in @remote_requests .person{:id => request.person.id} = image_tag(request.person.profile.image_url(:thumb_small)) unless request.person.profile.image_url.nil? .name = request.person.real_name - - - - requests - %h1{:id => 'group_title'} Relations diff --git a/public/javascripts/group-edit.js b/public/javascripts/group-edit.js index 28ab236de..25cf7c317 100644 --- a/public/javascripts/group-edit.js +++ b/public/javascripts/group-edit.js @@ -9,10 +9,15 @@ $('#move_friends_link').live( 'click', }); $(function() { - $("li .person").draggable({ + $("li .person").draggable({ revert: true }); - $(".group ul").droppable({ + + $("li .person").draggable({ + revert: true + }); + + $(".group ul").droppable({ drop: function(event, ui) { var move = {}; diff --git a/public/stylesheets/sass/application.sass b/public/stylesheets/sass/application.sass index c4bfc4760..912f42815 100644 --- a/public/stylesheets/sass/application.sass +++ b/public/stylesheets/sass/application.sass @@ -767,7 +767,7 @@ h1.big_text :border 1px #666 solid -.group, .requests +.group :list :style none From 9fd8cfcc990eb4edb78902188292f21605b03608 Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:31:00 -0700 Subject: [PATCH 07/11] DG IZ added the same structure as groups --- app/views/groups/edit.html.haml | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/app/views/groups/edit.html.haml b/app/views/groups/edit.html.haml index 9810c37fd..13887c5d5 100644 --- a/app/views/groups/edit.html.haml +++ b/app/views/groups/edit.html.haml @@ -6,9 +6,10 @@ %h1{:id => 'group_title'} Requests - .group + %li.group + %ul - for request in @remote_requests - .person{:id => request.person.id} + %li.person{:id => request.person.id, :request_id => request.id} = image_tag(request.person.profile.image_url(:thumb_small)) unless request.person.profile.image_url.nil? .name = request.person.real_name From b4e3ea393753a3036126b11ada20751e61d9e7dd Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:33:16 -0700 Subject: [PATCH 08/11] another little fix --- app/views/groups/edit.html.haml | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/app/views/groups/edit.html.haml b/app/views/groups/edit.html.haml index 13887c5d5..d5fb2b63f 100644 --- a/app/views/groups/edit.html.haml +++ b/app/views/groups/edit.html.haml @@ -8,11 +8,11 @@ %li.group %ul - - for request in @remote_requests - %li.person{:id => request.person.id, :request_id => request.id} - = image_tag(request.person.profile.image_url(:thumb_small)) unless request.person.profile.image_url.nil? - .name - = request.person.real_name + - for request in @remote_requests + %li.person{:id => request.person.id, :request_id => request.id} + = image_tag(request.person.profile.image_url(:thumb_small)) unless request.person.profile.image_url.nil? + .name + = request.person.real_name %h1{:id => 'group_title'} Relations From d3a8e27833a51a28ac7bfd9244db5b78d55268d3 Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:37:59 -0700 Subject: [PATCH 09/11] DG IZ; selector fix --- app/views/groups/edit.html.haml | 2 +- public/stylesheets/application.css | 21 ++++++++++++++------- public/stylesheets/sass/application.sass | 6 ++++-- 3 files changed, 19 insertions(+), 10 deletions(-) diff --git a/app/views/groups/edit.html.haml b/app/views/groups/edit.html.haml index d5fb2b63f..b8fe6e904 100644 --- a/app/views/groups/edit.html.haml +++ b/app/views/groups/edit.html.haml @@ -6,7 +6,7 @@ %h1{:id => 'group_title'} Requests - %li.group + %li.requests %ul - for request in @remote_requests %li.person{:id => request.person.id, :request_id => request.id} diff --git a/public/stylesheets/application.css b/public/stylesheets/application.css index 73d991f77..029ccad0d 100644 --- a/public/stylesheets/application.css +++ b/public/stylesheets/application.css @@ -582,15 +582,19 @@ h1.big_text { font-size: 12px; margin-top: -3px; } -.editing, .editing:hover { +.editing, +.editing:hover { background-color: yellow; border: 1px #666666 solid; } -.group { +.group, +.requests { list-style: none; } - .group h3 { + .group h3, + .requests h3 { display: inline-block; } - .group ul { + .group ul, + .requests ul { min-height: 20px; margin: 0; margin-left: 1em; @@ -600,7 +604,8 @@ h1.big_text { border-radius: 3px; list-style: none; padding: 15px; } - .group .person { + .group .person, + .requests .person { display: inline-block; margin-left: 0; padding: 5px; @@ -613,10 +618,12 @@ h1.big_text { height: 75px; padding: 5px; border: 1px solid #999999; } - .group .person img { + .group .person img, + .requests .person img { height: 50px; width: 50px; display: inline-block; } - .group .person .grey { + .group .person .grey, + .requests .person .grey { font-style: italic; color: #666666; } diff --git a/public/stylesheets/sass/application.sass b/public/stylesheets/sass/application.sass index 912f42815..887af39ae 100644 --- a/public/stylesheets/sass/application.sass +++ b/public/stylesheets/sass/application.sass @@ -761,13 +761,15 @@ h1.big_text :margin :top -3px -.editing, .editing:hover +.editing, +.editing:hover :background :color yellow :border 1px #666 solid -.group +.group, +.requests :list :style none From ef6ce98c12c370e4a8f59e96d74b9336befe339b Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:41:58 -0700 Subject: [PATCH 10/11] DG IZ alert on dragging a request, trying to check --- public/javascripts/group-edit.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/public/javascripts/group-edit.js b/public/javascripts/group-edit.js index 25cf7c317..85ce2cd7c 100644 --- a/public/javascripts/group-edit.js +++ b/public/javascripts/group-edit.js @@ -20,6 +20,10 @@ $(function() { $(".group ul").droppable({ drop: function(event, ui) { + if (ui.draggable[0].attr('request_id') != undefined){ + alert('This is a request!'); + } + var move = {}; move[ 'friend_id' ] = ui.draggable[0].id move[ 'to' ] = $(this)[0].id; From 0c5ab4f20ecacc208db9a998354c45d0a39af8ec Mon Sep 17 00:00:00 2001 From: Daniel Vincent Grippi Date: Fri, 10 Sep 2010 15:44:36 -0700 Subject: [PATCH 11/11] undo for now --- public/javascripts/group-edit.js | 3 --- 1 file changed, 3 deletions(-) diff --git a/public/javascripts/group-edit.js b/public/javascripts/group-edit.js index 85ce2cd7c..5c136ebff 100644 --- a/public/javascripts/group-edit.js +++ b/public/javascripts/group-edit.js @@ -20,9 +20,6 @@ $(function() { $(".group ul").droppable({ drop: function(event, ui) { - if (ui.draggable[0].attr('request_id') != undefined){ - alert('This is a request!'); - } var move = {}; move[ 'friend_id' ] = ui.draggable[0].id