[openstreetmap/openstreetmap-website] Add user_<uid> class to diary entry div (#2266)

Andy Allan notifications at github.com
Wed Aug 28 13:24:14 UTC 2019


I've ummed and aahhhed about this for weeks, since I both object to it ([for the same reasons as @tomhughes](https://github.com/openstreetmap/openstreetmap-website/pull/2266#issuecomment-502678401)) and also support it since it's pretty trivial and [nobody's going to mind if we break it in future](https://github.com/openstreetmap/openstreetmap-website/pull/2266#issuecomment-502684786). But it's not worth me considering it for any longer.

So it just comes down to having a slight bias for change. But if this gets stripped out again in a future refactoring, so be it.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/2266#issuecomment-525743518
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20190828/031ba488/attachment.html>


More information about the rails-dev mailing list