davsclaus commented on code in PR #24275:
URL: https://github.com/apache/camel/pull/24275#discussion_r3485486749
##########
components/camel-diagram/src/main/docs/diagram.adoc:
##########
@@ -352,3 +403,22 @@ camel-route-diagram {
--crd-color-default: #6366f1; /* all other EIP nodes */
}
----
+
+==== Topology Diagram Theming
+
+[source,css]
+----
+camel-topology-diagram {
+ --ctd-bg: #ffffff; /* canvas background */
+ --ctd-fg: #1e293b; /* text colour */
+ --ctd-edge: #94a3b8; /* edge/arrow colour */
+ --ctd-stat: #64748b; /* metric overlay text */
+ --ctd-font: system-ui; /* font family */
+ --ctd-font-size: 12px; /* base font size */
+ --ctd-color-route: #6366f1; /* route nodes */
+ --ctd-color-trigger: #f59e0b; /* trigger/timer routes */
+ --ctd-color-external-in: #6b7280; /* external input endpoints */
+ --ctd-color-external-out: #6b7280; /* external output endpoints */
+ --ctd-color-external-shared: #8b5cf6; /* shared external endpoints */
+}
+----
Review Comment:
The new `--ctd-node-bg` CSS custom property (introduced in the JS to fix the
dark-theme contrast bug) is missing from this theming reference. The Route
Diagram Theming section above documents the analogous `--crd-node-bg`. Consider
adding it here for consistency:
```suggestion
camel-topology-diagram {
--ctd-bg: #ffffff; /* canvas background */
--ctd-node-bg: #ffffff; /* node box fill, auto-inverts in
dark mode */
--ctd-fg: #1e293b; /* text colour */
--ctd-edge: #94a3b8; /* edge/arrow colour */
--ctd-stat: #64748b; /* metric overlay text */
--ctd-font: system-ui; /* font family */
--ctd-font-size: 12px; /* base font size */
--ctd-color-route: #6366f1; /* route nodes */
--ctd-color-trigger: #f59e0b; /* trigger/timer routes */
--ctd-color-external-in: #6b7280; /* external input endpoints */
--ctd-color-external-out: #6b7280; /* external output endpoints */
--ctd-color-external-shared: #8b5cf6; /* shared external endpoints */
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
