-
Notifications
You must be signed in to change notification settings - Fork 7
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
added new design #33
added new design #33
Conversation
|
@tridip1931 apologies for the delay on this one. I've been away. I really like this approach 👍. Will push copy changes just now and apart from the feedback Tejas has #32 (comment) this is good to go. |
Pardon me for the delay @tridip1931, will take a stab on the content by Friday. |
@tridip1931 thank you for your fixes. Here're few things we should do to get this MVP live:
|
Did a new thing here @geohacker Let me know if it does not work for you. The rationale behind this is I am following general UI pattern of how TOS are displayed. |
I'm ok with this, but I do think it's useful to have the CoC as a page which we can link to. I just made the modal wide and added a bit more padding. If you have time we should try to keep it as a page that's consistent with rest of the site. @tridip1931 I also noticed that the CSS I added to main.css got overridden in your last commit - do you have a build process or are all the source files committed? I added an app.css to override some of the bootstrap css. |
I'm going to merge this @tridip1931. This looks like a good time to get the new design out. Thank you for all your work! 🙇 |
Next Actions