Skip to content

fix(paragraph): migrate preview options to decorations #6425

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

Merged
mejo- merged 13 commits into from
Sep 19, 2024
Merged

fix(paragraph): migrate preview options to decorations #6425

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
28c3e0a
enh: add extractLinkParagraphs
max-nextcloud Sep 17, 2024
899886c
refactor(test): reuse link string
max-nextcloud Sep 17, 2024
4ab1cf8
enh(preview): include previews in list of link paragraphs
max-nextcloud Sep 17, 2024
2cb1c54
feat(previewOptions): Migrate preview options to prosemirror decorations
mejo- Sep 17, 2024
339dd63
chore(preview): Remove PreviewOptions from Paragraph and Preview
mejo- Sep 17, 2024
c76b6c0
enh(preview): add nodeSize in decoration data structure
max-nextcloud Sep 17, 2024
c635659
fix(preview): use deleteRange to remove the node
max-nextcloud Sep 17, 2024
1f40784
fix(preview): Fix alignment of preview options
mejo- Sep 17, 2024
c5fa33c
fix(paragraph): get rid of superfluous ParagraphView component
mejo- Sep 17, 2024
36d7954
fix(preview): Don't let preview options container overlay paragraph
mejo- Sep 18, 2024
a7d121f
fix(preview): Fix some linting issues
mejo- Sep 18, 2024
ae77d20
test(cy): Fix selectors for paragraphs in cypress tests
mejo- Sep 18, 2024
5d61e84
fix(preview): Further improvements
mejo- Sep 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix(preview): Don't let preview options container overlay paragraph 
Otherwise the overlay container would catch all the click events.

Signed-off-by: Jonas <[email protected]>
  • Loading branch information
@mejo-
mejo- committed Sep 18, 2024
commit 36d79543cffc1636b4df807d9721160ac3eb0e43
3 changes: 2 additions & 1 deletion src/components/Editor/PreviewOptions.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -112,13 +112,14 @@ div[contenteditable=false] {

.preview-options-container {
position: absolute;
width: 0 !important;
left: -44px;
top: 50%;
transform: translate(0, -50%);
}

// Inside details, button needs to be shifted further
.details-content div[data-text-preview-options] {
.details-content .preview-options-container {
left: calc(-44px - 24px);
}

Expand Down
2 changes: 2 additions & 0 deletions src/css/prosemirror.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -131,6 +131,7 @@ div.ProseMirror {
}

p {
position: relative;
margin-bottom: 1em;
line-height: 150%;
}
Expand Down Expand Up @@ -281,6 +282,7 @@ div.ProseMirror {
padding-left: 3px;

p {
position: relative;
margin-bottom: 0.5em;
}
}
Expand Down