[FixaMinGata] Add various updates

jonkri · jonkri · commit 0413e630ea6c · 2024-07-08T00:48:49.000+02:00
diff --git a/perllib/FixMyStreet/Cobrand/FixaMinGata.pm b/perllib/FixMyStreet/Cobrand/FixaMinGata.pm
@@ -18,6 +18,7 @@ sub language_override { 'sv' }
 
 sub enter_postcode_text {
     my ( $self ) = @_;
+
     return _('Enter a nearby postcode, or street name and area');
 }
 
@@ -87,43 +88,54 @@ sub geocoder_munge_results {
 
 sub area_types {
     my $self = shift;
+
     return $self->next::method() if FixMyStreet->staging_flag('skip_checks');
+
     [ 'KOM' ];
 }
 
 sub geocode_postcode {
     my ( $self, $s ) = @_;
+
     # Most people write Swedish postcodes like this:
     # XXX XX, so let's remove the space
     $s =~ s/\ //g;
+
     if ($s =~ /^\d{5}$/) {
         my $location = mySociety::MaPit::call('postcode', $s);
+
         if ($location->{error}) {
             return {
                 error => $location->{code} =~ /^4/
                     ? _('That postcode was not recognised, sorry.')
                     : $location->{error}
             };
         }
+
         return {
             latitude  => $location->{wgs84_lat},
             longitude => $location->{wgs84_lon},
         };
     }
+
     return {};
 }
 
 # Vad gör den här funktionen? Är "Sverige" rätt här?
 sub geocoded_string_check {
     my ( $self, $s ) = @_;
+
     return 1 if $s =~ /, Sverige/;
+
     return 0;
 }
 
 sub find_closest {
     my ( $self, $problem ) = @_;
+
     $problem = $problem->{problem} if ref $problem eq 'HASH';
-    return FixMyStreet::Geocode::OSM->closest_road_text( $self, $problem->latitude, $problem->longitude );
+
+    return FixMyStreet::Geocode::OSM::closest_road_text( $self, $problem->latitude, $problem->longitude );
 }
 
 # Used by send-reports, calling find_closest, calling OSM geocoding
@@ -132,13 +144,15 @@ sub guess_road_operator {
 
     my $highway = $inforef->{highway} || "unknown";
     my $refs    = $inforef->{ref}     || "unknown";
+
     return "Trafikverket"
         if $highway eq "trunk" || $highway eq "primary";
 
     for my $ref (split(/;/, $refs)) {
         return "Trafikverket"
             if $ref =~ m/E ?\d+/ || $ref =~ m/Fv\d+/i;
     }
+
     return '';
 }
 
@@ -167,9 +181,13 @@ sub filter_all_council_ids_list {
 # confirmed state), and red otherwise.
 sub pin_colour {
     my ( $self, $p, $context ) = @_;
+
     return 'green' if $p->is_closed;
+
     return 'green' if $p->is_fixed;
+
     return 'yellow' if $p->is_in_progress;
+
     return 'red';
 }
 
@@ -181,11 +199,6 @@ sub state_groups_inspect {
     ]
 }
 
-sub always_view_body_contribute_details {
-    my ( $self, $contributed_as ) = @_;
-    return $contributed_as eq '';
-}
-
 # Average responsiveness will only be calculated if a body
 # has at least this many fixed reports.
 # (Used in the Top 5 list in /reports)
@@ -195,6 +208,4 @@ sub body_responsiveness_threshold {
 
 sub suggest_duplicates { 1 }
 
-sub default_show_name { 1 }
-
 1;
diff --git a/templates/email/fixamingata/submit.html b/templates/email/fixamingata/submit.html
@@ -53,7 +53,7 @@ <h2 style="[% h2_style %]">[% report.title | html %]</h2>
       <a href="[% osm_url %]" title="Se den här platsen på OpenStreetMap">
         [%~ report.latitude %], [% report.longitude ~%]
       </a>
-      [% IF closest_address %]<br>[% closest_address | trim | replace("\n\n", "<br>") %][% END %]
+      [% IF closest_address %]<br>[% closest_address | trim | replace("\n\n", " ") %][% END %]
     </p>
 [% END %]
 
diff --git a/templates/web/fixamingata/about/faq-sv.html b/templates/web/fixamingata/about/faq-sv.html
@@ -1,7 +1,5 @@
 [% INCLUDE 'header.html', title => loc('Frequently Asked Questions'), bodyclass => 'twothirdswidthpage' %]
 
-[% INCLUDE 'about/_sticky-button-content-navbar.html' %]
-
 <div class="sticky-sidebar">
     <aside>
         <ul class="plain-list">
@@ -36,15 +34,14 @@ <h1><a name="faq"></a>Vanliga frågor</h1>
     <dt>Vad är inte FixaMinGata till för?</dt>
     <dd>
         FixaMinGata är inte lämpligt för andra typer av problem än de som anges ovan. Du måste kontakta din kommun eller fastighetsägare direkt för problem i stil med:
-        <ul>
+        <ul style="margin-bottom: 0;">
             <li>Brådskande och akuta problem</li>
             <li>Oljud i din närmiljö</li>
-            <li>Brand och rök</li>
             <li>Förslag på nya vägar, farthinder, etc.</li>
             <li>Klaga på dina grannar</li>
             <li>Klaga på din kommun</li>
-            <li>Droger, djurplågeri, stöld, eller andra kriminella aktiviteter.</li>
         </ul>
+        <p style="margin-bottom: 2em;">Vid pågående brott och vid fara för liv, hälsa eller egendom ska du ringa 112.</p>
     </dd>
 
     <dt>Hur använder jag e-tjänsten?</dt>
@@ -87,4 +84,4 @@ <h2><a name="privacy"></a>Sekretess och kakor</h2>
     <dd>Läs vår information om <a href="/about/privacy">Sekretess och kakor</a>.</dd>
 </dl>
 
-[% INCLUDE 'footer.html' %]
+[% INCLUDE 'footer.html' pagefooter = 'yes' %]
diff --git a/templates/web/fixamingata/about/privacy.html b/templates/web/fixamingata/about/privacy.html
@@ -1,20 +1,5 @@
 [% INCLUDE 'header.html', title => loc('Vanliga Frågor'), bodyclass => 'twothirdswidthpage' %]
 
-[% INCLUDE 'about/_sticky-button-content-navbar.html' %]
-
-<div class="sticky-sidebar">
-    <aside>
-        <ul class="plain-list">
-            <li><a href="/faq#faq">Vanliga frågor</a></li>
-            <li><a href="/faq#practical">Praktiska frågor</a></li>
-            <li><a href="/faq#organisation">Organisation</a></li>
-            <li><a href="/faq#pul">Dataskyddsombud</a></li>
-            <li><a href="about/privacy">Sekretess och kakor</a></li>
-        </ul>
-    </aside>
-</div>
-
-
 <h1>Sekretess och kakor</h1>
 <p><strong>Vår användning av din information och vad du behöver veta.</strong></p>
 <p>Offentlighetsprincipen är en viktig del i det demokratiska samhället. Genom tryckfrihetsförordningen och offentlighetslagen har du rätt att ta del av allmänna offentliga handlingar hos statliga och kommunala myndigheter. Det kan ge dig insyn i hur förtroendevalda (politiker) och tjänstemän handlägger ärenden, vilka handlingar som kommit in till kommunens olika förvaltningar och vilka som skickats ut.</p>
@@ -67,7 +52,7 @@ <h2>Kakor</h2>
 
 <dl>
   <dt>Vad är <em>INTE</em> FixaMinGata till för?</dt>
-  <dd>FixaMinGata är inte lämpligt för andra typer av problem än de som anges ovan. Du måste kontakta din kommun eller fastighetsägare direkt för problem i stil med:<ul><li>Brådskande och akuta problem</li><li>Oljud i din närmiljö</li><li>Brand och rök</li><li>Förslag på nya vägar, farthinder, etc.</li><li>Klaga på dina grannar</li><li>Klaga på din kommun</li><li>Droger, djurplågeri, stöld, eller andra kriminella aktiviteter.</li><li>Information kring brott/överträdelser eller misstanke om detsamma</li></ul></dd>
+  <dd>FixaMinGata är inte lämpligt för andra typer av problem än de som anges ovan. Du måste kontakta din kommun eller fastighetsägare direkt för problem i stil med:<ul style="margin-bottom: 0;"><li>Brådskande och akuta problem</li><li>Oljud i din närmiljö</li><li>Förslag på nya vägar, farthinder, etc.</li><li>Klaga på dina grannar</li><li>Klaga på din kommun</li><li>Information kring brott/överträdelser eller misstanke om detsamma</li></ul><p style="margin-bottom: 2em;">Vid pågående brott och vid fara för liv, hälsa eller egendom ska du ringa 112.</p></dd>
 </dl>
 
-[% INCLUDE 'footer.html' %]
+[% INCLUDE 'footer.html' pagefooter = 'yes' %]
diff --git a/templates/web/fixamingata/contact/address.html b/templates/web/fixamingata/contact/address.html
@@ -4,6 +4,7 @@
 Du kan kontakta Föreningen Sambruk via post:</p>
 
 <p>Föreningen Sambruk<br>
-c/o Sandvikens kommun<br>
-811 80 Sandviken</p>
+Sandbacka Park<br>
+Högbovägen 45<br>
+811 32 Sandviken</p>
 </div>
diff --git a/templates/web/fixamingata/contact/blurb.html b/templates/web/fixamingata/contact/blurb.html
@@ -1,7 +1,10 @@
-<p>
-Rapportera <strong>inte</strong> problem i din närmiljö genom det här formuläret. Breven går till FixaMinGata i stället för din kommun. Vill du rapportera ett problem i din närmiljö måste du <a href="/">gå till framsidan</a> och följa instruktionerna där.
-</p>
-
-<p>
-Vi vill gärna höra vad du tycker om FixaMinGata. Fyll i formuläret nedan och skicka in.
-</p>
+<p>Rapportera <strong>inte</strong> problem i din närmiljö genom det här
+formuläret. Breven går till FixaMinGata i stället för din kommun. Vill du
+rapportera ett problem i din närmiljö måste du <a href="/">gå till framsidan</a>
+och följa instruktionerna där.</p>
+<p>Vi vill gärna höra vad du tycker om FixaMinGata. Fyll i formuläret nedan och
+skicka in.</p>
+<p>Du kan även begära att få ditt konto borttaget via detta formulär. När kontot
+raderas anonymiseras dina uppgifter, inklusive din ditt namn, e-postadress och
+eventuella telefonnummer. Rapporter och uppdateringar finns dock kvar för
+statistik.</p>
diff --git a/templates/web/fixamingata/footer_extra.html b/templates/web/fixamingata/footer_extra.html
@@ -34,30 +34,30 @@ <h2 class="mysoc-footer__site-name">FixaMinGata</h2>
             <div class="col-sm-5">
                 <nav class="mysoc-footer__links">
                     <ul>
-                        <li><[% IF c.req.uri.path == '/' %]span[% ELSE %]a href="/"[% END %] class="report-a-problem-btn"
+                        <li><[% IF c.req.uri.path == '/' %]span[% ELSE %]a href="[% base %]/"[% END %] class="report-a-problem-btn"
                             >[% loc("Report a problem") %]</[% c.req.uri.path == '/' ? 'span' : 'a' %]></li>
-                        <li><[% IF c.req.uri.path == '/my' %]span[% ELSE %]a href="/my"[% END
+                        <li><[% IF c.req.uri.path == '/my' %]span[% ELSE %]a href="[% base %]/my"[% END
                             %]>[% loc("Your reports") %]</[% c.req.uri.path == '/my' ? 'span' : 'a' %]></li>
-                        <li><[% IF c.req.uri.path == '/reports' %]span[% ELSE %]a href="/reports"[% END
+                        <li><[% IF c.req.uri.path == '/reports' %]span[% ELSE %]a href="[% base %]/reports"[% END
                             %]>[% loc("All reports") %]</[% c.req.uri.path == '/reports' ? 'span' : 'a' %]></li>
-                        <li><[% IF c.req.uri.path == '/alert' %]span[% ELSE %]a href="/alert[% pc ? '/list?pc=' : '' %][% pc | uri %]"[% END
+                        <li><[% IF c.req.uri.path == '/alert' %]span[% ELSE %]a href="[% base %]/alert[% pc ? '/list?pc=' : '' %][% pc | uri %]"[% END
                             %]>[% loc("Local alerts") %]</[% c.req.uri.path == '/alert' ? 'span' : 'a' %]></li>
                     </ul>
                     <ul>
-                        <!-- <li><[% IF c.req.uri.path == '/posters' %]span[% ELSE %]a href="/posters"[% END
+                        <!-- <li><[% IF c.req.uri.path == '/posters' %]span[% ELSE %]a href="[% base %]/posters"[% END
                             %]>[% loc("FixMyStreet Goodies") %]</[% c.req.uri.path == '/posters' ? 'span' : 'a' %]></li> -->
-                        <!-- <li><[% IF c.req.uri.path == '/contact' %]span[% ELSE %]a href="/contact"[% END
+                        <!-- <li><[% IF c.req.uri.path == '/contact' %]span[% ELSE %]a href="[% base %]/contact"[% END
                             %]>[% loc("Contact") %]</[% c.req.uri.path == '/contact' ? 'span' : 'a' %]></li> -->
-                        <li><[% IF c.req.uri.path == '/contact' %]span[% ELSE %]a href="/contact"[% END
+                        <li><[% IF c.req.uri.path == '/contact' %]span[% ELSE %]a href="[% base %]/contact"[% END
                             %]>Kontakt</[% c.req.uri.path == '/contact' ? 'span' : 'a' %]></li>
-                        <li><[% IF c.req.uri.path == '/faq' %]span[% ELSE %]a href="/faq"[% END
+                        <li><[% IF c.req.uri.path == '/faq' %]span[% ELSE %]a href="[% base %]/faq"[% END
                             %]>[% loc("Help") %]</[% c.req.uri.path == '/faq' ? 'span' : 'a' %]></li>
-                        <!-- <li><[% IF c.req.uri.path == '/privacy' %]span[% ELSE %]a href="/privacy"[% END
+                        <!-- <li><[% IF c.req.uri.path == '/privacy' %]span[% ELSE %]a href="[% base %]/privacy"[% END
                             %]>[% loc("Privacy") %]</[% c.req.uri.path == '/privacy' ? 'span' : 'a' %]></li> -->
-                        <li><[% IF c.req.uri.path == '/about/privacy' %]span[% ELSE %]a href="/about/privacy"[% END
+                        <li><[% IF c.req.uri.path == '/about/privacy' %]span[% ELSE %]a href="[% base %]/about/privacy"[% END
                             %]>Sekretess</[% c.req.uri.path == '/about/privacy' ? 'span' : 'a' %]></li>
                         <li><a href="https://github.com/Sambruk/fixmystreet">Öppen källkod</a></li>
-                        <li><[% IF c.req.uri.path == '/about/accessibility' %]span[% ELSE %]a href="/about/accessibility"[% END
+                        <li><[% IF c.req.uri.path == '/about/accessibility' %]span[% ELSE %]a href="[% base %]/about/accessibility"[% END
                             %]>Tillgänglighet</[% c.req.uri.path == '/about/accessibility' ? 'span' : 'a' %]></li>
                     </ul>
                 </nav>
diff --git a/templates/web/fixamingata/footer_extra_js.html b/templates/web/fixamingata/footer_extra_js.html
@@ -1,3 +1,4 @@
 [% scripts.push(
     version('/cobrands/fixamingata/js.js'),
+    version('/cobrands/fixmystreet/assets.js'),
 ) %]
diff --git a/templates/web/fixamingata/report/inspect/public_update.html b/templates/web/fixamingata/report/inspect/public_update.html
diff --git a/templates/web/fixamingata/report/new/inline-tips.html b/templates/web/fixamingata/report/new/inline-tips.html
@@ -1,5 +1,4 @@
-<div class="description_tips">
-    <h4>Tips för bra rapporter:</h4>
+<div class="description_tips" aria-label="Tips för bra rapporter:">
     <ul class="do">
         <li>Var trevlig</li>
         <li>Ange platser noggrant</li>
diff --git a/web/cobrands/fixamingata/base.scss b/web/cobrands/fixamingata/base.scss
@@ -231,6 +231,18 @@ $grid-breakpoint-sm: $mysoc-footer-breakpoint-sm;
     padding-top: 0;
 }
 
+#geolocate_link {
+    margin-top: 2em !important;
+}
+
+.mobile #geolocate_link {
+    margin-top: 15px !important;
+}
+
 #pc-hint {
     display: none;
 }
+
+.sticky-sidebar li:last-child {
+    margin-bottom: 0;
+}
diff --git a/web/cobrands/fixamingata/js.js b/web/cobrands/fixamingata/js.js
@@ -1,23 +1,27 @@
 fixmystreet.inspect_form_no_scroll_on_load = 1;
 
-// Chrome ignores autocomplete="off" on the title input,
-// and incorrectly autocompletes it with the user's email address.
-// For now we'll reset the title to empty if it contains
-// an email address when the user has selected a category.
-// Hopefully we can get rid of this eventually if Chrome changes
-// its behaviour.
+// Chrome ignores autocomplete="off" on the title input, and incorrectly
+// autocompletes it with the user's email address. For now we'll reset the title
+// to empty if it contains an email address when the user has selected a
+// category. Hopefully we can get rid of this eventually if Chrome changes its
+// behaviour.
 fixmystreet.fixChromeAutocomplete = function() {
     var title = document.getElementById("form_title");
 
     if (title) {
-	if (title.value == "" ||
-	    /^[a-zA-Z0-9.!#$%&'*+\/=?^_`{|}~-]+@(?:\S{1,63})$/.test(title.value)) {
+        if (
+            title.value == "" ||
+            /^[a-zA-Z0-9.!#$%&'*+\/=?^_`{|}~-]+@(?:\S{1,63})$/.test(title.value)
+        ) {
             title.value = "";
         }
     }
 };
 
-// jQuery is not imported on every page
+// jQuery is not imported on every page.
 if (window.$) {
-    $(fixmystreet).on('report_new:category_change', fixmystreet.fixChromeAutocomplete);
+    $(fixmystreet).on(
+        "report_new:category_change",
+        fixmystreet.fixChromeAutocomplete
+    );
 }
