Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Update to new version of slate #2202

Merged
merged 3 commits into from
Oct 10, 2018
Merged

Update to new version of slate #2202

merged 3 commits into from
Oct 10, 2018

Conversation

dbkr
Copy link
Member

@dbkr dbkr commented Oct 9, 2018

Lots of fixes here as a lot of the API has changed (eg. anchorKey /
offsetKey are now anchor.key and offset.key, and collapseFocusToThing
is moveFocusToThing).

Also changes the ref to a function (sorry for lumping this into the
same PR).

Hopefully will fix element-hq/element-web#7105
Presumably fixes element-hq/element-web#7419

Lots of fixes here as a lot of the API has changed (eg. anchorKey /
offsetKey are now anchor.key and offset.key, and collapseFocusToThing
is moveFocusToThing).

Also changes the ref to a function (sorry for lumping this into the
same PR).

Hopefully will fix element-hq/element-web#7105
@dbkr dbkr requested a review from a team October 9, 2018 10:00
@dbkr dbkr mentioned this pull request Oct 9, 2018
dbkr added 2 commits October 9, 2018 16:05
For some reason the slate update means the composer doesn't
have the focus after closing the room settings, and the end to end
tests pick this up!
Copy link
Member

@turt2live turt2live left a comment

Choose a reason for hiding this comment

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

I have relatively little knowledge on the composer, but this all seems right. Eager to test it on /develop and find any bugs that way though :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants