AntonKhorev left a comment (openstreetmap/openstreetmap-website#6089)
...or wider buttons with subtler borders:

--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/opens
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6089)
A version with secondary-color buttons:

--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstr
@AntonKhorev pushed 1 commit.
7800f7f086e8688d7226d26ce763d6279898e62a Move profile location and company to
sidebar
--
View it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6089/files/611deb67867afe9de6ba5d55e05b254f4ef5b706..7800f7f086e8688d7226d26ce763d6279898e62a
Yo
hlfan left a comment (openstreetmap/openstreetmap-website#6065)
> That's a change to how link ordering currently works.
Separating them into a section would fit quite well to #6089.
But that aside, what do you think of badges like in #6072?
--
Reply to this email directly or view it on GitHub:
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6089)
This is how it's going to look with edit buttons added:

--
Reply to this email directly or view it on GitHub:
https://github.com/opens
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6089)
> Then it will be a distinct "section without links" above the "section with
> links".
Maybe we'll add links from company to organized mapping wiki page.
--
Reply to this email directly or view it on GitHub:
https://github.c
tomhughes left a comment (openstreetmap/openstreetmap-website#6089)
I think the general idea here is nice, and should help with the edit buttons so
I guess we just need to figure out the best presentation.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/op
hlfan left a comment (openstreetmap/openstreetmap-website#6089)
Having both the home location and the company in one section would be cleaner
imo.
Then it will be a distinct "section without links" above the "section with
links".
--
Reply to this email directly or view it on GitHub:
https://gi
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6084)
We can do #6089 and have all edit buttons in the content body instead of trying
to squeeze them in the heading, like the current version of this PR does:

Thanks... This look good to me now.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6083#issuecomment-2945128748
You are receiving this because you are subscribed
AntonKhorev left a comment (openstreetmap/openstreetmap-website#6086)
I think `bundle` doesn't try to write to system directories if you install a
version of ruby using `rbenv`.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/6
hlfan left a comment (openstreetmap/openstreetmap-website#6084)
I don't think even a tooltip would make it discoverable enough, I think it
needs a label, either "Edit" again or at least the icon currently used for
`changeset.num_changes`.
--
Reply to this email directly or view it on GitHub:
h
@hlfan approved this pull request.
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6083#pullrequestreview-2900614794
You are receiving this because you are subscribed to this thread.
Message ID:
__
@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
@tyrasd pushed 1 commit.
ac07dbfd69b5831d9f5ad2502ac1086aa1d3c6d2 make sure iD's iframe is focused on
startup
--
View it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6083/files/145ee61e56b426f66f40acd5bea92db73f20c910..ac07dbfd69b5831d9f5ad2502ac1086aa1d3c6d2
You are
19 matches
Mail list logo