-
Notifications
You must be signed in to change notification settings - Fork 20
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
Menu always visible #150
Comments
I have worked on this in my fork: https://github.com/fablab-luenen/dokuwiki-visual-editor. If you want I can cherry-pick and make a PR, but it looks like the current maintainers are not super responsive and my current focus is on getting ahead. |
PR would be welcome |
Yes, please make a PR I've installed your fork, but after starting it the first time, I have the problem described here: |
Did you use the address provided in the Readme? As described in the linked issues just cloning the repository doesn’t work, it had to be installed from a release version (which includes the compiled javascript bundle). |
Turns out it‘s already in my styling PR: #147 @Juergen-aus-Zuendorf are you sure the error is exactly that described in the issue? Because in testing I‘ve also discovered another one, in which the visual editor wouldn‘t open (but without an error message): fablab-luenen#13 Also, do you have a header? Because the toolbar should already be sticky through a conditional position: fixed set through some javascript, but you may not see it if it‘s obscured by a fixed header. You can see whether the stickiness implementation of the current code works by observing a slight jump to the top while scrolling down (this is when the menubar transitions to having position: fixed and thus is taken out of the page layout - the jump is removed by my change). |
I had indeed clicked on the link in the ReadMe file and installed the editor from there. The hint further down I had then admittedly overread. Have now installed everything according to the instructions and it works. Thanks a lot! |
Since this issue is quite old: Are your changes already included via Pull Request or is it still open? |
I made some little changes to style.less which makes the menubar sticky on top of page. The problem with the current code is, that the parent element has "overflow: hidden;" set. That is why "position: sticky;" does not work. So adding "overflow: visible;" to the parent element fixes the problem. I added "position: -webkit-sticky;" for compatible reasons to ".navbar": And I added "overflow: visible;" to "#dokuwiki__content": That's it. I made a Pull Request #179 for the changes. Until then, you can add the code also to your userstyle.css:
|
My code has been merged, but theme specific CSS has been rejected before: #147 (comment) It may make sense to contribute the relevant changes to the theme itself. |
Ok, so your changes work with the Original DokuWiki theme? This was not the case in my installation, that is why I thought the changes were not merged. |
@annda I think the IntersectionObserver is the preferable way. Using onscroll will execute many many times when scrolling, rerunning the same code again and again. The observer simply triggers an event on intersections and only then your code runs. You might even be able to see the difference in a CPU graph while scrolling... |
Implemented in #189 |
If you have a longer text, you can move the editor area down using the scroll bars. In this case, however, the menu buttons, e.g. for changing the font attributes, disappear. So you have to scroll up all the time for the menu keys to trigger an action with them at the bottom of the text.
The text was updated successfully, but these errors were encountered: