Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Priorities #24

Open
14 of 15 tasks
Noobulater opened this issue Oct 26, 2024 · 8 comments
Open
14 of 15 tasks

Priorities #24

Noobulater opened this issue Oct 26, 2024 · 8 comments

Comments

@Noobulater
Copy link
Contributor

Noobulater commented Oct 26, 2024

Great work so far

Living Issue so I'll update it as I think about it

Priority in order

@bmingles
Copy link
Contributor

@Noobulater I pushed hovering toolbar example last night

@Noobulater
Copy link
Contributor Author

Gave it a little inspection, everything looks to be in order. I checked out the slate editor state and the state seems to be mutating correctly!

Updated the priorities in the original post. Let's use this ticket for progress tracking, and this ticket for misc discussion about priorities and coordiantion. I'm doing some light investigation on the huge document stuff atm

@bmingles
Copy link
Contributor

bmingles commented Nov 5, 2024

@Noobulater I'm not familiar with the scroll into view behavior. Can you give me an example scenario to test this behavior?

@Noobulater
Copy link
Contributor Author

Noobulater commented Nov 5, 2024

@bmingles

I have some old code that had buttons linked to headers as scrollable links, since there is no example on slate's website, I'll handle it by making a forked version of the huge document

Edit: So looking at my old implementation, it uses a HTMLElement.scrollIntoView, I'll see about putting it

    const { selection } = editor
    if (selection) {
      const range = ReactEditor.toDOMRange(editor, selection)
      if (range && !isInViewport(range.endContainer.parentElement))
        range.endContainer.parentElement.scrollIntoView({ inline: "end", block: "end" })
    }

@Noobulater
Copy link
Contributor Author

I have a working example, I'll put it up for MR here in the next couple of days.

@bmingles
Copy link
Contributor

bmingles commented Nov 6, 2024

FWIW, there is an example in the slate repo, it just doesn't seem to be exposed in the app: https://github.com/ianstormtaylor/slate/blob/main/site/examples/ts/scroll-into-view.tsx

I just wasn't sure how it was intended to work. Great if you have a working example though.

@bmingles
Copy link
Contributor

bmingles commented Nov 13, 2024

@Noobulater I think I’ve worked through all of the examples + known issues (minus agredd upon exclusions like iframes). Would be good to get your input if there are any remaining priorities.

I’ve also published latest @slate-solid/core npm package

@Noobulater
Copy link
Contributor Author

@bmingles Sounds good, will try to look at this week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants