From aef31280f38c6ecff2b3c1a4a9f871c6be0c9fae Mon Sep 17 00:00:00 2001
From: Sarven Capadisli <csarven@status.net>
Date: Sun, 3 Jan 2010 01:48:41 +0000
Subject: [PATCH] Moved location container's buttons inside enabled state

---
 js/util.js | 59 +++++++++++++++++++++++++++---------------------------
 1 file changed, 30 insertions(+), 29 deletions(-)

diff --git a/js/util.js b/js/util.js
index 35530d7458..d43a99a180 100644
--- a/js/util.js
+++ b/js/util.js
@@ -442,37 +442,13 @@ var SN = { // StatusNet
         NoticeLocationAttach: function() {
             var NDG = $('#'+SN.C.S.NoticeDataGeo);
             if (NDG.length > 0) {
-                NDG.attr('title', NDG.text());
                 var NLE = $('#notice_data-location_wrap');
                 var geocodeURL = NLE.attr('title');
 
-                var S = '<div id="'+SN.C.S.NoticeDataGeoSelected+'" class="'+SN.C.S.Success+'"> <button class="minimize">&#95;</button> <button class="close">&#215;</button></div>';
-                var NDGS = $('#'+SN.C.S.NoticeDataGeoSelected);
-
-                if (NDGS.length > 0) {
-                    NDGS.replaceWith(S);
-                }
-                else {
-                    $('#'+SN.C.S.FormNotice).append(S);
-                }
-                NDGS = $('#'+SN.C.S.NoticeDataGeoSelected);
-
-                $('#'+SN.C.S.NoticeDataGeoSelected+' button.close').click(function(){
-                    $('#'+SN.C.S.NoticeDataGeoSelected).remove();
-                    $('#'+SN.C.S.NoticeDataGeo).attr('checked', false);
-                    $('label[for=notice_data-geo]').removeClass('checked');
-
-                    return false;
-                });
-
-                $('#'+SN.C.S.NoticeDataGeoSelected+' button.minimize').click(function(){
-                    $('#'+SN.C.S.NoticeDataGeoSelected).hide();
-
-                    return false;
-                });
+                $('label[for=notice_data-geo]').attr('title', NLE.text());
 
                 if (navigator.geolocation) {
-                    NLE.change(function() {
+                    NDG.change(function() {
                         NLE.removeAttr('title');
 
                         $.cookie(SN.C.S.NoticeLocationCookieName, $('#'+SN.C.S.NoticeDataGeo).attr('checked'));
@@ -482,14 +458,39 @@ var SN = { // StatusNet
                             NLN.remove();
                         }
 
+                        var S = '<div id="'+SN.C.S.NoticeDataGeoSelected+'" class="'+SN.C.S.Success+'"/div>';
+                        NDGS = $('#'+SN.C.S.NoticeDataGeoSelected);
+                        if (NDGS.length > 0) {
+                            NDGS.replaceWith(S);
+                        }
+                        else {
+                            $('#'+SN.C.S.FormNotice).append(S);
+                        }
+                        NDGS = $('#'+SN.C.S.NoticeDataGeoSelected);
                         NDGS.prepend('<span id="'+SN.C.S.NoticeLocationName+'">Geo</span>');
+
                         NLN = $('#'+SN.C.S.NoticeLocationName);
 
                         if ($('#'+SN.C.S.NoticeDataGeo).attr('checked') === true) {
-                            NDGS.show();
                             NLN.addClass('processing');
                             $('label[for=notice_data-geo]').addClass('checked');
 
+                            NDGS.append('<button class="minimize">&#95;</button> <button class="close">&#215;</button>');
+
+                            $('#'+SN.C.S.NoticeDataGeoSelected+' button.close').click(function(){
+                                $('#'+SN.C.S.NoticeDataGeoSelected).remove();
+                                $('#'+SN.C.S.NoticeDataGeo).attr('checked', false);
+                                $('label[for=notice_data-geo]').removeClass('checked');
+
+                                return false;
+                            });
+
+                            $('#'+SN.C.S.NoticeDataGeoSelected+' button.minimize').click(function(){
+                                $('#'+SN.C.S.NoticeDataGeoSelected).hide();
+
+                                return false;
+                            });
+
                             navigator.geolocation.getCurrentPosition(function(position) {
                                 $('#'+SN.C.S.NoticeLat).val(position.coords.latitude);
                                 $('#'+SN.C.S.NoticeLon).val(position.coords.longitude);
@@ -535,8 +536,8 @@ var SN = { // StatusNet
                     });
 
                     var cookieVal = $.cookie(SN.C.S.NoticeLocationCookieName);
-                    $('#'+SN.C.S.NoticeDataGeo).attr('checked', (cookieVal == null || cookieVal == 'true'));
-                    NLE.change();
+                    NDG.attr('checked', (cookieVal == null || cookieVal == 'true'));
+                    NDG.change();
                 }
             }
         },