8000 Popup positioning · Issue #36 · attoae/quill-table-better · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Popup positioning #36

New issue

Have a question about thi 8000 s 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
Closed
Intrflex opened this issue Oct 23, 2024 · 6 comments
Closed

Popup positioning #36

Intrflex opened this issue Oct 23, 2024 · 6 comments

Comments

@Intrflex
Copy link
Intrflex commented Oct 23, 2024 •
edited
Loading

your positioning logic doesn't account for scrolling documents, see examples the positioning does not seem to work

when the document is scrolled down the table properties appears at the top no where near the table itself
image

see below the table cell properties gets cut off
image

would fixed positioning not work better
@Intrflex
Copy link
Author

it appears the more columns you put in the worst the cut off gets

@attoae
Copy link
Owner
attoae commented Oct 29, 2024

when the document is scrolled down the table properties appears at the top no where near the table itself

The table properties disappear when you scroll, unless you focus the table again.

see below the table cell properties gets cut off

I will fix this.
Do you use iframe or the container uses overflow: hidden;.
It is usually shown in the following way：

@Intrflex
Copy link
Author

Hi thanks for getting back yeah i do have a an overflow hidden on a parent container its unfortunately needed though no iframes

@attoae
Copy link
Owner
attoae commented Oct 31, 2024

The parent container is <div id="root" class="ql-container ql-snow"> or other?

@Intrflex
Copy link
Author

no I have a container around it called a4 specifying the size, I've managed to fix it by restructuring the page content so the parent is no longer overflow hidden I do think a fixed positioning system would be better though as no matter what no one would have these issues

my structure goes something like this

<div class="a4"> <- being the overflow hidden
      <div class="editor></div>

@attoae
Copy link
Owner
attoae commented Oct 31, 2024

I limited the bounds of the popup to the editor.

@attoae attoae mentioned this issue Nov 3, 2024
Merged
@attoae attoae closed this as completed Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Intrflex @attoae
0