Merged
Conversation
This change adds support for the `transformPresence()` method that
[`sharedb` uses][1].
We add support for both `text0` and `json0`.
`text0`
-------
The `text0` implementation leans on the existing
[`transformPosition`][2], and takes its form and tests from
[`rich-text`][3].
Its shape takes the form:
```js
{
index: 3,
length: 5,
}
```
Where:
- `index` is the cursor position
- `length` is the selection length (`0` for a collapsed selection)
`json0`
-------
The `json0` implementation has limited functionality because of the
limitations of the `json0` type itself: we handle list moves `lm`, but
cannot infer any information when moving objects around the tree,
because the `oi` and `od` operations are destructive.
However, it will attempt to transform embedded subtypes that support
presence.
Its shape takes the form:
```js
{
p: ['key', 123],
v: {},
}
```
Where:
- `p` is the path to the client's position within the document
- `v` is the presence value
The presence value `v` can take any arbitrary value (in simple cases it
may even be omitted entirely).
The exception to this is when using subtypes, where `v` should take the
presence shape defined by the subtype. For example, when using `text0`:
```js
{
p: ['key'],
v: {index: 5, length: 0},
}
```
[1]: share/sharedb#322
[2]: https://github.com/ottypes/json0/blob/90a3ae26364c4fa3b19b6df34dad46707a704421/lib/text0.js#L147
[3]: ottypes/rich-text#32
dawidreedsy
approved these changes
Dec 23, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change adds support for the
transformPresence()method thatsharedbuses.We add support for both
text0andjson0.text0The
text0implementation leans on the existingtransformPosition, and takes its form and tests fromrich-text.Its shape takes the form:
Where:
indexis the cursor positionlengthis the selection length (0for a collapsed selection)json0The
json0implementation has limited functionality because of thelimitations of the
json0type itself: we handle list moveslm, butcannot infer any information when moving objects around the tree,
because the
oiandodoperations are destructive.However, it will attempt to transform embedded subtypes that support
presence.
Its shape takes the form:
Where:
pis the path to the client's position within the documentvis the presence valueThe presence value
vcan take any arbitrary value (in simple cases itmay even be omitted entirely).
The exception to this is when using subtypes, where
vshould take thepresence shape defined by the subtype. For example, when using
text0: