Skip to content
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

New forms style #104

Merged
merged 1 commit into from Jul 18, 2018
Merged

New forms style #104

merged 1 commit into from Jul 18, 2018

Conversation

Madeorsk
Copy link
Member

@Madeorsk Madeorsk commented Jun 30, 2018

Login example on phones and register example on a computer.
Mobile
Computer

@elegaanz elegaanz added C: Enhancement New feature or request A: Front-End Related to the front-end S: Ready for review This PR is ready to be reviewed Design UI/UX related issues and PRs labels Jun 30, 2018
@elegaanz elegaanz self-requested a review June 30, 2018 19:05
@elegaanz
Copy link
Member

Looking at your screens, it looks really nicer than before. My only concern would be about the centered text in inputs, since the text is moving in a quite strange way when typing like that. But I still should give it a live try.

Copy link
Member

@elegaanz elegaanz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I said, I'm not sure having text centered in inputs is a good idea.

And I don't know if the textarea to write an article should be that small. Maybe if it has a bigger default height?

Otherwise, it looks very good! Great job! 👌

(and as usual I'm open to discussion if there are good reasons behind these choices)

@elegaanz
Copy link
Member

I'm going to merge this and make a few changes after.

@elegaanz elegaanz merged commit 4aa184b into Plume-org:master Jul 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: Front-End Related to the front-end C: Enhancement New feature or request Design UI/UX related issues and PRs S: Ready for review This PR is ready to be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants