Fix title alignment in Outlook/WindowsMail

Don't display a box at all if the note event doesn't include a comment body

Before this it just displays en empty box, which looks wrong.

Removed italic styling on message body

Contributor BushmanK pointed out quite rightly that italic isn't appropriate for a full body of text.

Force the font on every <p> tag

Without this Outlook and Windows Mail don't honour the font set at the top level. Note that this change predates all the screenshots I uploaded to the wiki on 2017-02-02, I'd just forgotten to commit them.

Tweak cell width for proper centering on narrow screens

Only record a visible object if it exists

Fixes #1430

Update to iD v2.1.2

Update to iD v2.1.1

Merge remote-tracking branch 'openstreetmap/pull/1426'

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

Link GPX fixture files instead of stubbing

Cleanup double comment markers

Make TraceController#list sort by id instead of timestamp

The effect is much the same but id is guaranteed to give a stable
sort if two traces have the same timestamp.

Merge remote-tracking branch 'openstreetmap/pull/1347' into master

Fix rubocop warnings

Use .first in place of [0]

Merge remote-tracking branch 'openstreetmap/pull/1401'

Update bundle

Update rubocop

Update to iD v2.1.0

Merge remote-tracking branch 'openstreetmap/pull/1423'

Convert the test_users_nearby to user factories.

Use factory for user language tests

Refactor the friend_users test, and remove the tests which duplicate the (renamed) test_friends_with above.

Convert test_friend_with to use factories, and use alice/bob/charlie for easier understanding.

User factory for uniqueness tests

Convert class method tests to use User factory.

Also improve the tests by being explicit about which results should
be returned, rather than just hoping that the fixtures cover all
possibilities.

Add status traits to user factory.

Add moderator_user and administrator_user factories.

Add a user_role factory

Naming the association in the model makes the relationship easier
to express in the factory.

Add with_home_location trait for user factories.

Create a simple user factory, and convert some basic tests.

Reduce maximum changeset size to 10k changes

Ten thousand changes ought to be enough for any normal mapping activity.

Automatic edits and imports are controlled by scripts anyway so it doesn't
make a difference to them as they just have to adapt the limit.

The reason for my suggestion is that large changesets are becoming
increasingly difficult to handle due to frequent timeouts when trying
to load and process them.

The changeset size limit is returned by the API in the "capabilities" request
so client software that honours that information will automatically pick up
the new limit.

Closes #1259

Remove fake qqq locale

Localisation updates from https://translatewiki.net.

That new message in the HTML version of the mail should also be added to the plain text version of the mail

Duh.

Use each_with_object and make the code pithier

Renamed `apply_inline_css` to `style_message` as suggested

Set border=0 on that img

IE9 shows a big nasty blue border on linked images if you don't specify border=0

Convert client applications fixtures into a factory

Closes #1415

Merge remote-tracking branch 'openstreetmap/pull/1416'

Load user fixtures for ChangesetComment model tests

Localisation updates from https://translatewiki.net.

Left-align logo

Apply inline styling to all <p> tags

Solved the problem of <p> tags being given different padding or color by different clients, without having to copy-paste a string of CSS into every <p> tag in the templates.

Remove unused user_tokens fixture and pointless test.

Put the user avatar in a <p> block

This improves rendering on those clients that set the margin to 0 on <p> blocks.

Update to leaflet 1.0.3

Localisation updates from https://translatewiki.net.

Update Potlatch 2 to 2.4-32-gecf1e0b build

Localisation updates from https://translatewiki.net.

Rubocop says to use "hash rockets"

Rubocop will have its way.

Convert those <nobr> tags to CSS

Refactored the code to invoke the message body partial layout, as suggested in the PR

Move attach_project_logo to be its own before_action filter, as suggested

Renamed @user_message_author to @author as suggested

Localisation updates from https://translatewiki.net.

Revert "Added a greeting to the changeset comment message"

This reverts commit 39c3fa427ea273e9011c380f9d2f2a9759cddb24.

Remove that ugly style="margin: 0"

It's really only needed for rendering in Yahoo. Let's try to do without having to copy-paste that into every html template.

Simplified the partial layout

Rather than wrap the body in a :body block, just make it the default, unnamed block, which keeps the markup more lightweight.

Logo was being attached twice on changeset comment notifications

Dressed up note comment messages

Dressed up lost password mail

Dressed up gpx notification mails

Dressed up friend_notification mail

Added a greeting to the changeset comment message

The greeting includes the recipient's user name (as for the message notification mail), which should increase the recipient's trust that the message isn't spam. Greetings were reused from each language's respective locale flie.

Rubocop finds 'return' statements redundant

Modify tests to search only text parts

The tests assumed that every part in the multipart emails that we send were either plain text or HTML. Now we have image attachments, against whose contents the tests were still trying to match regexes. The tests have been modified to only run regexes on the text parts of the mails.

Removed unbalanced tag HTML tag

Don't try and look up traces until the user is logged in

Fixes #1411

Localisation updates from https://translatewiki.net.

Dressed up a few more messages.

email_confirm, message_notification and signup_confirm now use the new templates. This fixes some, but not all, of the currently broken tests.

Moved message table HTMl to a partial template

Several notifications are about a message having been sent from one user to another via OSM. I've moved the code for the HTML table that holds the actual user message, with the avatar and the body text, to its own partial template, so that it can be reused. I've updated a second notifier message to the new template, diary_comment_notification.

First steps towards using layouts

Since all HTML mails will presumably share the same overall layout, and the markup is fairly dense, it makes sense to use Rails layouts to share the structure across mail templates. This commit moves the core structure of the HTML mail to a layout. It will need to be further refined so that notifications that involve a text message sent by another user can share the avatar-and-message-text structure.

Place GPS tiles in the overlayPane

Localisation updates from https://translatewiki.net.

Use https compatible names for GPS tile servers

Add public gps tiles layer for debugging purposes

Fixes #1239
Closes #1406

Remove HTML markup from locale file

Rather than including HTML markup in the locale file (which also has the inconvenience of requiring both a plain-text and an HTML duplicate of the same string), move the markup to the template. Also added a helper to reduce clutter in the template slightly.

More dependency tweaks for gem2.3 and apache2-dev

Closes #1405

ruby2.0->ruby 2.3

change dependancies. nokogiri requires a later ruby now as pointed out here: https://github.com/openstreetmap/openstreetmap-website/issues/1402 ruby2.0->ruby 2.3 ...which then entails a later ubuntu if we're installing from packages.

Switch vagrant Ubuntu Trusty -> Xenial

More compact image paths

Make the file paths to image assets more compact. I investigated using `image_path` but could only get it to return the path for a public URL, which is different and also includes the asset pipeline digest.

Make export action send TOTP cookie

Make export form send authenticity token

Proper way to get path to small avatar

Rather than performing perilous string substitution on paths, just use the built-in way to select the small version of the avatar image file.

Remove dot prefix from cookie domain

Reduce TOTP cookie expiry to one hour

Add support for generating TOTP cookies

This allows other sites in the openstreetmap.org domain to validate
that requests are coming from a www.openstreetmap.org user.

Localisation updates from https://translatewiki.net.

Fixed quotes

Replaced single quotes with double quotes, to comply with rubocop rules.

Remove unused variable

Use parameterised server root URL rather than hardcoded

Remove unused import

Move localised text to locale file

Moved a string of English text that will require translation from the HTML template to the locale I18N file.

Better loading of attached images

Got rid of that big ugly base64 blob which was only a temporary crutch anyway. Added a png file instead. Use File.read rather than File.open for a 66% reduction in verbosity.

Update bundle

Tweaked cell padding

Tweaked cell padding to align the top of the avatar and of the message text a bit better.

Cleaned up HTML

Removed some cargo-cult HTML attributes and CSS. Styling HTML for email clients is trickier than for browsers, I'm trying to keep the code as succinct but also as compatible as possible.