fix: make plan to svg more comprehensive

[astandrik]

Closes #1656

Stand

CI Results

Test Status: ✅ PASSED

📊 Full Report

Total	Passed	Failed	Flaky	Skipped
176	176	0	0	0

Bundle Size: ✅

Current: 66.11 MB | Main: 66.11 MB
Diff: +0.00 MB (0.00%)

✅ Bundle size unchanged.

ℹ️ CI Information

• Test recordings for failed tests are available in the full report.
• Bundle size is measured for the entire 'dist' directory.
• 📊 indicates links to detailed reports.
• 🔺 indicates increase, 🔽 decrease, and ✅ no change in bundle size.

[Copilot]

Copilot reviewed 3 out of 6 changed files in this pull request and generated no suggestions.

Files not reviewed (3)

• src/containers/Tenant/Query/QuerySettingsDialog/i18n/en.json: Language not supported
• src/containers/Tenant/Query/QuerySettingsDialog/i18n/ru.json: Language not supported
• src/containers/UserSettings/i18n/en.json: Language not supported

[Copilot]

Copilot reviewed 3 out of 6 changed files in this pull request and generated no suggestions.

Files not reviewed (3)

• src/containers/Tenant/Query/QuerySettingsDialog/i18n/en.json: Language not supported
• src/containers/Tenant/Query/QuerySettingsDialog/i18n/ru.json: Language not supported
• src/containers/UserSettings/i18n/en.json: Language not supported

Comments skipped due to low confidence (1)

src/utils/hooks/useQueryExecutionSettings.ts:13

• [nitpick] The variable name useShowPlanToSvg is ambiguous. It should be renamed to isPlanToSvgEnabled for better clarity.

const [useShowPlanToSvg] = useSetting<boolean>(USE_SHOW_PLAN_SVG_KEY);

[Copilot]

Copilot reviewed 4 out of 7 changed files in this pull request and generated no suggestions.

Files not reviewed (3)

• src/containers/Tenant/Query/QuerySettingsDialog/i18n/en.json: Language not supported
• src/containers/Tenant/Query/QuerySettingsDialog/i18n/ru.json: Language not supported
• src/containers/UserSettings/i18n/en.json: Language not supported

Comments skipped due to low confidence (1)

src/containers/Tenant/Query/QuerySettingsDialog/QuerySettingsSelect.tsx:38

• The new disabled prop should be covered by tests to ensure it works as expected.

disabled={props.disabled}