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.

Sign up for GitHub

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

Jump to bottom

Peer-to-Peer Code Review Session #7

Open

Andrik27001 opened this issue Oct 27, 2020 · 5 comments

Andrik27001 commented Oct 27, 2020

Leave your questions below!

Honda99 commented Oct 27, 2020

The buttons shouldn't be inside of the label you have to separate them.

apricot12 commented Oct 27, 2020

I think the comment box border is too thick
Try to reduce the box width to fit with the article
Use lighter fonts if you can.

Otherwise, good work!

Collaborator

wandji20 commented Oct 27, 2020

You could reduce the form width to match the content size or more
give the form elements some margin so they can be more visible.

Honda99 commented Oct 27, 2020

and don't use "br" element to create new line.

Owner

Byabasaija commented Oct 27, 2020

Thanks for the feedback, i will work on the suggested changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment