-
Notifications
You must be signed in to change notification settings - Fork 123
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
1097 - Update Sign Up form UI #1194
Conversation
@Meri-MG Can you merge the main branch into this? The org setting's page link is fixed in main. You can then attach a test logo. The layout looks great!! Just want to make sure the logo displays properly. |
…into 1097-sign-up-form-update
Hi, @jmilljr24 , I merged main branch in but i dont see the new logo in the pages |
There isn't a default image. I just uploaded a photo from my computer. Any image you have available to attach should work that is mostly square shape as that is what logo's will most likely be. |
@Meri-MG how's this one going? Let us know if you have any questions. |
ohhh, it's been two weeks, for some reason I thought the branch was merged, will address @jmilljr24 feedback at the end of this week |
are we okay with this kind of layout? or should the image be smaller as below? @Meri-MG haha nice picture. Given it is a logo it should be small, like 30x30 pixels |
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.
@Meri-MG Looks good. Thanks
🔗 Issue
#1097
✍️ Description
Sign up form UI should look like Log in form UI.
📷 Screenshots/Demos