Closed dorono closed 7 years ago
Maybe we should have a max width on the form. Doesn't necessarily need to be in this PR.
Eventually we can put this in a modal and save the state with the add-event
url.
@pfarnach Totally agree about putting the routes into constants, much cleaner. As for the max-width, I do indeed see one on line 5 of src/components/AddEvent/AddEvent.sass:
max-width: $max-width
margin: 0 auto
padding: $base-spacing
I also see that the width of the form matches the width of the header looking at the site in the browser. Or am I maybe understanding?
At any rate, gotta hit the sack for an early trip out of town tomorrow, but if it looks good to you otherwise, my vote is to merge to keep it from getting stale.
iPhone simulator view