[superset] branch master updated: chore: no direct use of supersetTheme (or bad LESS vars) in SqlEditor (#16999)

[kgabryje]

This is an automated email from the ASF dual-hosted git repository.

kgabryje pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new ac50961  chore: no direct use of supersetTheme (or bad LESS vars) in 
SqlEditor (#16999)
ac50961 is described below

commit ac509611b6ca950bf373c065012414020dd224b7
Author: Evan Rusackas <e...@preset.io>
AuthorDate: Thu Oct 7 01:46:37 2021 -0600

    chore: no direct use of supersetTheme (or bad LESS vars) in SqlEditor 
(#16999)
    
    * chore: no direct use of supersetTheme (or bad LESS vars) in SqlEditor
    
    * putting the background back in, but using the theme.
    
    * should be no need for this import anymore.
---
 superset-frontend/src/SqlLab/components/SqlEditor/index.jsx | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/superset-frontend/src/SqlLab/components/SqlEditor/index.jsx 
b/superset-frontend/src/SqlLab/components/SqlEditor/index.jsx
index bd96acb..bb2115a 100644
--- a/superset-frontend/src/SqlLab/components/SqlEditor/index.jsx
+++ b/superset-frontend/src/SqlLab/components/SqlEditor/index.jsx
@@ -24,7 +24,7 @@ import { connect } from 'react-redux';
 import { bindActionCreators } from 'redux';
 import PropTypes from 'prop-types';
 import Split from 'react-split';
-import { t, styled, supersetTheme, withTheme } from '@superset-ui/core';
+import { t, styled, withTheme } from '@superset-ui/core';
 import debounce from 'lodash/debounce';
 import throttle from 'lodash/throttle';
 import StyledModal from 'src/components/Modal';
@@ -102,10 +102,10 @@ const LimitSelectStyled = styled.span`
 
 const StyledToolbar = styled.div`
   padding: ${({ theme }) => theme.gridUnit * 2}px;
-  background-color: @lightest;
+  background: ${({ theme }) => theme.colors.grayscale.light5};
   display: flex;
   justify-content: space-between;
-  border: 1px solid ${supersetTheme.colors.grayscale.light2};
+  border: 1px solid ${({ theme }) => theme.colors.grayscale.light2};
   border-top: 0;
 
   form {