]> git.openstreetmap.org Git - rails.git/log
rails.git
7 years agomerging master into html-mail-fixes
Herve Saint-Amand [Sun, 12 Feb 2017 21:05:38 +0000 (21:05 +0000)]
merging master into html-mail-fixes

7 years agoAllow context menu to add notes without panning the map
Tom Hughes [Sun, 12 Feb 2017 17:26:17 +0000 (17:26 +0000)]
Allow context menu to add notes without panning the map

7 years agoUpdate Leaflet.contextmenu to the 1.2.1 release
Tom Hughes [Sun, 12 Feb 2017 15:50:23 +0000 (15:50 +0000)]
Update Leaflet.contextmenu to the 1.2.1 release

7 years agoMake context menu entries translateable
Tom Hughes [Sun, 12 Feb 2017 15:03:56 +0000 (15:03 +0000)]
Make context menu entries translateable

7 years agoImprove context menu initialisation to avoid namespace pollution
Tom Hughes [Sun, 12 Feb 2017 14:59:28 +0000 (14:59 +0000)]
Improve context menu initialisation to avoid namespace pollution

7 years agoMerge remote-tracking branch 'openstreetmap/pull/1030' into next
Tom Hughes [Sun, 12 Feb 2017 15:04:50 +0000 (15:04 +0000)]
Merge remote-tracking branch 'openstreetmap/pull/1030' into next

7 years agoA better way to contrain the width of the message box whilst also allowing it to...
Herve Saint-Amand [Sun, 12 Feb 2017 12:04:43 +0000 (12:04 +0000)]
A better way to contrain the width of the message box whilst also allowing it to shrink on narrow displays

Setting a hard "width" it very compatible, but won't squeeze on narrow screens. Setting max-width works well, except in Outlook and Hotmail. We resort to old skool table tricks: a table with width=100%, with 3 columns, only oneof which, the middle one, had content, and a fixed with. This works.

7 years agoTwo more "nowrap" instructions removed to accommodate narrower displays
Herve Saint-Amand [Sat, 11 Feb 2017 21:08:36 +0000 (21:08 +0000)]
Two more "nowrap" instructions removed to accommodate narrower displays

7 years agoRemove nowrap on footer text, to accommodate narrow screens
Herve Saint-Amand [Sat, 11 Feb 2017 20:51:03 +0000 (20:51 +0000)]
Remove nowrap on footer text, to accommodate narrow screens

7 years agoSet font on title
Herve Saint-Amand [Sat, 11 Feb 2017 20:50:28 +0000 (20:50 +0000)]
Set font on title

7 years agoRemove invalid YML
Nick Ingalls [Thu, 9 Feb 2017 21:40:46 +0000 (16:40 -0500)]
Remove invalid YML

Closes #1434

7 years agoMake the mails squeezable on narrow screens
Herve Saint-Amand [Thu, 9 Feb 2017 21:39:49 +0000 (21:39 +0000)]
Make the mails squeezable on narrow screens

7 years agoOnly add traces for POST requests
Tom Hughes [Thu, 9 Feb 2017 21:07:44 +0000 (21:07 +0000)]
Only add traces for POST requests

7 years agoOnly create diary entries for POST requests
Tom Hughes [Thu, 9 Feb 2017 21:07:18 +0000 (21:07 +0000)]
Only create diary entries for POST requests

7 years agoTest that GET on message#new can't send a message
Tom Hughes [Thu, 9 Feb 2017 20:39:03 +0000 (20:39 +0000)]
Test that GET on message#new can't send a message

7 years agoFixed alignment & underlining bugs in Outlook / WinMail
Herve Saint-Amand [Thu, 9 Feb 2017 20:38:02 +0000 (20:38 +0000)]
Fixed alignment & underlining bugs in Outlook / WinMail

7 years agoRender message properly on error
Tom Hughes [Thu, 9 Feb 2017 20:20:55 +0000 (20:20 +0000)]
Render message properly on error

7 years agoOnly send messages for POST requests
Tom Hughes [Thu, 9 Feb 2017 19:37:48 +0000 (19:37 +0000)]
Only send messages for POST requests

7 years agoRemove fake qqq locale
Tom Hughes [Thu, 9 Feb 2017 08:47:59 +0000 (08:47 +0000)]
Remove fake qqq locale

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Thu, 9 Feb 2017 07:16:23 +0000 (08:16 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoAvoid squished-up grey borders around the message on narrow screens
Herve Saint-Amand [Wed, 8 Feb 2017 23:23:46 +0000 (23:23 +0000)]
Avoid squished-up grey borders around the message on narrow screens

7 years agoEven out the widths of the white bars around the message box
Herve Saint-Amand [Wed, 8 Feb 2017 23:00:52 +0000 (23:00 +0000)]
Even out the widths of the white bars around the message box

7 years agoBetter alt text for the logo
Herve Saint-Amand [Wed, 8 Feb 2017 22:37:00 +0000 (22:37 +0000)]
Better alt text for the logo

This is what clients that block images will display, so shorter is better.

7 years agoUse the same fonts for the footer text as for the rest of the mail
Herve Saint-Amand [Wed, 8 Feb 2017 22:36:21 +0000 (22:36 +0000)]
Use the same fonts for the footer text as for the rest of the mail

7 years agoFix title alignment in Outlook/WindowsMail
Herve Saint-Amand [Wed, 8 Feb 2017 22:35:42 +0000 (22:35 +0000)]
Fix title alignment in Outlook/WindowsMail

7 years agoDon't display a box at all if the note event doesn't include a comment body
Herve Saint-Amand [Wed, 8 Feb 2017 22:22:21 +0000 (22:22 +0000)]
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.

7 years agoRemoved italic styling on message body
Herve Saint-Amand [Wed, 8 Feb 2017 12:02:17 +0000 (12:02 +0000)]
Removed italic styling on message body

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

7 years agoForce the font on every <p> tag
Herve Saint-Amand [Fri, 3 Feb 2017 21:22:59 +0000 (21:22 +0000)]
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.

7 years agoTweak cell width for proper centering on narrow screens
Herve Saint-Amand [Mon, 30 Jan 2017 21:40:20 +0000 (21:40 +0000)]
Tweak cell width for proper centering on narrow screens

7 years agoOnly record a visible object if it exists
Tom Hughes [Wed, 8 Feb 2017 09:53:30 +0000 (09:53 +0000)]
Only record a visible object if it exists

Fixes #1430

7 years agoUpdate to iD v2.1.2
Bryan Housel [Tue, 7 Feb 2017 16:53:11 +0000 (11:53 -0500)]
Update to iD v2.1.2

7 years agoUpdate to iD v2.1.1
Bryan Housel [Mon, 6 Feb 2017 22:08:25 +0000 (17:08 -0500)]
Update to iD v2.1.1

7 years agoMerge remote-tracking branch 'openstreetmap/pull/1426'
Tom Hughes [Mon, 6 Feb 2017 19:10:25 +0000 (19:10 +0000)]
Merge remote-tracking branch 'openstreetmap/pull/1426'

7 years agoRemove fake qqq locale
Tom Hughes [Mon, 6 Feb 2017 07:11:25 +0000 (07:11 +0000)]
Remove fake qqq locale

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Mon, 6 Feb 2017 07:05:21 +0000 (08:05 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoLink GPX fixture files instead of stubbing
Tom Hughes [Sun, 5 Feb 2017 17:24:58 +0000 (17:24 +0000)]
Link GPX fixture files instead of stubbing

7 years agoCleanup double comment markers
Tom Hughes [Sun, 5 Feb 2017 17:25:54 +0000 (17:25 +0000)]
Cleanup double comment markers

7 years agoMake TraceController#list sort by id instead of timestamp
Tom Hughes [Sun, 5 Feb 2017 16:18:56 +0000 (16:18 +0000)]
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.

7 years agoMerge remote-tracking branch 'openstreetmap/pull/1347' into master
Tom Hughes [Sun, 5 Feb 2017 15:56:04 +0000 (15:56 +0000)]
Merge remote-tracking branch 'openstreetmap/pull/1347' into master

7 years agoFix rubocop warnings
Tom Hughes [Sun, 5 Feb 2017 14:33:48 +0000 (14:33 +0000)]
Fix rubocop warnings

7 years agoUse .first in place of [0]
Tom Hughes [Sun, 5 Feb 2017 13:57:25 +0000 (13:57 +0000)]
Use .first in place of [0]

7 years agoMerge remote-tracking branch 'openstreetmap/pull/1401'
Tom Hughes [Sun, 5 Feb 2017 13:50:07 +0000 (13:50 +0000)]
Merge remote-tracking branch 'openstreetmap/pull/1401'

7 years agoUpdate bundle
Tom Hughes [Sun, 5 Feb 2017 11:13:21 +0000 (11:13 +0000)]
Update bundle

7 years agoUpdate rubocop
Tom Hughes [Sun, 5 Feb 2017 11:02:10 +0000 (11:02 +0000)]
Update rubocop

7 years agoUpdate to iD v2.1.0
Bryan Housel [Sat, 4 Feb 2017 06:18:42 +0000 (01:18 -0500)]
Update to iD v2.1.0

7 years agoMerge remote-tracking branch 'openstreetmap/pull/1423'
Tom Hughes [Thu, 2 Feb 2017 17:21:41 +0000 (17:21 +0000)]
Merge remote-tracking branch 'openstreetmap/pull/1423'

7 years agoConvert the test_users_nearby to user factories.
Andy Allan [Thu, 2 Feb 2017 13:52:26 +0000 (13:52 +0000)]
Convert the test_users_nearby to user factories.

7 years agoUse factory for user language tests
Andy Allan [Thu, 2 Feb 2017 13:33:15 +0000 (13:33 +0000)]
Use factory for user language tests

7 years agoRefactor the friend_users test, and remove the tests which duplicate the (renamed...
Andy Allan [Thu, 2 Feb 2017 13:24:46 +0000 (13:24 +0000)]
Refactor the friend_users test, and remove the tests which duplicate the (renamed) test_friends_with above.

7 years agoConvert test_friend_with to use factories, and use alice/bob/charlie for easier under...
Andy Allan [Thu, 2 Feb 2017 13:20:51 +0000 (13:20 +0000)]
Convert test_friend_with to use factories, and use alice/bob/charlie for easier understanding.

7 years agoUser factory for uniqueness tests
Andy Allan [Thu, 2 Feb 2017 13:09:39 +0000 (13:09 +0000)]
User factory for uniqueness tests

7 years agoConvert class method tests to use User factory.
Andy Allan [Thu, 2 Feb 2017 13:05:37 +0000 (13:05 +0000)]
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.

7 years agoAdd status traits to user factory.
Andy Allan [Thu, 2 Feb 2017 12:55:32 +0000 (12:55 +0000)]
Add status traits to user factory.

7 years agoAdd moderator_user and administrator_user factories.
Andy Allan [Thu, 2 Feb 2017 12:38:19 +0000 (12:38 +0000)]
Add moderator_user and administrator_user factories.

7 years agoAdd a user_role factory
Andy Allan [Thu, 2 Feb 2017 12:35:29 +0000 (12:35 +0000)]
Add a user_role factory

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

7 years agoAdd with_home_location trait for user factories.
Andy Allan [Thu, 2 Feb 2017 12:08:36 +0000 (12:08 +0000)]
Add with_home_location trait for user factories.

7 years agoCreate a simple user factory, and convert some basic tests.
Andy Allan [Wed, 1 Feb 2017 17:42:05 +0000 (17:42 +0000)]
Create a simple user factory, and convert some basic tests.

7 years agoReduce maximum changeset size to 10k changes
Frederik Ramm [Sun, 31 Jul 2016 21:32:27 +0000 (23:32 +0200)]
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

7 years agoRemove fake qqq locale
Tom Hughes [Mon, 30 Jan 2017 16:01:52 +0000 (16:01 +0000)]
Remove fake qqq locale

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Mon, 30 Jan 2017 15:50:12 +0000 (16:50 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoThat new message in the HTML version of the mail should also be added to the plain...
Herve Saint-Amand [Sat, 28 Jan 2017 23:29:44 +0000 (23:29 +0000)]
That new message in the HTML version of the mail should also be added to the plain text version of the mail

Duh.

7 years agoUse each_with_object and make the code pithier
Herve Saint-Amand [Sat, 28 Jan 2017 23:27:04 +0000 (23:27 +0000)]
Use each_with_object and make the code pithier

7 years agoRenamed `apply_inline_css` to `style_message` as suggested
Herve Saint-Amand [Sat, 28 Jan 2017 23:08:25 +0000 (23:08 +0000)]
Renamed `apply_inline_css` to `style_message` as suggested

7 years agoSet border=0 on that img
Herve Saint-Amand [Fri, 27 Jan 2017 23:20:11 +0000 (23:20 +0000)]
Set border=0 on that img

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

7 years agoConvert client applications fixtures into a factory
Andy Allan [Thu, 26 Jan 2017 18:03:13 +0000 (18:03 +0000)]
Convert client applications fixtures into a factory

Closes #1415

7 years agoMerge remote-tracking branch 'openstreetmap/pull/1416'
Tom Hughes [Thu, 26 Jan 2017 17:51:13 +0000 (17:51 +0000)]
Merge remote-tracking branch 'openstreetmap/pull/1416'

7 years agoLoad user fixtures for ChangesetComment model tests
Tom Hughes [Thu, 26 Jan 2017 17:50:11 +0000 (17:50 +0000)]
Load user fixtures for ChangesetComment model tests

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Thu, 26 Jan 2017 10:16:34 +0000 (11:16 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoLeft-align logo
Herve Saint-Amand [Wed, 25 Jan 2017 23:01:37 +0000 (23:01 +0000)]
Left-align logo

7 years agoApply inline styling to all <p> tags
Herve Saint-Amand [Wed, 25 Jan 2017 22:50:52 +0000 (22:50 +0000)]
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.

7 years agoRemove unused user_tokens fixture and pointless test.
Andy Allan [Wed, 25 Jan 2017 16:43:04 +0000 (16:43 +0000)]
Remove unused user_tokens fixture and pointless test.

7 years agoPut the user avatar in a <p> block
Herve Saint-Amand [Mon, 23 Jan 2017 22:26:34 +0000 (22:26 +0000)]
Put the user avatar in a <p> block

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

7 years agoUpdate to leaflet 1.0.3
Tom Hughes [Mon, 23 Jan 2017 12:30:11 +0000 (12:30 +0000)]
Update to leaflet 1.0.3

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Mon, 23 Jan 2017 12:15:00 +0000 (13:15 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoUpdate Potlatch 2 to 2.4-32-gecf1e0b build
Tom Hughes [Thu, 19 Jan 2017 15:03:56 +0000 (15:03 +0000)]
Update Potlatch 2 to 2.4-32-gecf1e0b build

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Thu, 19 Jan 2017 04:47:17 +0000 (05:47 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoRubocop says to use "hash rockets"
Herve Saint-Amand [Mon, 16 Jan 2017 23:12:59 +0000 (23:12 +0000)]
Rubocop says to use "hash rockets"

Rubocop will have its way.

7 years agoConvert those <nobr> tags to CSS
Herve Saint-Amand [Mon, 16 Jan 2017 22:53:04 +0000 (22:53 +0000)]
Convert those <nobr> tags to CSS

7 years agoRefactored the code to invoke the message body partial layout, as suggested in the PR
Herve Saint-Amand [Mon, 16 Jan 2017 22:50:01 +0000 (22:50 +0000)]
Refactored the code to invoke the message body partial layout, as suggested in the PR

7 years agoMove attach_project_logo to be its own before_action filter, as suggested
Herve Saint-Amand [Mon, 16 Jan 2017 21:21:55 +0000 (21:21 +0000)]
Move attach_project_logo to be its own before_action filter, as suggested

7 years agoRenamed @user_message_author to @author as suggested
Herve Saint-Amand [Mon, 16 Jan 2017 21:13:52 +0000 (21:13 +0000)]
Renamed @user_message_author to @author as suggested

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Mon, 16 Jan 2017 02:38:06 +0000 (03:38 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoRevert "Added a greeting to the changeset comment message"
Herve Saint-Amand [Sat, 14 Jan 2017 21:48:46 +0000 (21:48 +0000)]
Revert "Added a greeting to the changeset comment message"

This reverts commit 39c3fa427ea273e9011c380f9d2f2a9759cddb24.

7 years agoRemove that ugly style="margin: 0"
Herve Saint-Amand [Fri, 13 Jan 2017 22:58:57 +0000 (22:58 +0000)]
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.

7 years agoSimplified the partial layout
Herve Saint-Amand [Fri, 13 Jan 2017 22:43:23 +0000 (22:43 +0000)]
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.

7 years agoLogo was being attached twice on changeset comment notifications
Herve Saint-Amand [Fri, 13 Jan 2017 21:45:52 +0000 (21:45 +0000)]
Logo was being attached twice on changeset comment notifications

7 years agoDressed up note comment messages
Herve Saint-Amand [Thu, 12 Jan 2017 23:02:57 +0000 (23:02 +0000)]
Dressed up note comment messages

7 years agoDressed up lost password mail
Herve Saint-Amand [Thu, 12 Jan 2017 22:30:31 +0000 (22:30 +0000)]
Dressed up lost password mail

7 years agoDressed up gpx notification mails
Herve Saint-Amand [Thu, 12 Jan 2017 22:28:09 +0000 (22:28 +0000)]
Dressed up gpx notification mails

7 years agoDressed up friend_notification mail
Herve Saint-Amand [Thu, 12 Jan 2017 22:11:23 +0000 (22:11 +0000)]
Dressed up friend_notification mail

7 years agoAdded a greeting to the changeset comment message
Herve Saint-Amand [Thu, 12 Jan 2017 21:52:17 +0000 (21:52 +0000)]
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.

7 years agoRubocop finds 'return' statements redundant
Herve Saint-Amand [Wed, 11 Jan 2017 23:26:56 +0000 (23:26 +0000)]
Rubocop finds 'return' statements redundant

7 years agoModify tests to search only text parts
Herve Saint-Amand [Wed, 11 Jan 2017 23:17:24 +0000 (23:17 +0000)]
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.

7 years agoRemoved unbalanced tag HTML tag
Herve Saint-Amand [Wed, 11 Jan 2017 22:03:00 +0000 (22:03 +0000)]
Removed unbalanced tag HTML tag

7 years agoDon't try and look up traces until the user is logged in
Tom Hughes [Wed, 11 Jan 2017 21:11:37 +0000 (21:11 +0000)]
Don't try and look up traces until the user is logged in

Fixes #1411

7 years agoLocalisation updates from https://translatewiki.net.
Niklas Laxström [Wed, 11 Jan 2017 18:37:30 +0000 (19:37 +0100)]
Localisation updates from https://translatewiki.net.

7 years agoDressed up a few more messages.
Herve Saint-Amand [Mon, 9 Jan 2017 23:42:43 +0000 (23:42 +0000)]
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.

7 years agoMoved message table HTMl to a partial template
Herve Saint-Amand [Sun, 8 Jan 2017 22:09:10 +0000 (22:09 +0000)]
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.

7 years agoFirst steps towards using layouts
Herve Saint-Amand [Fri, 6 Jan 2017 22:37:28 +0000 (22:37 +0000)]
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.

7 years agoPlace GPS tiles in the overlayPane
Tom Hughes [Thu, 5 Jan 2017 08:45:59 +0000 (08:45 +0000)]
Place GPS tiles in the overlayPane