Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: #167 support dt highlight style #168

Draft
wants to merge 3 commits into
base: next
Choose a base branch
from

Conversation

liuxy0551
Copy link
Collaborator

变更

  1. 亮色主题重新设计 #167
  2. 根据设计稿修改各个 token 类型的色号

预览地址

https://liuxy0551.github.io/monaco-sql-languages/

测试 SQL

-- SELECT、name、18、()、[]、{}、VARCHAR、COUNT、->、=、.、;、'kafka' 等的颜色,已经用 、 切分开了
SELECT t1.id, name FROM t1 WHERE age = 18 GROUP BY
    user_id,
    TUMBLE(activity[0].event_time, INTERVAL '1' HOUR)
HAVING
    COUNT(WITH activity[{i -> i % 2 == 0}.action]) > 5;

CREATE TABLE source_table (
    user_id VARCHAR,
    activity ARRAY<STRUCT<event_time: TIMESTAMP(3), action: STRING>>
) WITH (
    'connector' = 'kafka'
);

-- 注意下方的 CASE WHEN THEN 等的颜色
SELECT
    user_id,
    activity_type,
    CASE
        WHEN activity_type = 'login' THEN 'User Login'
        WHEN activity_type = 'purchase' THEN 'Purchase Made'
        ELSE 'Other Activity'
    END AS activity_description
FROM
    user_activities;


-- 注意两种自定义参数的颜色
SELECT ${aaa};
SELECT @@{bbb};

@liuxy0551
Copy link
Collaborator Author

liuxy0551 commented Jan 7, 2025

基于 v0.12.3-beta.12 切出的分支,请仅查看本次变更的 commit: ab7cf31

@liuxy0551 liuxy0551 requested a review from Cythia828 January 7, 2025 09:03
@liuxy0551
Copy link
Collaborator Author

@KinyooZ 请点击预览地址进行 UICheck

@LuckyFBB
Copy link
Collaborator

LuckyFBB commented Jan 8, 2025

基于 v0.12.3-beta.12 切出的分支,请仅查看本次变更的 commit: bff0142

后面这个是直接合并到 main 分支吗?还是基于你这个分支打对应的 beta 包啊

@liuxy0551
Copy link
Collaborator Author

image

@liuxy0551 liuxy0551 marked this pull request as draft January 9, 2025 07:54
@liuxy0551 liuxy0551 changed the base branch from main to next January 9, 2025 07:54
@liuxy0551
Copy link
Collaborator Author

基于 v0.12.3-beta.12 切出的分支,请仅查看本次变更的 commit: bff0142

后面这个是直接合并到 main 分支吗?还是基于你这个分支打对应的 beta 包啊

等其他的 PR 合入了再合入 next 分支吧,我改目标分支了。会基于分支打 beta 包在子产品中验证

@KinyooZ
Copy link

KinyooZ commented Jan 9, 2025

UI Check 完成

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants