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

loleaflet: fix "offsetWidth" property value of the sidebar panel #771

Merged
merged 1 commit into from
Nov 26, 2020

Conversation

hcvcastro
Copy link
Member

Unfortunately when show/hide sidebar the "offsetWidth" property
works only if the element is visible otherwise it returns parent
empty size.

To fix add temporary a visibility property, then obtain the sidebar
"offsetWidth" property and update document container size.

Change-Id: I8e1c821518a4c155340edcfc8f2313098edff8b0
Signed-off-by: Henry Castro hcastro@collabora.com

Unfortunately when show/hide sidebar the "offsetWidth" property
works only if the element is visible otherwise it returns parent
empty size.

To fix add temporary a visibility property, then obtain the sidebar
"offsetWidth" property and update document container size.

Change-Id: I8e1c821518a4c155340edcfc8f2313098edff8b0
Signed-off-by: Henry Castro <hcastro@collabora.com>
@hcvcastro hcvcastro merged commit bff84f8 into CollaboraOnline:master Nov 26, 2020
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

Successfully merging this pull request may close these issues.

1 participant