Skip to content

Commit 801b2d4

Browse files
committed
Address some review comments from @noamr.
1 parent c30391d commit 801b2d4

File tree

1 file changed

+6
-20
lines changed

1 file changed

+6
-20
lines changed

snap-to-activate-explainer.md

Lines changed: 6 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,8 @@
22
We'd like to make it easier and more reliable for developers to build user experiences
33
where navigation can be done by moving parts of the UI
44
in a way that can correspond to scrolling and then scroll snapping.
5-
This navigation should be able to change the URL
6-
(so that the resulting state can be linked to and shared)
7-
and [eventually](route-matching-explainer.md#declarative-patch-based-document-updates)
8-
should be able to trigger loading of the resources needed
9-
to display the new part of the UI.
5+
This navigation should be able to change the URL,
6+
so that the resulting state can be linked to and shared.
107

118
## Use cases
129

@@ -99,10 +96,10 @@ etc.
9996
> to load new content if any is available
10097
> (which in this case it is not).
10198
102-
### Swipe actions (like swipe to delete)
99+
### Swipe actions (like swipe to dismiss/delete)
103100

104101
Another use case that is similar in many ways to pull-to-refresh
105-
is swipe-to-delete.
102+
is swipe-to-dismiss or swipe-to-delete.
106103
Many user interfaces with lists of items that the user can delete
107104
(for example, lists of messages)
108105
allow swiping an individual item to the side to either delete an item
@@ -125,24 +122,13 @@ although some aspects may also be different.)
125122

126123
Some things worth thinking about when designing a solution for this are:
127124

128-
* Snap-to-activate should integrate with the work on
129-
[declarative patching](patching-explainer.md),
130-
since some of the use cases may involve
131-
activating substantial pieces of additional content.
132-
133125
* An element, particularly one within a single-page app,
134126
may be activated in this way more than once in the lifetime of the page.
135-
(This implies that while integration with declarative patching is useful,
127+
(This implies that while integration
128+
with [declarative patching](patching-explainer.md) is useful,
136129
the integration shouldn't be so tight that a second activation is
137130
difficult or has poor developer ergonomics.)
138131

139-
* Snap-to-activate should integrate with
140-
[declarative routing](route-matching-explainer.md) because
141-
this gives the developer access to a declarative way to
142-
start view transitions,
143-
or perhaps to respond to navigations in other ways.
144-
(TODO: Does this make sense? Clarify?)
145-
146132
* While in many cases the snapping involved will be
147133
`scroll-snap-type: mandatory`,
148134
the design should probably consider `scroll-snap-type: proximity` as well.

0 commit comments

Comments
 (0)