The Red toggle was actually affecting the blue color in some code paths, and vice versa, because the channel indices for ColorBgra are also used to index into the mask.

(cherry picked from commit 9412756)

This is recommended by upstream (https://gitlab.gnome.org/GNOME/gtk/-/issues/7214) and should fix issues with a Vulkan DLL being picked up which doesn't match GTK's requirements

Bug: #1497
(cherry picked from commit 54a53a1)

SetAccelsForAction() is quite expensive (I suspect it needs to go through and update any widgets which might be using the action), so this should be done once up front rather than when rebuilding the menu.

This improves the performance of opening 20 documents at once from 17s to 400ms

Bug: #1574
(cherry picked from commit 776ef8a)

- Don't use WindowActions to change the active document. This is UI code, and the WorkspaceManager is the correct place to do this. Instead, WindowActions should just update its menu according to the workspace's events

- Add an event which is fired before the active document is changed. This allows the tool manager to commit the current tool, decoupling the two

- Use the active_document_index as the signal for whether there are open documents, since the document created event is fired before the document is activated

- Remove the action manager parameter which is no longer needed

(cherry picked from commit b87cfac)

…e open

This improves the performance of opening 50 documents from ~14s to 1.5s

(cherry picked from commit 329e4a8)

This was from the docking layout in Pinta 1.7, and isn't used anymore. (This was removed in the master branch with the canvas rewrite, so there's no need to update this code to work with the changes for tracking the active document)