-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Comments
hi @svrnm can i work on this issue ? |
@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. |
Thank you @svrnm |
@svrnm can I work on this? |
@svrnm Can I help with this? |
@olamideTiana yes! Please take a look at the issue description and provide a PR |
@svrnm I understand what I need to work on now... I'm working on it. Thank you |
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
The text was updated successfully, but these errors were encountered: