You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by hu...@apache.org on 2021/01/05 20:40:38 UTC

[superset] branch hugh/feat-sql-toolbar-r2 updated: fix estimate cost specs

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

hugh pushed a commit to branch hugh/feat-sql-toolbar-r2
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/hugh/feat-sql-toolbar-r2 by this push:
     new 4860d23  fix estimate cost specs
4860d23 is described below

commit 4860d2374a4f5cec068a72048625f4450c1362c4
Author: hughhhh <hu...@gmail.com>
AuthorDate: Tue Jan 5 15:39:52 2021 -0500

    fix estimate cost specs
---
 .../src/SqlLab/components/EstimateQueryCostButton.jsx              | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/superset-frontend/src/SqlLab/components/EstimateQueryCostButton.jsx b/superset-frontend/src/SqlLab/components/EstimateQueryCostButton.jsx
index 04575d5..7fa01ca 100644
--- a/superset-frontend/src/SqlLab/components/EstimateQueryCostButton.jsx
+++ b/superset-frontend/src/SqlLab/components/EstimateQueryCostButton.jsx
@@ -83,22 +83,21 @@ const EstimateQueryCostButton = props => {
   const { disabled, selectedText, tooltip } = props;
   const btnText = selectedText
     ? t('Estimate Selected Query Cost')
-    : t('Estimate Query Cost');
+    : t('Estimate Cost');
   return (
     <span className="EstimateQueryCostButton">
       <ModalTrigger
-        modalTitle={t('Query Cost Estimate')}
+        modalTitle={t('Cost Estimate')}
         modalBody={renderModalBody()}
         triggerNode={
           <Button
-            buttonStyle="warning"
             buttonSize="small"
             onClick={onClick}
             key="query-estimate-btn"
             tooltip={tooltip}
             disabled={disabled}
           >
-            <i className="fa fa-clock-o" /> {btnText}
+            {btnText}
           </Button>
         }
       />