-
Notifications
You must be signed in to change notification settings - Fork 212
Events UI #3309 #3532
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
Events UI #3309 #3532
Conversation
|
@jvstme We need to add anything else? |
jvstme
left a comment
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.
We need to add anything else?
No, I think this covers the current requirements 👍
Although I see @peterschmidt85 in #3309 suggests to rework project and user events into tabs (probably still with the "Full view" button, like with other entities). But the current PR would already be useful in this week's release, so I think merging it as-is and reworking into tabs in a separate PR is also an option
Just quickly chatted with @olgenn. Agreed to start with tabs instead of adding too many links to the top menu (especially since their visual style differs from our common approach). We'll work with @olgenn on this together. |
peterschmidt85
left a comment
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 goal is to make the Events tabs look consistent across pages.
Added links to event list from project and user details pages