Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Feb 1, 2026

Changes proposed in this pull request

  • Consolidated Button closing tag in SettingsSettingsView.xaml from multi-line to single-line format
  • Removed trailing whitespace before closing angle bracket

Related issue(s)

Related to #3302 (addressing review feedback from #3302)

Copilot generated summary

Provide a Copilot generated summary of the changes in this pull request.

Copilot summary

Fixed XAML formatting inconsistency where a Button element's closing tag was split across two lines with trailing whitespace. The closing /> is now on the same line as the other attributes, matching the codebase's established formatting pattern.

Before:

<Button Content="{x:Static localization:Strings.Reset}" Command="{Binding ResetSettingsCommand}"
        Style="{StaticResource DefaultButton}" HorizontalAlignment="Left" 
        />

After:

<Button Content="{x:Static localization:Strings.Reset}" Command="{Binding ResetSettingsCommand}"
        Style="{StaticResource DefaultButton}" HorizontalAlignment="Left" />

To-Do

Contributing

By submitting this pull request, I confirm the following:


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on Automatic Profile and Settings backup PR Fix XAML formatting: consolidate Button closing tag to single line Feb 1, 2026
Copilot AI requested a review from BornToBeRoot February 1, 2026 01:12
@BornToBeRoot BornToBeRoot marked this pull request as ready for review February 1, 2026 01:15
@BornToBeRoot BornToBeRoot merged commit 1db52f4 into feature/configure_backup_retention Feb 1, 2026
2 of 4 checks passed
@BornToBeRoot BornToBeRoot deleted the copilot/sub-pr-3302-again branch February 1, 2026 01:16
mergify bot pushed a commit that referenced this pull request Feb 1, 2026
* Feature: Configure backup retention

* Feature: Daily Backup

* Feature: Add more backups before important operations

* Feature: Create daily backup

* Feature: Refactor ProfileManager to add properties to a ProfileFile

* Docs: #3302

* Update Source/NETworkManager.Settings/SettingsInfo.cs

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Fix: Wrong behaviour if null

* Fix XAML formatting: consolidate Button closing tag to single line (#3308)

* Initial plan

* Fix: Consolidate Button closing tag to single line

Co-authored-by: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>

* Remove trailing whitespace from SettingsSettingsView.xaml (#3307)

* Initial plan

* Remove trailing whitespace from SettingsSettingsView.xaml line 40

Co-authored-by: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: BornToBeRoot <16019165+BornToBeRoot@users.noreply.github.com>

---------

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
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