Add an OSM signup prompt to the reporting-method and app pages#133
Add an OSM signup prompt to the reporting-method and app pages#133munzzyy wants to merge 2 commits into
Conversation
Both reporting paths write to OpenStreetMap, but the only signup link lived in step 1 of the iD stepper, so anyone landing on /report or /app had no signal they needed an account until they hit the login wall. Adds a callout on the reporting-method page and a line under the app download CTA, both linking to openstreetmap.org/user/new. Closes FoggedLens#86.
|
Thanks! Chatting this over with @stopflock |
|
Thank you for this - had forgotten there was an issue open for it, but it came up again recently. The new blurb on the app page is not as prominent, being at the bottom. |
|
Done in e8ab8ab. Went with your wording, About link included, and moved the note up into the hero right under the download buttons instead of the bottom CTA, so it reads before anyone installs. Dropped the old bottom copy so it doesn't show twice. Also pointed the OpenStreetMap mention in the reporting-page alert at the About page to match. Built clean locally. |
|
LGTM. @frillweeman ? |
|
Thanks for looking it over, @stopflock. I'm around if @frillweeman wants anything changed — it's just the signup-prompt copy and the conditional around it, so it's easy to adjust. |
Closes #86.
Both reporting methods write to OpenStreetMap, but the only "create an account" link on the site is buried in step 1 of the iD stepper on
/report/id. Someone landing on/reportor/appgets no hint they need an account until they hit the login wall. That's the gap the issue is pointing at.Two changes:
ReportChoose.vue— an infov-alertunder the intro line, before the two method cards, saying both paths save to OSM and linking toopenstreetmap.org/user/new.DeflockApp.vue— a line under the download CTA noting the same thing, since the issue asked about the app page too. That section sits on the purple gradient, so there's a small scoped rule making the link inherit white instead of rendering default-blue on purple.Wording matches the existing link on
/report/id. No new deps, no state, no telemetry.Checked with
npm run build-only(no bun here, andpackage-lock.jsonis in the repo so npm works) and rendered both pages againstvite dev.npm run type-checkdoes fail, but onApp.vueand a@vitejs/plugin-vuetype declaration. Both are already broken onmasterand neither is touched here.