Description of problem:
After a patch for #1391418 disallowed detaching toolbars, they get messed up if dropped to a position where they can't be attached.
Version-Release number of selected component (if applicable):
Always reproducible with 188.8.131.52-6 and 184.108.40.206-7 if running on wayland.
Never reproducible with 220.127.116.11-5 and 18.104.22.168-4, both of them have bug #1391418 instead.
Steps to Reproduce:
1. open any Libreoffice component (e.g. writer)
2. drag any toolbar
3. drop it into the text pane (where it would detach before the latest update)
While dragging the toolbar shadow indicates that the toolbar will be detached.
After releasing the mouse button, the toolbar is either gone completely or is drawn behind other toolbars or the main content (text or cells or drawing area on Writer / Calc / Draw).
Allow detaching toolbars.
Don't draw a shadow if the toolbar won't be there after drop.
If the toolbar cannot be shown where it was dropped, snap back to its original position but don't put it to some random position.
This bug was reported as https://bugzilla.redhat.com/show_bug.cgi?id=1391418#c7 first, but caolanm told me to file a separate bug.
I'd like to contribute a screencast but I can't get it working, see https://email@example.com/thread/S37KBMW5A6RX5X2UXKG7XB45BFLAEBRK/ and https://bugzilla.redhat.com/show_bug.cgi?id=1391418#c10
"allow detaching toolbars", is not going to happen as far as I can see. Wayland is just a disaster for the detachable but dockable toolbar concept, gtk itself has deprecated its own GtkHandleBox and removed its own tearable menus and anything similar so the support just isn't there in the toolkit and as far as I can tell (and I've tried quite a few things) there's no reasonable way currently to do what we want.
things should definitely just snap back to where they came from though.
Created attachment 1222655 [details]
A sample screencast showning the problem described in comment #0.
Getting the toolbar to hide completely is not always working. Getting it into a broken state is fairly easy.
Additionally, adding new toolbars from the menu (view -> toolbars -> select any of them) doesn't always work. You often have to restart LibreOffice to make the toolbars appear again. This also is a regression from X11 or recent LibreOffice versions. It seems that toolbars which are detached by default or which were detached last time they were used are affected.
Steps to reproduce for this:
1. open an old LibreOffice version
2. detach any toolbar, remember the name and keep it detached
3. close the toolbar or hide it from view -> toolbars menu
4. update to newer LibreOffice version
5. open LibreOffice
6. add toolbar from view -> toolbars menu
Toolbar isn't shown until you restart LibreOffice.
What should happen:
Toolbar should be shown when activated.
Created attachment 1222702 [details]
bug demonstration video
I can reproduce the same thing in writer. The toolbars are not lost, just hidden behind each other, or behind the document, and can be restored. It happens when you try to move the toolbar, and the last docking position is vertical, and then you move it somewhere undockable and release it. It stays at the same position, but flips vertically, and thus gets hidden behind the content pane. When you try to move it back, it sometimes gets hidden behind a different toolbar. See the video.
libreoffice-22.214.171.124-9.fc25 has been submitted as an update to Fedora 25. https://bodhi.fedoraproject.org/updates/FEDORA-2016-04ed64a2ee
The toolbars look OK to me now, I can't mess them up anymore. What about you, Christian?
libreoffice-126.96.36.199-9.fc25 has been pushed to the Fedora 25 testing repository. If problems still persist, please make note of it in this bug report.
See https://fedoraproject.org/wiki/QA:Updates_Testing for
instructions on how to install test updates.
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-04ed64a2ee
Works fine now, thank you!
libreoffice-188.8.131.52-9.fc25 has been pushed to the Fedora 25 stable repository. If problems still persist, please make note of it in this bug report.