Skip to content

Add Guidepost Sports quest#496

Merged
Helium314 merged 16 commits intoHelium314:modifiedfrom
mcliquid:quest_guidepost_sport-467
Dec 24, 2023
Merged

Add Guidepost Sports quest#496
Helium314 merged 16 commits intoHelium314:modifiedfrom
mcliquid:quest_guidepost_sport-467

Conversation

@mcliquid
Copy link
Copy Markdown

Fixes #467

@mcliquid mcliquid changed the title First modifications of the recycling quest Add Guidepost Sports quest Nov 20, 2023
@Helium314
Copy link
Copy Markdown
Owner

Considering streetcomplete#5412, this PR should be done before the mentioned changes.
Do you want to continue here, or do you need anything from me?
As far as I understand from #467 you already have found some icons, so there should not be much work left.

and !disused
and !guidepost
""".toElementFilterExpression() }

This comment was marked as resolved.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would not include this globally. We also have a lot of maps for cycle routes, but only a very small proportion overall. I would say that would be almost spammy even for SCEE. If you want to, you can edit the quest query individually. What do you think?

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, this is ok too

@mcliquid
Copy link
Copy Markdown
Author

@mcliquid mcliquid marked this pull request as ready for review December 22, 2023 12:47
@mcliquid
Copy link
Copy Markdown
Author

@Helium314 What would be the easiest way to switch the title color from white to black?

@Helium314
Copy link
Copy Markdown
Owner

You will need to adjust the layout xml and set the new layout in the quest form in onCreate, e.g. imageSelector.cellLayoutId = R.layout.cell_icon_select_with_label_below
But note that it should also look ok with (nearly) black background

@mcliquid
Copy link
Copy Markdown
Author

Hm, should I leave it white instead?
I could add a background color to the icons instead.

@Helium314
Copy link
Copy Markdown
Owner

I noticed the recycling quest does have a text color suitable for the chosen theme, so there should be some cellLayoutId you can use.

@Helium314
Copy link
Copy Markdown
Owner

Do you still want to add the guidepost=simple other answer?

@mcliquid
Copy link
Copy Markdown
Author

@Helium314
Copy link
Copy Markdown
Owner

The answers should use actual sport, having Bicycle, Horse and Ski in there feels wrong.
The quest looks like it could actually be an OsmFilterQuestType instead of OsmElementQuestType.

Otherwise it looks good!

@mcliquid
Copy link
Copy Markdown
Author

Thank you for helping! I've switched to FilterQuestType and changed the strings. Changing MTB to "Mountain biking" feels wrong, so I've kept "MTB". The Olympic Committee calls it "Cycling Mountain Bike" - that's just even weirder.

@Helium314 Helium314 merged commit d3dab90 into Helium314:modified Dec 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New Quest: For which sport is this guidepost intended? (information=guidepost/route_marker)

3 participants