Skip to content

fix settings syntax error in settings#45

Merged
yonaries merged 1 commit intothe-ora:mainfrom
plyght:settings-fix
Sep 8, 2025
Merged

fix settings syntax error in settings#45
yonaries merged 1 commit intothe-ora:mainfrom
plyght:settings-fix

Conversation

@plyght
Copy link
Copy Markdown
Contributor

@plyght plyght commented Sep 8, 2025

This pull request makes a minor UI improvement to the SpacesSettingsView in the settings module. The change refines the appearance of the "Space-Specific Defaults" section by updating the styling of the explanatory text.

  • UI enhancement: The explanatory text for overriding global defaults now uses .foregroundStyle(.secondary) for improved readability and visual distinction, replacing the previous multiline Text and removing the associated ForEach block for search engines. (ora/Modules/Settings/Sections/SpacesSettingsView.swift)

Copilot AI review requested due to automatic review settings September 8, 2025 12:45
Copy link
Copy Markdown
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

Fixes a syntax error in the SpacesSettingsView by correcting malformed VStack and Picker code structure that was preventing compilation.

  • Removed orphaned closing parenthesis and misplaced ForEach block
  • Added missing .foregroundStyle(.secondary) modifier to complete the Text styling
  • Restructured the Picker implementation to properly contain the search engine options

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@yonaries yonaries merged commit 0abf094 into the-ora:main Sep 8, 2025
2 checks passed
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