-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Vis: Default editor] EUIficate raw json control #32888
[Vis: Default editor] EUIficate raw json control #32888
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@@ -161,7 +161,6 @@ | |||
"common.ui.aggTypes.ipRanges.toLabel": "到", | |||
"common.ui.aggTypes.ipRanges.useCidrMasksButtonLabel": "使用 CIDR 掩码", | |||
"common.ui.aggTypes.ipRanges.useFromToButtonLabel": "使用“从”/“到”", | |||
"common.ui.aggTypes.jsonInputLabel": "JSON 输入", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I removed this label because sentence case was applied: JSON Input
-> JSON input
Since |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for making those changes
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
src/legacy/ui/public/vis/editors/default/__tests__/utils.test.tsx
Outdated
Show resolved
Hide resolved
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
💚 Build Succeeded |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on Chrome Linux, seems to work. Code LGTM
* EUIficate raw json control * Remove unused validate-json directive * Update tests * Update styles * Move validation logic down to control * Fix type
Summary
EUIfication of
json
control for aggregation parameter in Default Editor, Data tab.Tooltip:
Part of #30922
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.[ ] This was checked for cross-browser compatibility, including a check against IE11[ ] Documentation was added for features that require explanation or tutorialsFor maintainers
[ ] This was checked for breaking API changes and was labeled appropriately[ ] This includes a feature addition or change that requires a release note and was labeled appropriately