Zinkue commented on code in PR #58411:
URL: https://github.com/apache/airflow/pull/58411#discussion_r2597746426
##########
airflow-core/src/airflow/ui/src/pages/TaskInstance/Logs/TaskLogHeader.tsx:
##########
@@ -85,6 +85,7 @@ export const TaskLogHeader = ({
const sources = searchParams.getAll(SearchParamsKeys.SOURCE);
const logLevels = searchParams.getAll(SearchParamsKeys.LOG_LEVEL);
const hasLogLevels = logLevels.length > 0;
+ const system = createTheme();
Review Comment:
Because in this same file, line 92, we have this constant using `system`
``` javascript
const zIndex = isFullscreen
? Number(system.tokens.categoryMap.get("zIndex")?.get("modal")?.value ??
1400) + 1
: undefined;
```
Since my proposal uses a function to generate the custom theme I need to
call it somewhere. I decided to call it in this file.
Another possibility could by to create a `defaultSystem` in `src/theme` and
import it in this file
--
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]
