Skip to content

Commit

Permalink
fix: Change downloadAsImage to use Superset theme (#22011)
Browse files Browse the repository at this point in the history
  • Loading branch information
lyndsiWilliams committed Nov 3, 2022
1 parent 429f246 commit ba65f66
Showing 1 changed file with 2 additions and 8 deletions.
10 changes: 2 additions & 8 deletions superset-frontend/src/utils/downloadAsImage.ts
Original file line number Diff line number Diff line change
Expand Up @@ -19,15 +19,9 @@
import { SyntheticEvent } from 'react';
import domToImage from 'dom-to-image-more';
import kebabCase from 'lodash/kebabCase';
import { t } from '@superset-ui/core';
import { t, supersetTheme } from '@superset-ui/core';
import { addWarningToast } from 'src/components/MessageToasts/actions';

/**
* @remark
* same as https://github.com/apache/superset/blob/c53bc4ddf9808a8bb6916bbe3cb31935d33a2420/superset-frontend/src/assets/stylesheets/less/variables.less#L34
*/
const GRAY_BACKGROUND_COLOR = '#F5F5F5';

/**
* generate a consistent file stem from a description and date
*
Expand Down Expand Up @@ -77,7 +71,7 @@ export default function downloadAsImage(
return domToImage
.toJpeg(elementToPrint, {
quality: 0.95,
bgcolor: GRAY_BACKGROUND_COLOR,
bgcolor: supersetTheme.colors.grayscale.light4,
filter,
})
.then(dataUrl => {
Expand Down

0 comments on commit ba65f66

Please sign in to comment.