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.

Sign up for GitHub

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

Jump to bottom

[stable25] fix: paste tables into document #4326

Merged
merged 2 commits into from
Jun 22, 2023
Merged

[stable25] fix: paste tables into document #4326

merged 2 commits into from
Jun 22, 2023

Commits on Jun 22, 2023

  1. fix: paste tables into document 

    Fixes #2708.

Try to read the first table row as headings.

Pasting works via `insertContent` while opening uses `setContent`.
They use the schema in different ways.
So we also need to make sure to test both for some corner cases.

`setContent` is fairly flexible in turning the input
into a valid document structure.
`insertContent` however fails to resolve structures
that would require picking lower priority parent elements.

Note: Some tests in src/tests/nodes/Table.spec.js
fail when using `insertContent` instead of `setContent`.
Pasting the correponding html table is fixed never the less.

Signed-off-by: Max <max@nextcloud.com>
Signed-off-by: Jonas <jonas@freesources.org>
    @max-nextcloud @juliusknorr
    max-nextcloud authored and juliusknorr committed Jun 22, 2023
    Configuration menu
    Copy the full SHA
    e38b5b4 View commit details
    Browse the repository at this point in the history

  2. chore(assets): Recompile assets 

    Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
    @nextcloud-command
    nextcloud-command committed Jun 22, 2023
    Configuration menu
    Copy the full SHA
    f7833f6 View commit details
    Browse the repository at this point in the history