-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Adding playground presets #4375
Conversation
What a fun PR! Per your todos:
I added a few other comments.. I know picking icons is tough, but I think we should not use the "entrance" icon for the slide, even though it does look kind of like a slide. Using the same icon for 2 very different concepts makes the map harder to use. |
@@ -0,0 +1,11 @@ | |||
{ | |||
"icon": "entrance", |
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.
Haha not sure how I feel about this one..
@@ -0,0 +1,5 @@ | |||
{ | |||
"key": "max_age", | |||
"type": "text", |
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.
min_age
and max_age
can both be fields and have type number
. This will add up/down buttons to the side.
"playground/baby": { | ||
"key": "baby", | ||
"type": "check", | ||
"label": "Baby Seat" |
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.
I think this is fine 👍
Closes #4252
Presets Added
Things to do
Add a new field for theswing
preset that says whether it has a special wheelchair seatbaby
field on theswing
presetI changed it from
Designed for babies
toBaby seat
, but I'm still unsure if I like this or notplayground
preset, such as operator and age restrictions