-
Notifications
You must be signed in to change notification settings - Fork 498
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
Tweak LegalDocsView. #3537
Tweak LegalDocsView. #3537
Conversation
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.
You do not prefer big buttoned forms? |
A loved the border, now it looks like a big text bucket. The new position of the button is OK but I would reduce the size of it and put something green there. A checkmark or a green border. |
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.
Might be a bit "too much" green, but still, I like it - ACK
I like it. Can you make the button fixed size? |
@molnard Done :) Please re-check |
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.
tACK
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.
Tested and it looks better, maybe the color is too green.
Trying to get some work done while recovering :)
Should fix #3478 though I'm not considering the green button design change until we have some further discussions for changing the overall look of our buttons app-wide.
cc: @UkolovaOlga @molnard