Reusable notice form fixes for geolocation

diff --git a/js/util.js b/js/util.js
index 3a901b6c09122d43143bc3ed7c24f5f0c16b81d1..1bd011cb0d8069621c17d47c00bc5a3ffe9c3936 100644 (file)
--- a/js/util.js
+++ b/js/util.js
@@ -990,7 +990,7 @@ var SN = { // StatusNet
             var NLN = ''; // @fixme
             var NDGe = form.find('[name=notice_data-geo]');
             var check = form.find('[name=notice_data-geo]');
-            var label = $('label[for='+check.attr('id')+']');
+            var label = form.find('label.notice_data-geo');
 
             function removeNoticeDataGeo(error) {
                 label
@@ -1164,7 +1164,6 @@ var SN = { // StatusNet
          */
         NoticeGeoStatus: function(form, status, lat, lon, url)
         {
-            var form = $('#form_notice');
             var wrapper = form.find('.geo_status_wrapper');
             if (wrapper.length == 0) {
                 wrapper = $('<div class="'+SN.C.S.Success+' geo_status_wrapper"><button class="close" style="float:right">&#215;</button><div class="geo_status"></div></div>');

diff --git a/lib/noticeform.php b/lib/noticeform.php
index ae8023275a0f7a0b2d39605b3ee4af1e0c1c1d58..0f13147d003c4bea8f2760b9804d0fe07c17ae30 100644 (file)
--- a/lib/noticeform.php
+++ b/lib/noticeform.php
@@ -94,6 +94,8 @@ class NoticeForm extends Form
 
     function __construct($out=null, $action=null, $content=null, $user=null, $inreplyto=null, $lat=null, $lon=null, $location_id=null, $location_ns=null)
     {
+        $this->id_suffix = time();
+
         parent::__construct($out);
 
         $this->action  = $action;
@@ -125,7 +127,7 @@ class NoticeForm extends Form
 
     function id()
     {
-        return 'form_notice';
+        return 'form_notice_' . $this->id_suffix;
     }
 
    /**
@@ -208,7 +210,21 @@ class NoticeForm extends Form
 
                 $this->out->elementStart('div', array('class' => 'notice_data-geo_wrap',
                                                       'title' => common_local_url('geocode')));
-                $this->out->checkbox('notice_data-geo', _('Share my location'), true);
+
+                // @fixme checkbox method allows no way to change the id without changing the name
+                //$this->out->checkbox('notice_data-geo', _('Share my location'), true);
+                $this->out->element('input', array(
+                    'name' => 'notice_data-geo',
+                    'type' => 'checkbox',
+                    'class' => 'checkbox',
+                    'id' => $this->id() . '-notice_data-geo',
+                    'checked' => true, // ?
+                ));
+                $this->out->text(' ');
+                $this->out->element('label', array('class' => 'notice_data-geo',
+                                              'for' => $this->id() . '-notice_data-geo'),
+                               _('Share my location'));
+
                 $this->out->elementEnd('div');
                 $this->out->inlineScript(' var NoticeDataGeo_text = {'.
                     'ShareDisable: ' .json_encode(_('Do not share my location')).','.

diff --git a/plugins/Realtime/realtimeupdate.css b/plugins/Realtime/realtimeupdate.css
index b277b30a14e74369314bf2c4cead9d5c53fb4cfb..7fd7fd5ed35c6dbc4097b43ceb0df1ab0ac76ced 100644 (file)
--- a/plugins/Realtime/realtimeupdate.css
+++ b/plugins/Realtime/realtimeupdate.css
@@ -19,7 +19,7 @@ display:none;
 
 .realtime-popup #form_notice label[for=notice_data-attach],
 .realtime-popup #form_notice #notice_data-attach,
-.realtime-popup #form_notice label[for=notice_data-geo] {
+.realtime-popup #form_notice label.notice_data-geo {
 top:0;
 }
 

diff --git a/theme/biz/css/display.css b/theme/biz/css/display.css
index e562f7072a49f2fc5d7e1193ce50f3299832ed1c..0bb53f74a84d5744268fc2b97f52d0153b0e2633 100644 (file)
--- a/theme/biz/css/display.css
+++ b/theme/biz/css/display.css
@@ -118,10 +118,10 @@ box-shadow:3px 3px 3px rgba(194, 194, 194, 0.1);
 text-shadow:none;
 }
 
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
 background-position:0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
 background-position:0 -1846px;
 }
 
@@ -216,7 +216,7 @@ button.close,
 .entity_role_administrator input.submit,
 .entity_role_moderator input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,

diff --git a/theme/cloudy/css/display.css b/theme/cloudy/css/display.css
index ab23896b2d3702cfdf649a834e951d5cd3ee1eab..a5eabac4adafc5c56310bc8ada3a204c2817a866 100644 (file)
--- a/theme/cloudy/css/display.css
+++ b/theme/cloudy/css/display.css
@@ -1636,7 +1636,7 @@ button.close,
 .entity_silence input.submit,
 .entity_delete input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,
@@ -1746,10 +1746,10 @@ background-position:0 47%;
 .form_notice a#notice_data-geo_name {
 background-position:0 -1711px;
 }
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
 background-position:0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
 background-position:0 -1846px;
 }
 

diff --git a/theme/default/css/display.css b/theme/default/css/display.css
index 63554aae18eee55bf11f4970d1cd466542400e2b..fbd3afb12c4b3e46745b181d9933e93952412faa 100644 (file)
--- a/theme/default/css/display.css
+++ b/theme/default/css/display.css
@@ -116,10 +116,10 @@ box-shadow:3px 3px 3px rgba(194, 194, 194, 0.1);
 text-shadow:none;
 }
 
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
 background-position:0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
 background-position:0 -1846px;
 }
 
@@ -196,7 +196,7 @@ button.close,
 .entity_role_administrator input.submit,
 .entity_role_moderator input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,

diff --git a/theme/identica/css/display.css b/theme/identica/css/display.css
index 211c539dcb96bfdfb2182612e9597985c82584d5..fba72a09d695cac1ed433db5f6aafd4c6c9d01f8 100644 (file)
--- a/theme/identica/css/display.css
+++ b/theme/identica/css/display.css
@@ -117,10 +117,10 @@ box-shadow:3px 3px 3px rgba(194, 194, 194, 0.1);
 text-shadow:none;
 }
 
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
 background-position:0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
 background-position:0 -1846px;
 }
 
@@ -197,7 +197,7 @@ button.close,
 .entity_role_administrator input.submit,
 .entity_role_moderator input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,

diff --git a/theme/mnml/css/display.css b/theme/mnml/css/display.css
index f5b80b0e9596ebec1b8b8f3e808b2fb57ec1657f..57ad426c5592f2fef038b029b3fd4f1184da38e2 100644 (file)
--- a/theme/mnml/css/display.css
+++ b/theme/mnml/css/display.css
@@ -271,7 +271,7 @@ button.close,
 .entity_role_administrator input.submit,
 .entity_role_moderator input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,
@@ -286,10 +286,10 @@ button.minimize,
     background-color:transparent;
 }
 
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
     background-position:0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
     background-position:0 -1846px;
 }
 .form_notice label[for=notice_data-attach] {

diff --git a/theme/pigeonthoughts/css/display.css b/theme/pigeonthoughts/css/display.css
index 155a473519354f97b9e0ede0930ab4177a55ace2..b4f2f297b0dc0f821d1314cba9cc607321563c8d 100644 (file)
--- a/theme/pigeonthoughts/css/display.css
+++ b/theme/pigeonthoughts/css/display.css
@@ -84,10 +84,10 @@ box-shadow:3px 3px 3px rgba(194, 194, 194, 0.1);
 text-shadow:none;
 }
 
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
 background-position:0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
 background-position:0 -1846px;
 }
 
@@ -233,7 +233,7 @@ button.close,
 .entity_silence input.submit,
 .entity_delete input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,

diff --git a/theme/rebase/css/display.css b/theme/rebase/css/display.css
index 4038ab614a3014acf70d70aed46a39a9d0cf467c..32df41b93afd4e5ed9a75b3982e65a3e3f54cdc2 100644 (file)
--- a/theme/rebase/css/display.css
+++ b/theme/rebase/css/display.css
@@ -1771,7 +1771,7 @@ button.close,
 .entity_role_administrator input.submit,
 .entity_role_moderator input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,
@@ -1924,10 +1924,10 @@ background-position:0 -1054px;
 .form_notice label[for=notice_data-attach] {
     background-position: 0 -328px;
 }
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
     background-position: 0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
     background-position: 0 -1846px;
 }
 

diff --git a/theme/victorian/css/display.css b/theme/victorian/css/display.css
index 3ab7fc601a799d51a0ab7067dc38e4d312e18f5a..6252fdf908f0a1fabed9b63031db6c4d9f42a8a8 100644 (file)
--- a/theme/victorian/css/display.css
+++ b/theme/victorian/css/display.css
@@ -294,10 +294,10 @@ input.submit:focus {
     text-shadow:none;
 }
 
-.form_notice label[for=notice_data-geo] {
+.form_notice label.notice_data-geo {
     background-position:0 -1780px;
 }
-.form_notice label[for=notice_data-geo].checked {
+.form_notice label.notice_data-geo.checked {
     background-position:0 -1846px;
 }
 
@@ -373,7 +373,7 @@ button.close,
 .entity_role_administrator input.submit,
 .entity_role_moderator input.submit,
 .notice-options .repeated,
-.form_notice label[for=notice_data-geo],
+.form_notice label.notice_data-geo,
 button.minimize,
 .form_reset_key input.submit,
 .entity_clear input.submit,