-
-
Notifications
You must be signed in to change notification settings - Fork 366
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cycle barrier installation quest #4406
Cycle barrier installation quest #4406
Conversation
there are like... 2 |
Sure, it seems rare enough (at the moment), so we can drop it if you like, and have user leave a note instead.
I cannot say what is the significant difference in use as I was not involved in tag creation (if it was up to me, both for But we seem to ask that difference at StreetComplete/app/src/main/java/de/westnordost/streetcomplete/quests/bollard_type/BollardType.kt Lines 5 to 11 in a4d68fe
Let me know how you'd prefer to handle this one (it's all the same to me, this is just a first draft). |
bollard is an established tagging scheme already, |
When I proposed Currently, the values "removable" and "openable" are used fairly in the same quantity - but I don't know if this is also due to a "wrong" understanding of the term "removable". However, I think these values should appear in the quest/should be distinguished from each other to keep consistency with the bollard key, but others ("foldable") are not needed because of their rarity. |
To be consistent with the bollard type quest, it should be "What type of cycle barrier is this?". But we have exactly the same question for the cycle barrier type (single/double/triple/tilted/...). Could both quests have the same question or would this be confusing and misleading to users? |
It would be confusing. |
Hm, I was aiming to showing concrete example how it might usually look like so it would be instantly recognizable, and providing info about "what they should look for" via text overlay. But perhaps that is the wrong approach.
Something like this mockup would be more OK with you then @westnordost? If yes, I can do that. While I certainly see some advantages there, I'd like other opinions which version seems more understandable to novice user.
Yes, there is. I could use for example this one, if you think that would be better (and save few kB) ?
It used to be 3 rows, but I have reduced it to two rows since we had 4 answers which both fit 2x2 answers, and provide somewhat larger images to hopefully make detail little easier to see. Now that we're back to 3 answers, I can certainly revert to 3 rows in one line, if you find that bigger pictures are more of a distraction then help.
I did try to position the picture so text at the bottom does not obscure any important part of the picture. While it could be a little better, I'm limited by source pictures too, and there is a lot of clarification texts, so all things considered it does not look too bad to me (I might be biased though).
As it is, that absolute freedom (in my case) is quite limited by my technical abilities (i.e. I can copy and paste from other quest if it is not too complicated, but I can not really invent what I would like - e.g. even seemingly easy changes like having text on upper part of 3x1 image selection [instead of on lower part] are likely beyond my current abilities). |
This one is also a good photo for |
Maybe it would be helpful to show an animated image (GIF) or an animated illustration of the functionality of the gate instead of a static picture? |
#4406 (comment) is better. For consistency, should also show the fixed bicycle barrier from close up. Note about with or without key is necessary? Because, what if you need a key for openable too? I do not like your last mockup with swing gate + hinged upwards in one picture. Also, is it really necessary to mention how it can be opened? |
ok, will try that.
I don't know. It does not seem that important to me. It was copied from I think that was related to possible confusion of users who might not call something removable if it is locked (or something like that). I'm more than happy to reduce the text here; but if it is unwanted here, we might also consider if there is benefit for those texts in
yeah, that one would definitely get too long with both explanations.
It's ok, I'm not a big fan of it myself (although I'm sure someone with more graphic sense could make a much nicer version). I was experimenting because @legofahrrad seems to thinks both pictures should be shown to users or else they might not realize that picture of horizontal-swing
See above (especially if the picture shows only one Extra texts are also there to address @SupaplexOSM concern that But I myself am content also with terse |
OK, here is test APK, which compiles and works fine for me. Does it now looks better @westnordost ? I've done:
|
Yes, it looks better! Is this ready for review? |
Ready for review now. Note that it still misses a different icon if wanted (currently it reuses |
Any ideas for an icon? |
Nothing particularly good... Only that it might be based on (However there is no precedent - for If we were to expand on Dunno, I'm not really good with that artistic stuff 🥲 , perhaps somebody else have an idea? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code looks good, two nitpicks
.../streetcomplete/quests/barrier_bicycle_barrier_installation/AddBicycleBarrierInstallation.kt
Outdated
Show resolved
Hide resolved
.../streetcomplete/quests/barrier_bicycle_barrier_installation/AddBicycleBarrierInstallation.kt
Outdated
Show resolved
Hide resolved
How should the average SC user know that the “hinge to the top” is also tagged as |
I'm guessing from the word itself. And if Notes start popping up asking for clarification (it can be checked a month of few after this gets released, for example on https://ent8r.github.io/NotesReview/), one can open a new issue so it can be improved. Feel free to do so then @legofahrrad! (I'd be happy to help with guidance on how to do it) Because, if we insist on 100% absolute perfection on the first try, the PR will simply never get released 😺 |
Closes #4293
The code is implemented and seems to compile and work fine (debug apk available here); however:
foldable
cycle barrier at all, I took zoomed-in part of one from bollard as an example), withauthors.txt
update.ic_quest_no_bicycles
So if someone could find/take/produce square images of the features, that would be greatly appreciated, as well as any comments on the other issues.
Currently it looks like this (so you can see above issues, e.g. text covering important parts of the images, etc.):
![small_Screenshot_20220923_033307_de westnordost streetcomplete mn debug](https://user-images.githubusercontent.com/156656/191877594-3a1fc3f5-6719-4496-bc67-935a087b7f5c.jpg)