suwebdev / spring-ghost

Spring Ghost repository for WATS 4020 18SQ
MIT License
1 stars 2 forks source link

created web form #13

Open rosahughes opened 6 years ago

rosahughes commented 6 years ago

Issue https://github.com/suwebdev/spring-ghost/issues/7 Goal to have basic web form structure in place.

I copied this code from w3schools for a basic web form, but it uses a form element with a php action for processing. If this is not working, it can be removed.

laura-reeve commented 6 years ago

Hi Rosa - I tried to leave a block of code as part of a code review but then it reformatted instead of printing the code in the <pre> container. Hopefully this will help to explain, but let me know if you have any questions.

What I had basically done is to use this code in the first example here (WC3 Bootstrap Forms) to add Bootstrap styling to the form:

Just a few quick changes and you get a nice looking and responsive Bootstrap web form, but I would also support a merge without these changes. What do you think?

rosahughes commented 6 years ago

Laura, I like both the pre-selected location addition and the suggestion for the Bootstrap styling. I would do both before merging back to the master.