Skip to content

Conversation

@SeraphimaZykova
Copy link
Contributor

Summary of the Pull Request

What is this about:

  • Keep custom layouts in a separate file.
  • Create custom-layouts.json from zones-settings on FZ start
  • Save custom layouts in the custom-layout.json in Editor.
  • Unit tests

What is included in the PR:

How does someone test / validate:

  • Run PT, verify custom-layouts.json created
  • Run tests
  • Open editor, create or edit the custom layout, verify it was saved properly.

Quality Checklist

Contributor License Agreement (CLA)

A CLA must be signed. If not, go over here and sign the CLA.

<ClInclude Include="ZonesOverlay.h" />
</ItemGroup>
<ItemGroup>
<ClCompile Include="FancyZonesData\CustomLayouts.cpp">
Copy link
Contributor

Choose a reason for hiding this comment

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

this looks a bit weird. Does it have to be this way?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Otherwise, we'll have a pipeline build fail. See #15624

@crutkas
Copy link
Member

crutkas commented Jan 19, 2022

what is the background for the issue + fix?

@SeraphimaZykova
Copy link
Contributor Author

what is the background for the issue + fix?

Users lose data on driver updates or windows crashes. It happens because FZ tries to rewrite zone-settings right before crash/restart. So I split zone-settings into different files and FZ wouldn't have even the possibility to rewrite them. From now, they can be changed from the Editor only.

@SeraphimaZykova SeraphimaZykova merged commit 714ca34 into microsoft:main Jan 20, 2022
@SeraphimaZykova SeraphimaZykova deleted the 14782-fz-custom-zone-sets branch January 20, 2022 11:02
gregjacobson added a commit to gregjacobson/windows-uwp that referenced this pull request May 21, 2022
The `zones-settings.json` file was removed and split into different files in microsoft/PowerToys#15642.  However, the current documentation still references the removed `zones-settings.json` file.   This update fixes the file location and also mentions the other newly available configuration files.
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