[openstreetmap/openstreetmap-website] Bump rack from 3.1.15 to 3.1.16 (PR #6088)
Bumps [rack](https://github.com/rack/rack) from 3.1.15 to 3.1.16.
[openstreetmap/openstreetmap-website] Bump @stylistic/eslint-plugin-js from 4.4.0 to 4.4.1 (PR #6087)
Bumps [@stylistic/eslint-plugin-js](https://github.com/eslint-stylistic/eslint-stylistic/tree/HEAD/packages/eslint-plugin-js) from 4.4.0 to 4.4.1.Release notes
Sourced from
@​stylis
Re: [openstreetmap/openstreetmap-website] Reuse bounds size functions (PR #6076)
hlfan left a comment (openstreetmap/openstreetmap-website#6076) > Why do you want to add a Leaflet dependency to the code that doesn't use > Leaflet? Good question, I removed it! -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6
Re: [openstreetmap/openstreetmap-website] Profile edit tabs (PR #6084)
tomhughes left a comment (openstreetmap/openstreetmap-website#6084) So this is what a user's own profile looks like with the latest proposal:  I'm not certain I like it so I'd be interested to hear what othe
Re: [openstreetmap/openstreetmap-website] make sure iD's iframe is focused on startup (PR #6083)
@tomhughes commented on this pull request. > @@ -7,5 +7,5 @@ data[:zoom] = @zoom if @zoom data[:gpx] = trace_data_url(params[:gpx], :format => :xml) if params[:gpx] data[:url] = id_url(:locale => params[:locale]) %> - <%= tag.iframe "", :frameBorder => 0, :id => "id-embed", :c
Re: [openstreetmap/openstreetmap-website] Truncate rich text description on word boundaries if possible (PR #6071)
Merged #6071 into master. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6071#event-17986272583 You are receiving this because you are subscribed to this thread. Message ID: ___ rails-
Re: [openstreetmap/openstreetmap-website] Truncate rich text description on word boundaries if possible (PR #6071)
tomhughes left a comment (openstreetmap/openstreetmap-website#6071) This looks good to me, thanks. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6071#issuecomment-2940846150 You are receiving this because you are subscribed to t
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
NeatNit left a comment (openstreetmap/openstreetmap-website#6086) > Please understand that installing and running the code in this repository is > requires a degree of technical ability - it's not really something that > non-technical end users are likely to find very easy. Of course, I underst
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
tomhughes left a comment (openstreetmap/openstreetmap-website#6086) Try something like this in he top level of your checkout: ``` bundle config set path vendor/bundle ``` which should configure bundler to install to the `vendor/bundle` directory. -- Reply to this email directly or view it on G
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
tomhughes left a comment (openstreetmap/openstreetmap-website#6086) Did my answer not support you by trying to address the problem and explain what is happening and possible solutions? I'm not sure why your bundler hasn't prompted for sudo access as my understanding was that modern versions sho
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
NeatNit left a comment (openstreetmap/openstreetmap-website#6086) @tomhughes I was under the impression that the INSTALL.md instructions should at least work: https://github.com/openstreetmap/openstreetmap-website/issues/5733#issuecomment-2691272414 > we try to support developers with fresh ins
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
Closed #6086 as completed. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/issues/6086#event-17985191705 You are receiving this because you are subscribed to this thread. Message ID: ___ rai
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
tomhughes left a comment (openstreetmap/openstreetmap-website#6086) Firstly this has nothing to do with snap. As to the error - if you want bundler to install to the system directory then you'll need to run it as root, or provide the password when it (via sudo) asks for it. If you want it to in
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
NeatNit left a comment (openstreetmap/openstreetmap-website#6086) Maybe the COPY lines are needed? https://github.com/NeatNit/openstreetmap-website/blob/48f864f98a2cd072d7730036a43d5c6aeae13bc4/Dockerfile#L44-L51 I've never used Docker so I'm pretty clueless, honestly. -- Reply to this email d
[openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
NeatNit created an issue (openstreetmap/openstreetmap-website#6086) I'm on Linux Mint 22.1, which is based on Ubuntu 24.04, so I expected the install instructions to work. https://github.com/openstreetmap/openstreetmap-website/blob/48f864f98a2cd072d7730036a43d5c6aeae13bc4/INSTALL.md?plain=1#L12
Re: [openstreetmap/openstreetmap-website] bundle version mismatch - can't install with INSTALL.md instructions (Issue #6086)
kcne left a comment (openstreetmap/openstreetmap-website#6086) I'm not entirely sure about the installation instructions, but I'm confident that the Dockerfile image works, as I've used it for quite some time. You can take a look inside the Dockerfile to see if there are any additional steps be
Re: [openstreetmap/openstreetmap-website] Truncate rich text description on word boundaries if possible (PR #6071)
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6071) Splitting on `\b` requires agreeing with the regexp rules of `\b` on what a word is. For example "-" is not a word character. "abc-def" is two words with two boundaries inside: "abc" + "-def" and "abc-" + "def". -- Reply to
Re: [openstreetmap/openstreetmap-website] make sure iD's iframe is focused on startup (PR #6083)
@tyrasd pushed 1 commit. 145ee61e56b426f66f40acd5bea92db73f20c910 use autofocus attribute -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6083/files/d50614f6a9c5354a857048be0bff43ef30a5a44b..145ee61e56b426f66f40acd5bea92db73f20c910 You are receiving this becaus
Re: [openstreetmap/openstreetmap-website] make sure iD's iframe is focused on startup (PR #6083)
@tyrasd commented on this pull request. > @@ -7,5 +7,5 @@ data[:zoom] = @zoom if @zoom data[:gpx] = trace_data_url(params[:gpx], :format => :xml) if params[:gpx] data[:url] = id_url(:locale => params[:locale]) %> - <%= tag.iframe "", :frameBorder => 0, :id => "id-embed", :clas
Re: [openstreetmap/openstreetmap-website] make sure iD's iframe is focused on startup (PR #6083)
@hlfan commented on this pull request. > @@ -37,7 +37,8 @@ $(function () { id.attr("src", idData.url + "#" + params); id.ready(function () { -if (!this.contentWindow.location.href.startsWith(idData.url)) { +id.focus(); +if (this.contentWindow && !this.contentWindow.location.
Re: [openstreetmap/openstreetmap-website] make sure iD's iframe is focused on startup (PR #6083)
@HolgerJeromin approved this pull request. -- Reply to this email directly or view it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6083#pullrequestreview-2895610001 You are receiving this because you are subscribed to this thread. Message ID: __
Re: [openstreetmap/openstreetmap-website] make sure iD's iframe is focused on startup (PR #6083)
@tyrasd pushed 1 commit. c63df1d6b0b4358c06538f79640ed6e7595fe471 use autofocus attribute -- View it on GitHub: https://github.com/openstreetmap/openstreetmap-website/pull/6083/files/82c5034b0977fec2cf36b24eeb0a75aa30e4a905..c63df1d6b0b4358c06538f79640ed6e7595fe471 You are receiving this becaus
Re: [openstreetmap/openstreetmap-website] make sure iD's iframe is focused on startup (PR #6083)
tyrasd left a comment (openstreetmap/openstreetmap-website#6083) Ok, I changed it to use the `autofocus` attribute now. The most relevant browser with only "recent" implementation of the attribute is Firefox with its full implementation in early 2023. I guess that's acceptable. -- Reply to thi
