aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSteven Day <steve@mysociety.org>2015-07-30 16:46:07 +0100
committerMatthew Somerville <matthew@mysociety.org>2015-07-31 13:26:51 +0100
commitb748677e472552c45c2fdc32c7ae9462cd2c0ce8 (patch)
tree8ac56db8f02f490be1508ef4b25a056ccc4f76ea
parent366f6b1fe18eae55836ba879b40bd8f1e28d1727 (diff)
Fix sidebar running over the footer on alerts page
This page used the .sticky-sidebar class that is used in the help section to present the in-page navigation of a lot of text. There, there's no danger of the sidebar being scrolled over the footer because it's very short. On this page though, there is. This commit removes that class, so that the recent photos are fixed in position with the top of the content. I don't think this is a great loss for usability or design. It also removes the alertindex class, which was only used to work around this issue. Closes #1168
-rw-r--r--templates/web/base/alert/index.html4
-rw-r--r--templates/web/base/alert/list.html2
-rw-r--r--templates/web/fixmybarangay/alert/index.html2
-rw-r--r--web/cobrands/eastsussex/layout.scss4
-rw-r--r--web/cobrands/fixmystreet/layout.scss8
5 files changed, 4 insertions, 16 deletions
diff --git a/templates/web/base/alert/index.html b/templates/web/base/alert/index.html
index e3dead34a..314008846 100644
--- a/templates/web/base/alert/index.html
+++ b/templates/web/base/alert/index.html
@@ -1,4 +1,4 @@
-[% INCLUDE 'header.html', title = loc('Local RSS feeds and email alerts'), bodyclass = 'twothirdswidthpage alertindex' %]
+[% INCLUDE 'header.html', title = loc('Local RSS feeds and email alerts'), bodyclass = 'twothirdswidthpage' %]
<h1>[% loc('Local RSS feeds and email alerts') %]</h1>
@@ -37,7 +37,7 @@ To find out what local alerts we have for you, please enter your [% c.cobrand.co
</form>
[% IF photos.size %]
-<div class="sticky-sidebar" id="alert_recent">
+<div id="alert_recent">
<aside>
<h2>[% loc('Some photos of recent reports') %]</h2>
[% FOREACH p IN photos;
diff --git a/templates/web/base/alert/list.html b/templates/web/base/alert/list.html
index 447bfcd76..7b262f9e7 100644
--- a/templates/web/base/alert/list.html
+++ b/templates/web/base/alert/list.html
@@ -21,7 +21,7 @@
<form id="alerts" name="alerts" method="post" action="/alert/subscribe">
[% IF photos.size %]
- <div id="alert_photos" class="sticky-sidebar">
+ <div id="alert_photos">
<aside>
<h2>[% loc('Photos of recent nearby reports') %]</h2>
[% FOREACH p IN photos;
diff --git a/templates/web/fixmybarangay/alert/index.html b/templates/web/fixmybarangay/alert/index.html
index 58c4080e8..c168fa41b 100644
--- a/templates/web/fixmybarangay/alert/index.html
+++ b/templates/web/fixmybarangay/alert/index.html
@@ -27,7 +27,7 @@ FixMyBarangay has a RSS feeds and email alerts for local problems.
<form id="alerts" name="alerts" method="post" action="/alert/subscribe">
[% IF photos.size %]
-<div class="sticky-sidebar" id="alert_recent">
+<div id="alert_recent">
<aside>
<h2>[% loc('Some photos of recent reports') %]</h2>
[% FOREACH p IN photos;
diff --git a/web/cobrands/eastsussex/layout.scss b/web/cobrands/eastsussex/layout.scss
index dad118198..9e818f0a5 100644
--- a/web/cobrands/eastsussex/layout.scss
+++ b/web/cobrands/eastsussex/layout.scss
@@ -108,10 +108,6 @@ body.mappage {
line-height: 14px;
}
-body.twothirdswidthpage.alertindex .content .sticky-sidebar aside {
- top: -17em;
-}
-
body.twothirdswidthpage .content .sticky-sidebar aside {
top: 0;
position: absolute;
diff --git a/web/cobrands/fixmystreet/layout.scss b/web/cobrands/fixmystreet/layout.scss
index 1d56e675c..b6698055d 100644
--- a/web/cobrands/fixmystreet/layout.scss
+++ b/web/cobrands/fixmystreet/layout.scss
@@ -249,14 +249,6 @@ body.twothirdswidthpage {
}
}
-/* hide the footer on the alert page as the interaction with the
- * photo sidebar is awkward to fix */
-body.alertindex {
- #footer-nav {
- display: none;
- }
-}
-
.next-steps {
@include clearfix;
margin-bottom: 2em; // add some space between this and the footer