Skip to content

Conversation

@Kyzyl-ool
Copy link
Contributor

No description provided.

@gravity-ui-bot
Copy link
Contributor

Preview is ready.

yuberdysheva
yuberdysheva previously approved these changes Jul 24, 2023
NikitaCG
NikitaCG previously approved these changes Jul 24, 2023

export const YamlEditor = ({content}: YamlEditorProps) => (
<MonacoEditor
value={yaml.dump(content)}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

useMemo

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good idea, done

<MonacoEditor
value={yaml.dump(content)}
language="yaml"
options={{
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

probably for options too

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also wrapped with useMemo

@Kyzyl-ool Kyzyl-ool dismissed stale reviews from NikitaCG and yuberdysheva via b6365f5 July 25, 2023 11:07
@Kyzyl-ool
Copy link
Contributor Author

Also added copy button

@Kyzyl-ool Kyzyl-ool requested a review from gorgeousvlad July 25, 2023 12:47
@@ -0,0 +1,42 @@
import React, {useState} from 'react';
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we use ClipboardButton from uikit instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we could. Should look for it first

gorgeousvlad
gorgeousvlad previously approved these changes Jul 26, 2023
@Kyzyl-ool Kyzyl-ool merged commit 7a8e187 into main Jul 31, 2023
@Kyzyl-ool Kyzyl-ool deleted the feat/yaml-tab branch July 31, 2023 13:23
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.

6 participants