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
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