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

Update colors for Demo Architecture to match website theme #5360

Open
svrnm opened this issue Oct 9, 2024 · 8 comments · May be fixed by #5467
Open

Update colors for Demo Architecture to match website theme #5360

svrnm opened this issue Oct 9, 2024 · 8 comments · May be fixed by #5467
Labels
help wanted Extra attention is needed ux

Comments

@svrnm
Copy link
Member

svrnm commented Oct 9, 2024

The mermaid based architecture diagram for the demo uses colors that do not match the colors of the website theme:

https://opentelemetry.io/docs/demo/architecture/

It would be great if someone would update the colors in the underlying mermaid definition to be in line with other graphics like https://opentelemetry.io/img/otel-diagram.svg or https://opentelemetry.io/img/waterfall-trace.svg

Here is where the colors are defined: https://github.com/open-telemetry/opentelemetry.io/blob/main/content/en/docs/demo/architecture.md?plain=1#L62

Special Note for Outreachy Contributors: if you want to work on this issue, please make sure that you have completed the small steps and at least one assigned issue as outlined here: #5290

@Tharanishwaran
Copy link
Contributor

hi @svrnm can i work on this issue ?

@svrnm svrnm added outreachy Issues for Outreachy Participants and removed good first issue Good for newcomers labels Oct 9, 2024
@svrnm
Copy link
Member Author

svrnm commented Oct 9, 2024

@Tharanishwaran we are currently running the participation phase for outreachy, and we need to reserve some good first issues for the participants. If you participate in outreachy please take a look in #5290, otherwise I can currently not assign this issue to you. When the participation phase ends, and this issue is still available, I am happy to assign it.

@Tharanishwaran
Copy link
Contributor

Thank you @svrnm

@ladyami
Copy link

ladyami commented Oct 14, 2024

@svrnm can I work on this?

@svrnm
Copy link
Member Author

svrnm commented Oct 14, 2024

@svrnm can I work on this?

Please complete #5399 first

@olamideTiana
Copy link
Contributor

@svrnm Can I help with this?

@svrnm
Copy link
Member Author

svrnm commented Oct 21, 2024

@olamideTiana yes! Please take a look at the issue description and provide a PR

@olamideTiana
Copy link
Contributor

olamideTiana commented Oct 21, 2024

@svrnm I understand what I need to work on now... I'm working on it. Thank you

@svrnm svrnm added help wanted Extra attention is needed and removed outreachy Issues for Outreachy Participants labels Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed ux
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

4 participants