Apply suggestions from code review

Co-authored-by: sofietoft <[email protected]>

Author: AndyButland

13/umbraco-forms/developer/configuration/README.md

@@ -526,7 +526,7 @@ A custom date format can be provided in [momentjs format](https://momentjscom.re
 
 #### DatePickerFormatForValidation
 
-If a custom date format is provided it will be used on the client-side. A matching string in [C# date format](https://learn.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings) should be provided, in order that server-side validation will match the expected format of the entry.
+If a custom date format is provided it will be used on the client-side. A matching string in [C# date format](https://learn.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings) should be provided, so that server-side validation will match the expected format of the entry.
 
 ### reCAPTCHA v2 field type configuration
 

13/umbraco-forms/developer/extending/adding-a-fieldtype.md

@@ -114,7 +114,7 @@ To support this, a read-only view of the field is necessary.
 
 For most fields, nothing is required here, as the default read-only display defined in the built-in `ReadOnly.cshtml` file suffices.
 
-However, if you want to provide a custom read-only display for your field, you can do so by creating a second partial view. This should be named with a `.ReadOnly` suffix. For this example, we would create `FieldType.Slider.ReadOnly.cshtml`.
+However, if you want to provide a custom read-only display for your field, you can do so by creating a second partial view. This should be named with a `.ReadOnly` suffix. For this example, you would create `FieldType.Slider.ReadOnly.cshtml`.
 
 ## Umbraco backoffice view
 

13/umbraco-forms/developer/themes.md

@@ -45,11 +45,12 @@ If adding or amending client-side scripts, you need to copy the `Script.cshtml`
 
 Umbraco Forms provides it's built-in themes as part of a Razor Class Library for ease of distribution. This can be useful for custom themes, particularly those used in multiple solutions or released as an Umbraco package.
 
-From Forms 13.3 it's possible to do this for custom themes.
+From Forms 13.3 it is possible to do this for custom themes.
 
-Firstly you'll create a new Razor Class Library project to hold the theme.  You then create the necessary partial views for your theme as usual within `Views\Partials\Forms\Themes\<my-custom-theme>`.
-
-You then need to provide the names of the files in your theme via an implementation of `ITheme`. For example, if only overriding a single file, your class would look like this:
+1. Create a new Razor Class Library project to hold the theme.
+2. Create the necessary Partial Views for your theme within `Views\Partials\Forms\Themes\<my-custom-theme>`.
+3. Provide the names of the files in your theme via an implementation of `ITheme`.
+  * For example, if only overriding a single file, your class would look like the code snippet below:
 
 ```csharp
 using Umbraco.Forms.Core.Interfaces;
@@ -67,7 +68,7 @@ public class MyCustomTheme : ITheme
 }
 ```
 
-In your project that consumes the theme, you register the ones you want to use via a composer:
+4. Register the themes you want to use via a composer:
 
 ```csharp
 public class MyComposer : IComposer
@@ -80,11 +81,11 @@ public class MyComposer : IComposer
 }
 ```
 
-With that in place your theme will be picked up for selection in the theme picker. And the partial view files included will be used when rendering forms.
+Your theme will now be available in the Theme Picker and the partial view files will be used when rendering forms.
 
 #### Email Templates
 
-Email templates provided for the send email workflow can be provided in a Razor Class Library in a similar way.
+Email templates provided for the send email workflow can be provided in a Razor Class Library similar to the Theme files.
 
 The partial view will be created in `Views\Partials\Forms\Emails`.
 

13/umbraco-forms/release-notes.md

@@ -23,21 +23,21 @@ This section contains the release notes for Umbraco Forms 13 including all chang
 
 The 13.3 release of Forms contains features that can improve the user experience of completing multi-page forms.
 
-We've added the option for [editors to choose to display paging details on the forms](./editor/creating-a-form/form-settings.md#multi-page-forms). This will allow those completing forms to get a better understanding of progress as well as see details of the pages still to complete. [#281](https://github.com/umbraco/Umbraco.Forms.Issues/issues/281) [#648](https://github.com/umbraco/Umbraco.Forms.Issues/issues/648).
+We have added the option for [editors to choose to display paging details on the forms](./editor/creating-a-form/form-settings.md#multi-page-forms). This will allow those completing forms to get a better understanding of progress as well as see details of the pages still to be completed. [#281](https://github.com/umbraco/Umbraco.Forms.Issues/issues/281) [#648](https://github.com/umbraco/Umbraco.Forms.Issues/issues/648).
 
-These can be options are enabled and configured on a per-form basis by editors in the form settings section. We also provide a [configuration based toggle for the feature as a whole](./developer/configuration/README.md#enablemultipageformsettings). In this way editors can be given access to use the feature only once the styling or theme is prepared.
+These options are enabled and configured by editors in the Forms settings section on a per-form basis. We also provide a [configuration-based toggle for the feature as a whole](./developer/configuration/README.md#enablemultipageformsettings). In this way, editors can be given access to use the feature only once the styling or theme is prepared.
 
 ##### Ship themes in Razor Class Libraries
 
 Forms ships it's themes and email templates as part of a razor class library for ease of distribution. With this release we make that feature [available to your own custom themes and templates](./developer/themes.md#shipping-themes-in-a-razor-class-library) (or those created by package developers) [#795](https://github.com/umbraco/Umbraco.Forms.Issues/issues/795).
 
 ##### Date picker field type
 
-We've also made a couple of updates to the date picker field type. The format for the field can now be provided in configuration [#1276](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1276). And you can now override and localize the aria label provided for assistive technologies such as screen readers [https://github.com/umbraco/Umbraco.Forms.Issues/issues/1082](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1082).
+We have made a couple of updates to the Date Picker field type. The format for the field can now be provided in configuration [#1276](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1276). And you can now override and localize the aria-label provided for assistive technologies such as screen readers [https://github.com/umbraco/Umbraco.Forms.Issues/issues/1082](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1082).
 
 ##### Finer grained entries permissions
 
-To allow finer control over editor permissions, we've introduced a "delete entries" setting available on users and user groups. Thus you can now give editors explicit permissions to view, edit or delete entries [#1303](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1303).
+To allow finer control over editor permissions, we have introduced a "delete entries" setting for users and user groups. Thus you can now give editors explicit permissions to view, edit, or delete entries [#1303](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1303).
 
 ##### Other
 

14/umbraco-forms/developer/configuration/README.md

@@ -523,7 +523,7 @@ A custom date format can be provided in [momentjs format](https://momentjscom.re
 
 #### DatePickerFormatForValidation
 
-If a custom date format is provided it will be used on the client-side. A matching string in [C# date format](https://learn.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings) should be provided, in order that server-side validation will match the expected format of the entry.
+If a custom date format is provided it will be used on the client-side. A matching string in [C# date format](https://learn.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings) should be provided, so that server-side validation will match the expected format of the entry.
 
 ### reCAPTCHA v2 field type configuration
 

14/umbraco-forms/developer/extending/adding-a-fieldtype.md

@@ -133,7 +133,7 @@ To support this, a read-only view of the field is necessary.
 
 For most fields, nothing is required here, as the default read-only display defined in the built-in `ReadOnly.cshtml` file suffices.
 
-However, if you want to provide a custom read-only display for your field, you can do so by creating a second partial view. This should be named with a `.ReadOnly` suffix. For this example, we would create `FieldType.Slider.ReadOnly.cshtml`.
+However, if you want to provide a custom read-only display for your field, you can do so by creating a second partial view. This should be named with a `.ReadOnly` suffix. For this example, you would create `FieldType.Slider.ReadOnly.cshtml`.
 
 ## Field Settings
 

14/umbraco-forms/developer/property-editors.md

@@ -1,32 +1,32 @@
 # Property Editors
 
-When forms are created, editors will want to add them to pages in Umbraco. To do this they need a Document Type with a property that uses a datatype based on a form picker property editor.
+When forms are created, editors will want to add them to pages in Umbraco. To do this they need a Document Type with a property that uses a Data Type based on a Form Picker property editor.
 
 Umbraco Forms provides three variations of a form picker.
 
 <figure><img src="images/form-pickers.png" alt=""><figcaption><p>Form Pickers</p></figcaption></figure>
 
 Most commonly used is **Form Picker (single)**. This will allow the editor to select a single form for display on page.
 
-Rarely but feasibly, you'll have a requirement to present multiple forms on a page. Should this be appropriate, you can use **Form Picker (multiple)**.
+Rarely but feasibly, you will have a requirement to present multiple forms on a page. Should this be appropriate, you can use **Form Picker (multiple)**.
 
 {% hint style="info" %}
 Internally this is used for presenting the list of "Allowed forms" you can select when setting up a form picker datatype.
 {% endhint %}
 
-Finally you can provide further flexibility for the editor to select not only a form, but also the theme and redirect as well. For this you'll use the **Form Details Picker**.
+Finally you can provide further flexibility for the editor to select not only a form, but also the theme and redirect as well. For this you will use the **Form Details Picker**.
 
 ## Configuring the Data Type
 
-Each property editor allows you to restrict the forms that can be chosen with the datatype. You do this by setting either or both of the list of "Allowed folders" or "Allowed forms".
+Each property editor allows you to restrict the forms that can be chosen with the Data Type. You do this by setting either or both of the list of "Allowed folders" or "Allowed forms".
 
 <figure><img src="images/form-picker-config.png" alt=""><figcaption><p>Form Picker DataType Configuration</p></figcaption></figure>
 
-THe "Form Details Picker" also allow you to select whether theme or redirect selection is available.
+The "Form Details Picker" also allows you to select whether a theme or redirect selection is available.
 
 ## Property Value Conversion
 
-The type of a property based on the form picker presented in a Razor class library is as followes:
+The type of a property based on the Form Picker presented in a Razor class library is as follows:
 
 | Option                           | Description                                                                                                                                                                                                     |
 | -------------------------------- | --------------------------------------------------------------------------------------------------------------------------------------- |
@@ -36,4 +36,4 @@ The type of a property based on the form picker presented in a Razor class libra
 
 ## Content Delivery API Expansion
 
-Each reference to a form supports expansion via the Umbraco Content Delivery API, as described [here](./ajaxforms.md#working-with-the-cms-content-delivery-api).
+Each reference to a form supports expansion via the Umbraco Content Delivery API, as described in the [Working with the CMS Content Delivery API](./ajaxforms.md#working-with-the-cms-content-delivery-api) article.

14/umbraco-forms/developer/themes.md

@@ -46,11 +46,12 @@ If adding or amending client-side scripts, you need to copy the `Script.cshtml`
 
 Umbraco Forms provides it's built-in themes as part of a Razor Class Library for ease of distribution. This can be useful for custom themes, particularly those used in multiple solutions or released as an Umbraco package.
 
-From Forms 14.2 it's possible to do this for custom themes.
+From Forms 14.2 it is possible to do this for custom themes.
 
-Firstly you'll create a new Razor Class Library project to hold the theme.  You then create the necessary partial views for your theme as usual within `Views\Partials\Forms\Themes\<my-custom-theme>`.
-
-You then need to provide the names of the files in your theme via an implementation of `ITheme`. For example, if only overriding a single file, your class would look like this:
+1. Create a new Razor Class Library project to hold the theme.
+2. Create the necessary Partial Views for your theme within `Views\Partials\Forms\Themes\<my-custom-theme>`.
+3. Provide the names of the files in your theme via an implementation of `ITheme`.
+  * For example, if only overriding a single file, your class would look like the code snippet below:
 
 ```csharp
 using Umbraco.Forms.Core.Interfaces;
@@ -68,7 +69,7 @@ public class MyCustomTheme : ITheme
 }
 ```
 
-In your project that consumes the theme, you register the ones you want to use via a composer:
+4. Register the themes you want to use via a composer:
 
 ```csharp
 public class MyComposer : IComposer
@@ -81,11 +82,11 @@ public class MyComposer : IComposer
 }
 ```
 
-With that in place your theme will be picked up for selection in the theme picker. And the partial view files included will be used when rendering forms.
+Your theme will now be available in the Theme Picker and the partial view files will be used when rendering forms.
 
 #### Email Templates
 
-Email templates provided for the send email workflow can be provided in a Razor Class Library in a similar way.
+Email templates provided for the send email workflow can be provided in a Razor Class Library similar to the Themes files.
 
 The partial view will be created in `Views\Partials\Forms\Emails`.
 

14/umbraco-forms/release-notes.md

@@ -24,9 +24,9 @@ This section contains the release notes for Umbraco Forms 14 including all chang
 
 The 14.2 release of Forms contains features that can improve the user experience of completing multi-page forms.
 
-We've added the option for [editors to choose to display paging details on the forms](./editor/creating-a-form/form-settings.md#multi-page-forms). This will allow those completing forms to get a better understanding of progress as well as see details of the pages still to complete. [#281](https://github.com/umbraco/Umbraco.Forms.Issues/issues/281) [#648](https://github.com/umbraco/Umbraco.Forms.Issues/issues/648).
+We have added the option for [editors to choose to display paging details on the forms](./editor/creating-a-form/form-settings.md#multi-page-forms). This will allow those completing forms to get a better understanding of progress as well as see details of the pages still to be completed. [#281](https://github.com/umbraco/Umbraco.Forms.Issues/issues/281) [#648](https://github.com/umbraco/Umbraco.Forms.Issues/issues/648).
 
-These can be options are enabled and configured on a per-form basis by editors in the form settings section. We also provide a [configuration based toggle for the feature as a whole](./developer/configuration/README.md#enablemultipageformsettings). In this way editors can be given access to use the feature only once the styling or theme is prepared.
+These options are enabled and configured by editors in the Forms settings section on a per-form basis. We also provide a [configuration-based toggle for the feature as a whole](./developer/configuration/README.md#enablemultipageformsettings). In this way, editors can be given access to use the feature only once the styling or theme is prepared.
 
 ##### Form picker enhancements
 
@@ -48,7 +48,7 @@ When creating a prevalue source based on Umbraco documents, you can now select c
 
 ##### Finer grained entries permissions
 
-To allow finer control over editor permissions, we've introduced a "delete entries" setting available on users and user groups. Thus you can now give editors explicit permissions to view, edit or delete entries [#1303](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1303).
+To allow finer control over editor permissions, we have introduced a "delete entries" setting for users and user groups. Thus you can now give editors explicit permissions to view, edit, or delete entries [#1303](https://github.com/umbraco/Umbraco.Forms.Issues/issues/1303).
 
 ##### Backoffice localization
 

15/umbraco-forms/developer/configuration/README.md

@@ -516,7 +516,7 @@ A custom date format can be provided in [momentjs format](https://momentjscom.re
 
 #### DatePickerFormatForValidation
 
-If a custom date format is provided it will be used on the client-side. A matching string in [C# date format](https://learn.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings) should be provided, in order that server-side validation will match the expected format of the entry.
+If a custom date format is provided it will be used on the client side. A matching string in [C# date format](https://learn.microsoft.com/en-us/dotnet/standard/base-types/custom-date-and-time-format-strings) should be provided, so that server-side validation will match the expected format of the entry.
 
 ### reCAPTCHA v2 field type configuration