webxdc/editor
16
9
Fork
You've already forked editor
2

toolbar scrolls away, mostly noticable on iOS #43

Open
opened 2023年08月12日 17:57:58 +02:00 by r10s · 10 comments
Owner
Copy link

extracted from #11

failed attempt to fix it #32

the whole toolbar probably needs to have classic, sticky CSS; currently, this is not the case afaik

extracted from https://codeberg.org/webxdc/editor/issues/11 failed attempt to fix it https://codeberg.org/webxdc/editor/pulls/32 the whole toolbar probably needs to have classic, sticky CSS; currently, this is not the case afaik

Someone said that position: sticky; works for them https://github.com/ProseMirror/prosemirror-menu/issues/3#issuecomment-482245327

Someone said that `position: sticky;` works for them https://github.com/ProseMirror/prosemirror-menu/issues/3#issuecomment-482245327

778c4a2378be5683b7f065db8c763e631b2675a2 also didn't solve the issue, it still gets scrolled away (thanks @r10s for testing).
It's literally height: 100vh and position: static, I don't know how it works.

778c4a2378be5683b7f065db8c763e631b2675a2 also didn't solve the issue, it still gets scrolled away (thanks @r10s for testing). It's literally `height: 100vh` and `position: static`, I don't know how it works.
Author
Owner
Copy link

below is an example that shows what goes wrong.

it is not only that the toolbar scrolls away, partly, the cursor also goes out of view when hitting ENTER. and, as said, eg. it is very hard to mark a word to make it eg. "bold"

i think, the issue is that sticky is not used currently. there are lots of toolbars and non-scrolling areas on ios (see calendar and tons of other examples), it is not a general issue, prosemirror either does sth. wrong here or the current implementation is desired (i doubt that),

@WofWca however, it is fine to leave this issue open for now, this is hard to solve without an ios/safari; i or someone else with ios/safari should have a look at that

below is an example that shows what goes wrong. it is not only that the toolbar scrolls away, partly, the cursor also goes out of view when hitting ENTER. and, as said, eg. it is very hard to mark a word to make it eg. "bold" i think, the issue is that `sticky` is _not_ used currently. there are lots of toolbars and non-scrolling areas on ios (see calendar and tons of other examples), it is not a general issue, prosemirror either does sth. wrong here or the current implementation is desired (i doubt that), @WofWca however, it is fine to leave this issue open for now, this is hard to solve without an ios/safari; i or someone else with ios/safari should have a look at that

i think, the issue is that sticky is not used currently

We tried it in #32

And in 778c4a2378be5683b7f065db8c763e631b2675a2 it's even positioned statically

> i think, the issue is that sticky is not used currently We tried it in #32 And in 778c4a2378be5683b7f065db8c763e631b2675a2 it's even positioned statically
Author
Owner
Copy link

yes, the issues seems to be trickier :)

yes, the issues seems to be trickier :)
Author
Owner
Copy link

if one looks closely on desktop, when scrolling in a larger document, also there, the toolbar is moved up for a fraction of a second. it is then immediately corrected. but still, this is a bit weird and obviously error prone.

see the video attached and look at the toolbar on startup. it should just not move.

if one looks closely on **desktop**, when scrolling in a larger document, also there, the **toolbar is moved up for a fraction of a second.** it is then immediately corrected. but still, this is a bit weird and obviously error prone. see the video attached and look at the toolbar on startup. it should just not move.
r10s changed title from (削除) toolbar scrolls away on iOS (削除ここまで) to toolbar scrolls away, mostly noticable on iOS 2023年09月21日 16:38:03 +02:00
Owner
Copy link

on desktop after window-resizing i definitely get weird toolbar behaviour (missing buttons etc.) which only heals when i click into the text-area.

on desktop after window-resizing i definitely get weird toolbar behaviour (missing buttons etc.) which only heals when i click into the text-area.
Contributor
Copy link

Is it easily reproducible on any safari browser or does it have to be on iOS? E.g. would a normal Mac lappy work?

Is it easily reproducible on any safari browser or does it have to be on iOS? E.g. would a normal Mac lappy work?

works in desktop mostly, but on iOS this issue is really annoying and prevents me from making headings in the bottom of my document.

we could probably change the layout to:

container (flex box flex-column or grid)
 menu bar
 document container (flex grow, overflow-y scroll)

this would avoid needing to deal with absolute/fixed/sticky position stuff.

works in desktop mostly, but on iOS this issue is really annoying and prevents me from making headings in the bottom of my document. we could probably change the layout to: ``` container (flex box flex-column or grid) menu bar document container (flex grow, overflow-y scroll) ``` this would avoid needing to deal with absolute/fixed/sticky position stuff.

I believe what you're talking about has been attempted in 778c4a2378be5683b7f065db8c763e631b2675a2

I believe what you're talking about has been attempted in 778c4a2378be5683b7f065db8c763e631b2675a2
Sign in to join this conversation.
No Branch/Tag specified
main
tooling
add-presence
link2xt/autosave-interval-limit
hpk/tryquill
hpk/cleanup-indexhtml
target-legacy
inline-b64
webxdc-x-web-version
pages-demo
status-string
toolbar-position-sticky
v3.2.0
v3.1.1
v3.1.0
v3.0.0
v2.7.0
v2.6.0
v2.5.1
v2.5.0
v2.4.0
v2.3.0
v2.2.1
v2.2.0
v2.1.0
v2.0.2
v2.0.1
v2.0.0
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
5 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
webxdc/editor#43
Reference in a new issue
webxdc/editor
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?