Skip to content

Conversation

@0xWDG
Copy link
Owner

@0xWDG 0xWDG commented Dec 20, 2025

No description provided.

@0xWDG 0xWDG self-assigned this Dec 20, 2025
Copilot AI review requested due to automatic review settings December 20, 2025 15:43
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR standardizes the indentation of JSON code examples across documentation files for various DynamicUI view components. The changes improve code consistency and readability in the documentation.

Key Changes

  • Corrected indentation of JSON objects within code examples from inconsistent spacing to a standardized 12-space indent for nested children
  • Aligned closing brackets to match their corresponding opening brackets
  • Applied consistent formatting across 13 documentation files

Reviewed changes

Copilot reviewed 9 out of 13 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
ZStack.md Fixed indentation of children array content in JSON example
VStack.md Fixed indentation of children array content in JSON example
VSplitView.md Fixed indentation of children array content in JSON example
Section.md Fixed indentation and aligned closing brackets in JSON example
ScrollView.md Fixed indentation and aligned closing brackets in JSON example
Picker.md Fixed indentation for multiple picker items and aligned closing brackets
NavigationView.md Fixed indentation and aligned closing brackets in JSON example
List.md Fixed indentation and aligned closing brackets in JSON example
HStack.md Fixed indentation and aligned closing brackets in JSON example
HSplitView.md Fixed indentation and aligned closing brackets in JSON example
GroupBox.md Fixed indentation and aligned closing brackets in JSON example
Group.md Fixed indentation and aligned closing brackets in JSON example
DisclosureGroup.md Fixed indentation of children array content in JSON example

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 20, 2025 15:46
0xWDG and others added 2 commits December 20, 2025 16:46
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 13 changed files in this pull request and generated 13 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 15 to 16
}
}
Copy link

Copilot AI Dec 20, 2025

Choose a reason for hiding this comment

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

The JSON structure has an extra closing brace. After line 14 which closes the "children" array, there should only be ONE closing brace to close the object that started on line 7. Currently there are two closing braces (lines 15 and 16), which creates invalid JSON syntax. Line 15 should be removed.

Copilot uses AI. Check for mistakes.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 20, 2025 15:58
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
0xWDG and others added 3 commits December 20, 2025 16:59
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 13 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 20, 2025 16:00
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
0xWDG and others added 3 commits December 20, 2025 17:01
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 11 out of 13 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings December 20, 2025 16:03
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 12 out of 13 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@0xWDG 0xWDG merged commit e123942 into main Dec 20, 2025
5 checks passed
@0xWDG 0xWDG deleted the fix-typos-and-identation branch December 20, 2025 16:08
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.

2 participants