Skip to content

Design form UI#19

Merged
adityabisoi merged 3 commits intoadityabisoi:mainfrom
luckykumarirai:Design-Form-UI
Dec 19, 2020
Merged

Design form UI#19
adityabisoi merged 3 commits intoadityabisoi:mainfrom
luckykumarirai:Design-Form-UI

Conversation

@luckykumarirai
Copy link
Copy Markdown
Contributor

Screenshot from 2020-12-15 10-22-21
I have designed this form using Material UI.

@adityabisoi
Copy link
Copy Markdown
Owner

The design can be improved. Nice work with the implementation, though. Please apply your creativity a little more.

@luckykumarirai
Copy link
Copy Markdown
Contributor Author

Thanks , Sure I will do that.

@adityabisoi
Copy link
Copy Markdown
Owner

@luckykumarirai Are you on this?

@luckykumarirai
Copy link
Copy Markdown
Contributor Author

Yes,
I am working but can u explain what kind of design u want for better UI.i have added Navbar on this page.plz explain
Screenshot from 2020-12-18 18-06-55

@adityabisoi
Copy link
Copy Markdown
Owner

The postfix operation should be invisible, and should appear only when conversion results are shown. Also, what does the leftmost button do?

@luckykumarirai
Copy link
Copy Markdown
Contributor Author

okay, I will add all those features which u mentation.

@luckykumarirai
Copy link
Copy Markdown
Contributor Author

luckykumarirai commented Dec 19, 2020

Now postfix operation only visible after a click on submit and the design also improved.
##Before submit-
Screenshot from 2020-12-19 16-48-20
##After submit-
Screenshot from 2020-12-19 17-11-39

@adityabisoi
Copy link
Copy Markdown
Owner

Nice work with the UI. However, I do not see the conversion pages for other routes. Can you do that, if not, I would have to create new issue.

@luckykumarirai
Copy link
Copy Markdown
Contributor Author

Please accept this PR and you should create new issues sure I can work on that issue also.

@adityabisoi adityabisoi merged commit 8b4f4a8 into adityabisoi:main Dec 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants