Skip to content

Conversation

lgecse
Copy link
Member

@lgecse lgecse commented Jun 24, 2025

Solves #120

Adds copyright information.
Supports dark and light AGNTCY themes, logos included.

image
image
image
image

lgecse added 4 commits June 24, 2025 18:51
Signed-off-by: Laszlo Gecse <lgecse@cisco.com>
Signed-off-by: Laszlo Gecse <lgecse@cisco.com>
…pyright partial

Signed-off-by: Laszlo Gecse <lgecse@cisco.com>
…stom.css

Signed-off-by: Laszlo Gecse <lgecse@cisco.com>
@muscariello muscariello self-requested a review June 24, 2025 17:49
@lgecse lgecse changed the title Agntcy theme feat(ui): added AGNTCY colors Jun 24, 2025
@lgecse lgecse marked this pull request as ready for review June 24, 2025 19:32
@lgecse lgecse requested a review from a team as a code owner June 24, 2025 19:32
@lgecse lgecse requested review from ramizpolic and keraron June 24, 2025 19:33
Copy link
Member

@muscariello muscariello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nits: The word AGNTCY appears two times in a row via the logo in capital letters and then as text. Can we remove the text?

@vaesposito
Copy link

@lgecse it looks good to me!
Screenshot 2025-06-25 at 10 30 28
would it be possible to

  • reduce spacing between agntcy logo and title in the header
  • get rid of "agntcy" title in the sidenav, before introduction
  • reduce vertical space between nav and header
    I'll schedule some time for a tour as you suggested, thanks!

Copy link
Member

@muscariello muscariello left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@muscariello muscariello merged commit 79d4642 into main Jun 26, 2025
5 checks passed
@muscariello muscariello deleted the agntcy_theme branch June 26, 2025 08:19
@lgecse lgecse removed this from Documentations Aug 25, 2025
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.

3 participants