Apply suggestions from code review

Co-authored-by: Khushal Sagar <63884798+khushalsagar@users.noreply.github.com>

Author: noamr

css-view-transitions-2/Overview.bs

@@ -508,7 +508,7 @@ The {{CSSViewTransitionRule}} represents a ''@view-transition'' rule.
 		Manipulating the {{ViewTransition/types}} in the old document does not affect the types in the new document,
 		which would be read from the [=@view-transition/types=] descriptor once the document is revealed.
 
-		Note: the {{ViewTransition}} is skipped once the old document is hidden.
+		Note: The {{ViewTransition}} is skipped once the old document is hidden.
 
 	1. Set |outboundTransition|'s [=outbound post-capture steps=] to the following steps given a [=view transition params=]-or-null |params|:
 		1. Set |newDocument|'s [=inbound view transition params=] to |params|.
@@ -525,7 +525,7 @@ The {{CSSViewTransitionRule}} represents a ''@view-transition'' rule.
 		* The {{Window/pagereveal}} event is fired.
 		* its [=active view transition=]'s [=ViewTransition/phase=] is "`done`".
 
-		Note: this is to ensure that there are no unintended flashes between displaying the old and new state, to keep the transition smooth.
+		Note: This is to ensure that there are no unintended flashes between displaying the old and new state, to keep the transition smooth.
 
 	1. Return |outboundTransition|.
 </div>
@@ -543,7 +543,7 @@ To <dfn>update the opt-in state for outbound transitions</dfn> for a {{Document}
 	Append the following steps to [=skip the view transition=] given a {{ViewTransition}} |transition|:
 		1. If |transition|'s [=outbound post-capture steps=] is not null, then run |transition|'s [=outbound post-capture steps=] with null.
 
-	Note: This is written in a monkey-patch manner, and will be merged into the algorithm once the L1 spec graduates.
+	Note: This is written in a monkey-patch manner, and will be merged into the algorithm once the L2 spec graduates.
 </div>
 
 ### Proceed with cross-document view transition after capturing the old state ### {#cross-doc-after-capture}
@@ -570,13 +570,13 @@ Prepend the following step to the [=Perform pending transition operations=] algo
 
 		1. Call |transition|'s [=outbound post-capture steps=] given |viewTransitionParams|.
 
-	Note: This is written in a monkey-patch manner, and will be merged into the algorithm once the L1 spec graduates.
+	Note: This is written in a monkey-patch manner, and will be merged into the algorithm once the L2 spec graduates.
 </div>
 
 ## Activating the view transition in the new {{Document}} ## {#access-view-transition-in-new-doc}
 
 <div algorithm>
-	To <dfn export>resolve inbound cross-document view-transition</dfn> for {{Document}} |document|:
+	To <dfn export>activate inbound cross-document view-transition</dfn> for {{Document}} |document|:
 
 	1. [=Assert=]: |document| is [=fully active=].
 
@@ -592,14 +592,14 @@ Prepend the following step to the [=Perform pending transition operations=] algo
 
 	1. If |document|'s [=active view transition=] is not null, then return null.
 
-		Note: this means that starting a same-document transition before revealing the document would cancel a pending cross-document transition.
+		Note: This means that starting a same-document transition before revealing the document would cancel a pending cross-document transition.
 
 	1. [=Resolve @view-transition rule=] for |document| and let |resolvedRule| be the result.
 
 	1. If |resolvedRule| is "<code>skip transition</code>", then return null.
 
 	1. Let |transition| be a new {{ViewTransition}} in |document|'s [=relevant Realm=],
-		whose [=ViewTransition/named elements=] is |inboundViewTransitionParams|'s [=view transition params/named elements=],
+		whose [=ViewTransition/named elements=] is a clone of |inboundViewTransitionParams|'s [=view transition params/named elements=],
 		and [=ViewTransition/initial snapshot containing block size=] is |inboundViewTransitionParams|'s [=view transition params/initial snapshot containing block size=].
 
 	1. Set |document|'s [=active view transition=] to |transition|.
@@ -612,6 +612,7 @@ Prepend the following step to the [=Perform pending transition operations=] algo
 
 	1. At any given time, the UA may decide to skip the inbound transition, e.g. after an [=implementation-defined=] timeout.
 		To do so, the UA should [=queue a global task=] on the [=DOM manipulation task source=] given |document|'s [=relevant global object=] to perform the following step:
+			
 			If |transition|'s [=ViewTransition/phase=] is not "`done`", then [=skip the view transition=] |transition| with a "{{TimeoutError}}" {{DOMException}}.
 
 	1. Return |transition|.
@@ -631,7 +632,7 @@ A {{Document}} additionaly has:
 	:: a boolean.
 		Initially false.
 
-		Note: this value can be read [=in parallel=] while navigating.
+		Note: This value can be read [=in parallel=] while navigating.
 </dl>
 
 ### Additions to {{ViewTransition}} ### {#cross-doc-data-structure-vt}
@@ -718,7 +719,7 @@ The <dfn id='active-view-transition-pseudo'>:active-view-transition</dfn> pseudo
 
 The [=specificity=] of an '':active-view-transition'' is one pseudo-class selector.
 
-An '':active-view-transition'' pseudo-class matches the [=document element=] when its [=node document=] has an non-null [=active view transition=].
+An '':active-view-transition'' pseudo-class matches the [=document element=] when its [=node document=] has a non-null [=active view transition=].
 
 ### '':active-view-transition-type()'' ### {#the-active-view-transition-type-pseudo}
 
@@ -731,10 +732,10 @@ It has the following syntax definition:
 
 The [=specificity=] of an '':active-view-transition-type()'' is one pseudo-class selector.
 
-An '':active-view-transition-type()'' pseudo-class matches the [=document element=] when its [=node document=] has an non-null [=active view transition=],
+An '':active-view-transition-type()'' pseudo-class matches the [=document element=] when its [=node document=] has a non-null [=active view transition=],
 whose [=ViewTransition/active types=] [=list/contains=] at least one of the <<custom-ident>> arguments.
 
-## Activating the transition type using JavaScript ## {#types-js}
+## Providing types in JavaScript ## {#types-js}
 
 ### Extending {{Document/startViewTransition|document.startViewTransition()}} ### {#extend-document-types}
 	<xmp class=idl>
@@ -798,7 +799,7 @@ A {{ViewTransition}} additionally has:
 	:: A [=/set] of strings, initially empty.
 </dl>
 
-## Activating the transition type for cross-document view transitions ## {#types-cross-doc}
+## Providing types in CSS ## {#types-cross-doc}
 
 ### The [=@view-transition/types=] descriptor ### {#view-transition-type-descriptor}
 
@@ -810,9 +811,9 @@ A {{ViewTransition}} additionally has:
 	</pre>
 
 	The '<dfn for="@view-transition">types</dfn>' descriptor sets the [=ViewTransition/active types=] for the transition
-	when capturing or performing the transition, equivalent to calling {{Document/startViewTransition(callbackOptions)}} with that {{StartViewTransitionOptions/types}}.
+	when capturing or performing cross-document transitions, equivalent to calling {{Document/startViewTransition(callbackOptions)}} with the same {{StartViewTransitionOptions/types}}.
 
-	Note: the [=@view-transition/types=] descriptor only applies to the {{Document}} in which it is defined.
+	Note: The [=@view-transition/types=] descriptor only applies to the {{Document}} in which it is defined.
 	The author is responsible for using their chosen set of types in both documents.
 
 # Sharing styles between view transition pseudo-elements # {#shared-style-with-vt-classes}
@@ -826,7 +827,7 @@ setting a name that's shared across multiple elements, and then using the [=clas
 
 [=Named view transition pseudo-elements|The view transition pseudo-elements=] (e.g. ''view-transition-group()'') are not defined in the DOM, but rather by using the ''view-transition-name'' property.
 For that purpose, the ''view-transition-class''' CSS property provides view transitions with the equivalent of HTML [=Element/class|classes=].
-When an element with a ''view-transition-name'' also has a ''view-transition-class'' value, that class would be selectable by the pseudo-elements, as per the <a href="#vt-class-example">examples</a>.
+When an element with a ''view-transition-name'' also has a ''view-transition-class'' value, that class can be used to select multiple pseudo-elements, as shows in the <a href="#vt-class-example">examples</a>.
 
 ## Examples ## {#vt-class-example}
 
@@ -879,9 +880,9 @@ div.box {
 	</pre>
 
 	The 'view-transition-class' can be used to apply the same style rule to multiple [=named view transition pseudo-elements=] which may have a different 'view-transition-name'.
-	While 'view-transition-name' is used to match between the element in the old state with its corresponding element in the new state, 'view-transition-class' is used
-	only to apply styles using the view transitionpseudo-elements
-	(''::view-transition-group()'', ''::view-transition-image-pair()'', ''::view-transition-old()'', ''::view-transition-new()'').
+	While 'view-transition-name' is used to match between the element in the old state with its corresponding element in the new state,
+	'view-transition-class' is used
+	only to apply styles to the [=named view transition pseudo-elements=].
 
 	Note that 'view-transition-class' by itself doesn't mark an element for capturing, it is only used as an additional
 	way to style an element that already has a 'view-transition-name'.