| Commit message (Collapse) | Author | Age | Lines |
|---|
| |
|
|
|
| |
The fix in de36c49d1 broke pin clicking on other
map list pages, such as /my and /reports.
|
| |
|
|
|
|
| |
Dealing with the map UI changes earlier leads to a better display
position, and we add a global variable check to allow cobrands to
not use this behaviour. Fixes #1863.
|
| |
|
|
|
| |
The scroll position wa being remmbered, meaning
the full-page map was not being shown correctly.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
On a mobile device that implements pointer events, there are two events that
can happen on an /around page – touching the map starts a new report (or goes
back to the map if already on a report page); touching a pin pulls in that
report’s page. The map touch, which uses an OpenLayers.Handler.Click, operates
on click, whereas the pin touch, which uses an OpenLayers.Handler.Feature,
operates on touchstart. Neither event cancels the other, which means you can
end up either starting a new report, and then it tries to pull in an undefined
pin ID report and errors, or the pin report loads and then you instantly go
back to the map page.
In order to work around this, we disable the map click control when the feature
control event starts, and reactivate once the report data has been loaded.
|
| | |
|
| |
|
|
|
|
| |
If a TfL road is clicked, but no asset selected, show a message that it might
not be a Bromley road and set single_body_only to TfL. Otherwise hide message
and set single_body_only to Bromley.
|
| |
|
|
|
| |
Adds street lights, park bins, trees, rights of way,
and TfL red route asset layers.
|
| | |
|
| | |
|
| |
|
|
|
| |
Otherwise a non-interactive layer applying to the same category
as an asset layer can cause an ‘undefined’ message to appear.
|
| | |
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
Going back to /around from /report/new can trigger loadend (and the
spinner hiding function) twice (if the strategy activation starts a
read), with triggerRead aborting that read and triggering the event
itself. This means the spinner is not displayed, as the count falls
below 0. We could pin the count above 0, but instead let’s log each
layer ID while ‘active’ and switch off when they’re all gone.
|
| |/
|
|
|
|
| |
Editing a contact with extra fields would cause those fields to be
deleted because they weren't included in the POST request when saving.
This change ensures they're present in the DOM, but hidden by default.
|
| |\ |
|
| | | |
|
| | | |
|
| |/ |
|
| |\ |
|
| | |
| |
| |
| | |
Useful for simple tables like the cookie lists on Privacy Policy pages.
|
| | | |
|
| |/
|
|
|
|
| |
This fixes a bug whereby a double quote in an item would not be JSON-escaped
due to being HTML-escaped first, meaning it would not parse as JSON on the
client.
|
| |
|
|
| |
Fixes #2124.
|
| | |
|
| |
|
|
| |
Fixes mysociety/fixmystreetforcouncils#231
|
| |
|
|
| |
Part of #2115.
|
| |
|
|
| |
Part of #2115.
|
| |
|
|
|
| |
Only the contacts table gets a button added to toggle the display
of deleted rows; elsewhere the rows need to continue to be shown.
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since #map_sidebar no longer disappears behind #site_header, there is
no need for .internal-link-fixed-header to compensate for the height
of the header. Just an arbitrary 2em offset is enough to prevent it
butting up against the top of the #map_sidebar scrollable element.
Meanwhile, moving the .internal-link-fixed-header anchor element to
the top of its parent list item means it no longer has to compensate
for the height of the moderation form elements, because those elements
(indeed, the entire list item <form>) now comes *after* the anchor.
Fixes mysociety/fixmystreet-commercial#1027.
|
| |\ |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The easy thing first - IE appears to 'stack' :before entries, so if a
stylesheet is loaded multiple times (e.g. due to critical CSS loading
on the front page), a font-size of 2.5em gets multipled. Switching to
rems fixes this.
The preload polyfill in IE8/9 loads the base stylesheet as if it were
after the layout stylesheet, which breaks the navigation display. The
polyfill has a new version which is somewhat simplified, updating the
existing <link> rather than creating a new one, so we upgrade to that
plus a PR to prevent browser hanging.
However, in IE8 this now incorrectly uses the print styles on screen,
meaning the navigation and footer disappear. So use an IE conditional
comment to just cut IE8 out entirely. This means we can also cut down
the polyfill somewhat.
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
This is already defined in fixmystreet.utils further up.
|
| |/ |
|
| | |
|
| |
|
|
|
|
|
| |
If JavaScript is enabled hide the deleted contacts from the list on the
body page. Display as normal if no JavaScript.
Fixes #1962
|
| |
|
|
|
|
|
|
|
| |
If the user visits /report/new directly and doesn't change the pin
location, then the assets:selected/maps:update_pin events are never
fired and fixmystreet.usrn.select is never called. This results in a
report whose location was never looked up against the USRN layer,
which can cause issues for Open311 endpoints that require a USRN
value.
|
| |
|
|
|
| |
Makes a bit more sense to group those relating to sending/receiving
comments together and those relating to fetching comments together.
|
| |
|
|
|
| |
If a layer has an attribution option then add it to the attribution
control when the layer becomes visible.
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
|
|
| |
To match the more generic changes in open311-adapter
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
BANES have some staff-only assets, so this adds a cobrand hook
to enable extra JS to be included on the page for staff users.
- Add custom OpenLayers build with GeoJSON support
- Add grit bins/street lights/LLPG asset layers
- Displays non-BANES street light assets in grey
- Add adopted highways to map
This means USRNs will be set for reports made by members of the public.
To keep the map simple, however, the highways layer is invisible to
non-staff users.
|
| | |
|
| |\ |
|
