-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
feat(www): polish the contact view UI #1746
Conversation
Codecov ReportAll modified lines are covered by tests ✅ see 40 files with indirect coverage changes 📢 Thoughts on this report? Let us know!. |
cbec0e3
to
7ffd26b
Compare
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.
Do you mind posting a screenshot of the form pages in the description? Since the app is currently static for the most part let's not worry about responsiveness so much at the moment (probably would require a lot more changes anyway)
Okay, design nits:
|
Yes, but the buttons are disabled once you pick them. I am actually going to undo that, since you can already step back and they get re-enabled so it's kind of pointless right now.
The white background on an outlined select would be against spec. The point is that outlined fields have less emphasis to simplify the layout. I made the change, but it does look kind of odd to be honest. I agree re: copy, though we're trying to stick to the same copy as https://support.getoutline.org/s/contactsupport, which uses the generic "Outline issue".
Set the same gutter. I'll update the screenshots. |
ab50dd0
to
bce9d33
Compare
Yeah it looks weird - okay, what if we put the radio button and the dropdown in it's own card/white box? That way it's consistent with the other pages. Maybe that's what was bothering me. |
Done. |
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.
The space above "Do you have an open ticket [..] ?" is a bit big, but otherwise looks great.
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.
lol i meant to approve
This seems to break consistency with other text fields across our apps. Let's discuss. See the "Outline Design" email thread. |
Screenshots: