Skip to content
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

Added navigations to project-details #1132 #1422

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

VarshaUN
Copy link
Contributor

Regarding the issue #1132 I have added Navigations to project details.

screenshots attached below
This is when I hovered on Packages similarly on other elements as well.
Screenshot (91)
Signed-off-by: Varsha U N [email protected]

@VarshaUN VarshaUN force-pushed the make-navigations-1132 branch from 707b958 to b2ecacf Compare October 30, 2024 12:54
@VarshaUN
Copy link
Contributor Author

@tdruez please suggest me on how can I improve this.
Thank you

@tdruez
Copy link
Contributor

tdruez commented Nov 12, 2024

@VarshaUN please suggest me on how can I improve this.

Well, you handle the "link" part of #1132 but I don't think you entirely addressed this issue:

visually these names are in small caps and it would be better to make these in the same style and font as other UI elements from the page like the Inputs, Codebase and related block in white-on-blue below. These are the most important things and they are relatively rather small compared to other parts.

@VarshaUN VarshaUN force-pushed the make-navigations-1132 branch from f54754c to 21b548f Compare November 12, 2024 11:36
@VarshaUN
Copy link
Contributor Author

VarshaUN commented Nov 12, 2024

@tdruez I have added the same font as inputs now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants