-
Notifications
You must be signed in to change notification settings - Fork 34
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
Members: create mockup of grid-based page #279
base: main
Are you sure you want to change the base?
Conversation
Awesome start! 🥳 A few requests:
Feel free to merge this and follow up in new PRs.
|
@chadwhitacre What about now? I think having the roles on there is a nice touch. |
Closes #281. |
Good progress! I would like to bring this to Sentry's design team for refinement. Also, @selviano and I chatted about this ... wdyt of having both views, with the new card view as the default? Similar to how Google Drive and others offer both list and card view. Card would be the new default, but list would be accessible for digging. |
Cool! I can mock that up, and we can see how we like it. |
e176b71
to
0a435bb
Compare
Sentry's design team isn't going to get to this for a few weeks. I say let's go ahead and land and iterate later. Is grid the default here, @vladh? |
@chadwhitacre Yep, grid is default! Let me do another design pass on this tomorrow, I could make some things better, and also add icons. |
Can we try out these tweaks?
Wdyt? |
Yep, sounds good! Will send some screenshots over early tomorrow. :) |
a766691
to
d96cc91
Compare
I think this is nicer:
Note: this includes PR #299 for conflict avoidance purposes. We should be good to merge if you think this looks good enough visually. |
d96cc91
to
e658f36
Compare
@chadwhitacre Were you thinking of something like this?