You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2023/03/28 03:05:19 UTC

[kylin] 11/12: KYLIN-5488 refine query chart issue

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

xxyu pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git

commit 479f4e45fbee4e8534d80205153a52f3b5d3506c
Author: Qian Xia <la...@gmail.com>
AuthorDate: Fri Mar 24 18:40:39 2023 +0800

    KYLIN-5488 refine query chart issue
---
 kystudio/src/components/query/insight.vue      | 23 -----------------------
 kystudio/src/components/query/query_result.vue | 13 +------------
 2 files changed, 1 insertion(+), 35 deletions(-)

diff --git a/kystudio/src/components/query/insight.vue b/kystudio/src/components/query/insight.vue
index 348b28c84a..11ad7de46b 100644
--- a/kystudio/src/components/query/insight.vue
+++ b/kystudio/src/components/query/insight.vue
@@ -471,14 +471,6 @@ export default class NewQuery extends Vue {
         line-height: 21px;
       }
     }
-    // .nodata {
-    //   text-align: center;
-    //   color: @text-disabled-color;
-    //   position: absolute;
-    //   top: 50%;
-    //   left: 50%;
-    //   transform: translate(-50%, -50%);
-    // }
     .form_block {
       .query_check {
         display: flex;
@@ -603,13 +595,6 @@ export default class NewQuery extends Vue {
         margin: 20px;
       }
       .el-tabs{
-        // margin-top: 3px;
-        // .el-tabs__nav-wrap {
-        //   width: calc(~'100% - 250px');
-        //   &::after {
-        //     background-color: transparent;
-        //   }
-        // }
         .el-tabs__content{
           padding: 0px;
           .el-tab-pane{
@@ -620,14 +605,6 @@ export default class NewQuery extends Vue {
           .el-tab-pane{
             padding-top: 8px;
           }
-          .el-tabs__header {
-            width: 136px;
-          }
-        }
-        &.en-model {
-          .el-tabs__header {
-            width: 174px !important;
-          }
         }
       }
     }
diff --git a/kystudio/src/components/query/query_result.vue b/kystudio/src/components/query/query_result.vue
index 9b8438a003..021d21f8e5 100644
--- a/kystudio/src/components/query/query_result.vue
+++ b/kystudio/src/components/query/query_result.vue
@@ -126,7 +126,7 @@
             </form>
           </el-tab-pane>
           <el-tab-pane :label="$t('visualizationBtn')" name="visualization">
-            <div class="chart-headers" v-if="charts.dimension && charts.measure">
+            <div class="chart-headers" v-if="charts.dimension && charts.measure && activeResultType === 'visualization'">
               <el-row class="ksd-mt-10" :gutter="5">
                 <el-col :span="4" class="title">{{$t('chartType')}}</el-col>
                 <el-col :span="10" class="title">{{$t('chartDimension')}}</el-col>
@@ -836,15 +836,6 @@ export default class queryResult extends Vue {
         width: 245px;
       }
     }
-    // .result-layout-btns {
-    //   margin-top: 16px;
-    //   .el-button.active {
-    //     color: #5c5c5c;
-    //     background: #f4f4f4;
-    //     border: 1px solid #cccccc;
-    //     box-shadow: inset 1px 1px 2px 0 #ddd;
-    //   }
-    // }
     .chart-headers {
       .title {
         font-weight: bold;
@@ -879,8 +870,6 @@ export default class queryResult extends Vue {
     }
   }
   .table-cell-text{
-    // word-wrap: break-word;
-    // word-break: break-all;
     white-space: pre;
     color: @text-normal-color;
     font-family: Lato,"Noto Sans S Chinese",sans-serif;