-
Notifications
You must be signed in to change notification settings - Fork 1
Issue/173 #228
base: main
Are you sure you want to change the base?
Issue/173 #228
Conversation
|
closes #173 |
|
Got all the install errors sorted with Jacob. Will finish testing no the morrow. |
|
Everything looks like it's working well. I think you should add some help text into the fields though |
patrickbrown-io
left a comment
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.
Works well! I agree with Josh that it could use some help text on the fields, I think specifically the steps and substeps could use it the most
jnicholCU
left a comment
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.
This installs and works well.
@jcsparks I forgot to tag you earlier to let you know this is good for testing. My only change would be to add information to the Help Text areas.
|
Jacob has added the help text in, everything installs correctly still |
patrickbrown-io
left a comment
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.
Help text is great, still installs clean!
One last thing I noticed, if you have particularly long step text, the image for that step gets crushed to the left as you scale screen size down. Maybe stack image on top of the step paragraph for md & sm screens so they're equal no matter what the length of the step is? I think with that responsive adjustment it'll be good to go!
|
Sounds like a great change, will definitely implement. |

https://search.google.com/test/rich-results