Skip to content

Commit e3b5092

Browse files
authored
Merge pull request #454 from rsnyman/fix-result-summary-labels
Fix up font colour issues with the labels on various charts/widgets
2 parents 46210b3 + 9ef0e75 commit e3b5092

File tree

2 files changed

+14
-8
lines changed

2 files changed

+14
-8
lines changed

frontend/src/index.css

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -251,3 +251,9 @@ div[data-id=widget-param-dropdown] > div {
251251
:where(.pf-theme-dark) .pf-c-clipboard-copy .pf-c-form-control[readonly] {
252252
border-bottom-color: var(--pf-global--BorderColor--400) !important;
253253
}
254+
255+
:where(.pf-theme-dark) {
256+
--pf-chart-global--label--Fill: var(--pf-global--Color--100);
257+
--pf-chart-axis--tick-label--Fill: var(--pf-global--Color--200);
258+
--pf-chart-donut--label--title--Fill: var(--pf-global--Color--100);
259+
}

frontend/src/widgets/resultsummary.js

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -77,14 +77,14 @@ export class ResultSummaryWidget extends React.Component {
7777

7878
render() {
7979
const themeColors = [
80-
'var(--pf-global--success-color--100)',
81-
'var(--pf-global--danger-color--100)',
82-
'var(--pf-global--info-color--100)',
83-
'var(--pf-global--warning-color--100)',
84-
'var(--pf-global--palette--purple-400)',
85-
'var(--pf-global--palette--purple-700)',
86-
'var(--pf-global--primary-color--100)'
87-
];
80+
'var(--pf-global--success-color--100)',
81+
'var(--pf-global--danger-color--100)',
82+
'var(--pf-global--info-color--100)',
83+
'var(--pf-global--warning-color--100)',
84+
'var(--pf-global--palette--purple-400)',
85+
'var(--pf-global--palette--purple-700)',
86+
'var(--pf-global--primary-color--100)'
87+
];
8888
return (
8989
<Card>
9090
<WidgetHeader title={this.title} getDataFunc={this.getResultSummary} onEditClick={this.props.onEditClick} onDeleteClick={this.props.onDeleteClick}/>

0 commit comments

Comments
 (0)