Skip to content

Commit 39bcf54

Browse files
jonkridracos
jonkri
authored and
dracos
committed
[FixaMinGata] Add various updates
Text changes, remove a couple of cobrand overrides, and some styling tweaks.
1 parent 316e92d commit 39bcf54

File tree

15 files changed

+54
-76
lines changed

15 files changed

+54
-76
lines changed

Diff for: ‎perllib/FixMyStreet/Cobrand/FixaMinGata.pm

-8
Original file line numberDiff line numberDiff line change
@@ -201,12 +201,6 @@ sub state_groups_inspect {
201201
]
202202
}
203203

204-
sub always_view_body_contribute_details {
205-
my ( $self, $contributed_as ) = @_;
206-
207-
return $contributed_as eq '';
208-
}
209-
210204
# Average responsiveness will only be calculated if a body
211205
# has at least this many fixed reports.
212206
# (Used in the Top 5 list in /reports)
@@ -216,8 +210,6 @@ sub body_responsiveness_threshold {
216210

217211
sub suggest_duplicates { 1 }
218212

219-
sub default_show_name { 1 }
220-
221213
sub path_to_email_templates {
222214
my ( $self, $lang_code ) = @_;
223215

Diff for: ‎t/cobrand/fixamingata.t

-12
Original file line numberDiff line numberDiff line change
@@ -107,18 +107,6 @@ subtest "Test ajax decimal points" => sub {
107107
};
108108
};
109109

110-
subtest "check user details shown when staff user comments as themselves" => sub {
111-
FixMyStreet::override_config {
112-
ALLOWED_COBRANDS => [ 'fixamingata' ],
113-
}, sub {
114-
$user2->update({ from_body => $body });
115-
$mech->get_ok('/report/' . $report->id);
116-
my $update_meta = $mech->extract_update_metas;
117-
like $update_meta->[0], qr/Body \(Commenter\) /;
118-
$user2->update({ from_body => undef });
119-
};
120-
};
121-
122110
subtest "check user details aren't shown when staff user comments as body" => sub {
123111
FixMyStreet::override_config {
124112
ALLOWED_COBRANDS => [ 'fixamingata' ],

Diff for: ‎templates/email/fixamingata/submit.html

+1-1
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ <h2 style="[% h2_style %]">[% report.title | html %]</h2>
5353
<a href="[% osm_url %]" title="Se den här platsen på OpenStreetMap">
5454
[%~ report.latitude %], [% report.longitude ~%]
5555
</a>
56-
[% IF closest_address %]<br>[% closest_address | trim | replace("\n\n", "<br>") %][% END %]
56+
[% IF closest_address %]<br>[% closest_address | trim | replace("\n\n", " ") %][% END %]
5757
</p>
5858
[% END %]
5959

Diff for: ‎templates/web/fixamingata/about/faq-sv.html

+5-6
Original file line numberDiff line numberDiff line change
@@ -37,23 +37,22 @@ <h1><a name="faq"></a>Vanliga frågor</h1>
3737
<dt>Vad är inte FixaMinGata till för?</dt>
3838
<dd>
3939
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:
40-
<ul>
40+
<ul style="margin-bottom: 0;">
4141
<li>Brådskande och akuta problem</li>
4242
<li>Oljud i din närmiljö</li>
43-
<li>Brand och rök</li>
4443
<li>Förslag på nya vägar, farthinder, etc.</li>
4544
<li>Klaga på dina grannar</li>
4645
<li>Klaga på din kommun</li>
47-
<li>Droger, djurplågeri, stöld, eller andra kriminella aktiviteter.</li>
4846
</ul>
47+
<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>
4948
</dd>
5049

5150
<dt>Hur använder jag e-tjänsten?</dt>
5251
<dd>Efter att du angivit postnummer eller adress ser du en karta över närområdet. Där får du en överblick över problem som redan har rapporterats, och nya rapporteras genom att klicka på kartan där problemet finns.</dd>
5352
<dt>Hur löses problem?</dt>
5453
<dd>Problem som rapporteras skickas till berörd kommun via e-post eller direkt till deras datorsystem. Kommunen hanterar sedan problemet enligt en intern process, exempelvis genom att delegera det till rätt enhet inom kommunen som får i uppdrag att lösa problemet.</dd>
5554
<dt>Är det gratis?</dt>
56-
<dd>Den här e-tjänsten används gratis för att rapportera problem. Tjänsten tillhandahålls av <a href="http://sambruk.se/">Föreningen Sambruk</a>, en förening för kommunal samverkan kring e-tjänster. Totalt ingår runt 100 kommuner och landsting i Sambruk.</dd>
55+
<dd>Den här e-tjänsten används gratis för att rapportera problem. Tjänsten tillhandahålls av <a href="http://sambruk.se/">Föreningen Sambruk</a>, en förening för kommunal samverkan kring e-tjänster. Totalt ingår runt 150 kommuner och landsting i Sambruk.</dd>
5756
<dt>Kan jag använda FixaMinGata på min mobil?</dt>
5857
<dd>FixaMinGata fungerar både i din mobil (storleken på skärmen anpassas automatiskt) och dator. Alternativt laddas appen FixaMinGata ner på Google Play eller App Store.</dd>
5958
<dt>Hur tillgänglig är tjänsten?</dt>
@@ -71,7 +70,7 @@ <h2><a name="practical"></a>Praktiska frågor</h2>
7170
<h2><a name="organisation"></a>Organisation</h2>
7271
<dl>
7372
<dt>Vem tillhandahåller FixaMinGata?</dt>
74-
<dd>Den här e-tjänsten tillhandahålls av <a href="http://sambruk.se/">Föreningen Sambruk</a>, en förening för kommunal samverkan kring e-tjänster. Totalt ingår runt 100 kommuner och landsting i Sambruk. Tjänsten togs fram i samarbete med <a href="http://kivos.se/">KIVOS</a> och <a href="http://ffkp.se/">Föreningen fri kultur och programvara</a>.</dd>
73+
<dd>Den här e-tjänsten tillhandahålls av <a href="http://sambruk.se/">Föreningen Sambruk</a>, en förening för kommunal samverkan kring e-tjänster. Totalt ingår runt 150 kommuner och landsting i Sambruk. Tjänsten togs fram i samarbete med <a href="http://kivos.se/">KIVOS</a> och <a href="http://ffkp.se/">Föreningen fri kultur och programvara</a>.</dd>
7574
<dt>Finns källkoden tillgänglig?</dt>
7675
<dd>Programvaran bakom den här tjänsten är öppen programvara och tillgänglig under Affero GPL (AGPL). Du kan <a href="http://github.com/mysociety/fixmystreet">ladda ner</a> källkoden och hjälpa oss utveckla den. Du är också välkommen att använda den i dina egna projekt!</dd>
7776
</dl>
@@ -88,4 +87,4 @@ <h2><a name="privacy"></a>Sekretess och kakor</h2>
8887
<dd>Läs vår information om <a href="/about/privacy">Sekretess och kakor</a>.</dd>
8988
</dl>
9089

91-
[% INCLUDE 'footer.html' %]
90+
[% INCLUDE 'footer.html' pagefooter = 'yes' %]

Diff for: ‎templates/web/fixamingata/about/privacy.html

+3-18
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,4 @@
1-
[% INCLUDE 'header.html', title = loc('Vanliga Frågor'), sidebar='sidebar' %]
2-
3-
[% BLOCK sidebar %]
4-
[% INCLUDE 'about/_sticky-button-content-navbar.html' %]
5-
<div class="sticky-sidebar sticky-sidebar--help">
6-
<aside>
7-
<ul>
8-
<li><a href="/faq#faq">Vanliga frågor</a></li>
9-
<li><a href="/faq#practical">Praktiska frågor</a></li>
10-
<li><a href="/faq#organisation">Organisation</a></li>
11-
<li><a href="/faq#pul">Dataskyddsombud</a></li>
12-
<li><a href="about/privacy">Sekretess och kakor</a></li>
13-
</ul>
14-
</aside>
15-
</div>
16-
[% END %]
1+
[% INCLUDE 'header.html', title = loc('Vanliga Frågor') %]
172

183
<h1>Sekretess och kakor</h1>
194
<p><strong>Vår användning av din information och vad du behöver veta.</strong></p>
@@ -67,7 +52,7 @@ <h2>Kakor</h2>
6752

6853
<dl>
6954
<dt>Vad är <em>INTE</em> FixaMinGata till för?</dt>
70-
<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>
55+
<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>
7156
</dl>
7257

73-
[% INCLUDE 'footer.html' %]
58+
[% INCLUDE 'footer.html' pagefooter = 'yes' %]

Diff for: ‎templates/web/fixamingata/contact/address.html

+5-3
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,11 @@
11
<div class="contact-details">
22
<p>FixaMinGata är en tjänst som tillhandahålls av Föreningen Sambruk, en
3-
förening med ca 100 medlemskommuner, med gemensamt intresse av e-tjänster.
3+
förening med ca 150 medlemskommuner, med gemensamt intresse av e-tjänster.
44
Du kan kontakta Föreningen Sambruk via post:</p>
55

66
<p>Föreningen Sambruk<br>
7-
c/o Sandvikens kommun<br>
8-
811 80 Sandviken</p>
7+
Box 112<br>
8+
736 22 Kungsör</p>
9+
10+
<p>Föreningen Sambruks organisationsnummer är 802428-2785.</p>
911
</div>

Diff for: ‎templates/web/fixamingata/contact/blurb.html

+10-7
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
1-
<p>
2-
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.
3-
</p>
4-
5-
<p>
6-
Vi vill gärna höra vad du tycker om FixaMinGata. Fyll i formuläret nedan och skicka in.
7-
</p>
1+
<p>Rapportera <strong>inte</strong> problem i din närmiljö genom det här
2+
formuläret. Breven går till FixaMinGata i stället för din kommun. Vill du
3+
rapportera ett problem i din närmiljö måste du <a href="/">gå till framsidan</a>
4+
och följa instruktionerna där.</p>
5+
<p>Vi vill gärna höra vad du tycker om FixaMinGata. Fyll i formuläret nedan och
6+
skicka in.</p>
7+
<p>Du kan även begära att få ditt konto borttaget via detta formulär. När kontot
8+
raderas anonymiseras dina uppgifter, inklusive din ditt namn, e-postadress och
9+
eventuella telefonnummer. Rapporter och uppdateringar finns dock kvar för
10+
statistik.</p>

Diff for: ‎templates/web/fixamingata/footer_extra_js.html

+1
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
[% scripts.push(
22
version('/cobrands/fixamingata/js.js'),
3+
version('/cobrands/fixmystreet/assets.js'),
34
) %]

Diff for: ‎templates/web/fixamingata/report/inspect/public_update.html

Whitespace-only changes.

Diff for: ‎templates/web/fixamingata/report/new/inline-tips.html

+1-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
<div class="description_tips">
2-
<h4>Tips för bra rapporter:</h4>
1+
<div class="description_tips" aria-label="Tips för bra rapporter:">
32
<ul class="do">
43
<li>Var trevlig</li>
54
<li>Ange platser noggrant</li>

Diff for: ‎web/cobrands/fixamingata/base.scss

+13-1
Original file line numberDiff line numberDiff line change
@@ -216,8 +216,12 @@ $grid-breakpoint-sm: $mysoc-footer-breakpoint-sm;
216216
padding: 0;
217217
}
218218

219+
.btn {
220+
@include button-variant($focus-bg-bottom: #e6f2f8, $focus-bg-top: #e6f2f8);
221+
}
222+
219223
.btn--primary {
220-
@include button-variant(#1a9ab8, #05617a, #05617a, #fff, #045369, #1a9ab8, #05617a, #fff);
224+
@include button-variant($bg-top: #0078bb, $bg-bottom: #006ba7, $border: #006ba7, $text: #fff, $hover-bg-bottom: #0078bb, $hover-bg-top: #006ba7, $hover-border: #006ba7, $hover-text: #fff, $focus-bg-bottom: #0078bb, $focus-bg-top: #006ba7, $focus-border: #006ba7, $focus-text: #fff);
221225
}
222226

223227
.postcode-form-box label {
@@ -229,6 +233,14 @@ $grid-breakpoint-sm: $mysoc-footer-breakpoint-sm;
229233
padding-top: 0;
230234
}
231235

236+
#geolocate_link {
237+
margin-top: 2em !important;
238+
}
239+
240+
.mobile #geolocate_link {
241+
margin-top: 15px !important;
242+
}
243+
232244
#pc-hint {
233245
display: none;
234246
}

Diff for: ‎web/cobrands/fixamingata/js.js

+14-10
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,27 @@
11
fixmystreet.inspect_form_no_scroll_on_load = 1;
22

3-
// Chrome ignores autocomplete="off" on the title input,
4-
// and incorrectly autocompletes it with the user's email address.
5-
// For now we'll reset the title to empty if it contains
6-
// an email address when the user has selected a category.
7-
// Hopefully we can get rid of this eventually if Chrome changes
8-
// its behaviour.
3+
// Chrome ignores autocomplete="off" on the title input, and incorrectly
4+
// autocompletes it with the user's email address. For now we'll reset the title
5+
// to empty if it contains an email address when the user has selected a
6+
// category. Hopefully we can get rid of this eventually if Chrome changes its
7+
// behaviour.
98
fixmystreet.fixChromeAutocomplete = function() {
109
var title = document.getElementById("form_title");
1110

1211
if (title) {
13-
if (title.value == "" ||
14-
/^[a-zA-Z0-9.!#$%&'*+\/=?^_`{|}~-]+@(?:\S{1,63})$/.test(title.value)) {
12+
if (
13+
title.value == "" ||
14+
/^[a-zA-Z0-9.!#$%&'*+\/=?^_`{|}~-]+@(?:\S{1,63})$/.test(title.value)
15+
) {
1516
title.value = "";
1617
}
1718
}
1819
};
1920

20-
// jQuery is not imported on every page
21+
// jQuery is not imported on every page.
2122
if (window.$) {
22-
$(fixmystreet).on('report_new:category_change', fixmystreet.fixChromeAutocomplete);
23+
$(fixmystreet).on(
24+
"report_new:category_change",
25+
fixmystreet.fixChromeAutocomplete
26+
);
2327
}

Diff for: ‎web/cobrands/fixamingata/layout.scss

+1-1
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,7 @@ body.mappage {
162162
color: rgb(34, 34, 34);
163163
}
164164

165-
.content {
165+
.content, .sticky-sidebar {
166166
margin-bottom: 2em;
167167
}
168168

Diff for: ‎web/cobrands/hylte/base.scss

-6
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,6 @@
88
@include button-variant($focus-bg-bottom: #e6f2f8, $focus-bg-top: #e6f2f8);
99
}
1010

11-
.btn-primary,
12-
.green-btn,
1311
.btn--primary {
1412
@include button-variant($bg-top: #0078bb, $bg-bottom: #006ba7, $border: #006ba7, $text: #fff, $hover-bg-bottom: #0078bb, $hover-bg-top: #006ba7, $hover-border: #006ba7, $hover-text: #fff, $focus-bg-bottom: #0078bb, $focus-bg-top: #006ba7, $focus-border: #006ba7, $focus-text: #fff);
1513
}
@@ -29,7 +27,3 @@
2927
#report-cta {
3028
margin-right: 1em;
3129
}
32-
33-
.sticky-sidebar li:last-child {
34-
margin-bottom: 0;
35-
}

Diff for: ‎web/cobrands/surrey/base.scss

-1
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,6 @@ input.form-error, textarea.form-error {
110110
}
111111

112112
/* BUTTONS */
113-
.btn-primary,
114113
.btn--primary,
115114
.postcode-form-box input#sub,
116115
.mappage .btn{

0 commit comments

Comments
 (0)