As explained in Slack. The revealer was filling the screen and stealing mouse events from the canvas underneath.
Can test by zooming into a photo in fullscreen mode and attempting to pan by dragging with and without this branch.
Juno possibly seems unaffected.
This fixes the root cause of the crash in Vala 0.39.
For some reason the base constructor isn't setting those properties properly on the base class despite having the code there to do so. So do that in the child classes too.
There are two versions of the `_fetch` method for fetching a thumbnail from the cache. The async version generates the thumbnail if it doesn't exist and then returns it.
The synchronous method doesn't. In my mind, the synchronous and asynchronous methods should do the same thing...
Removes an assertion that doesn't make sense; checking a tag exists before creating it. Say what?
Also cleans up some errors about trying to bind null strings to database queries that I noticed while testing this.
You can add a hierarchical tag by adding a tag to a photo in th...
There were quite a lot of errors in the console relating to the metadata sidebar when opening a photo directly in the viewer (i.e. via the file browser). This is because the sidebar is only available in the library mode and the cast to `LibraryWindow` was failing and producing a null pointer.
The SQLite library was throwing assertions about trying to bind null pointers to strings in database queries. This should fix the ones that occur on importing new photos by simply using empty strings instead of `null`.
Should be able to test this by running from the console and ...
Also removes two settings keys that weren't settable anywhere other than DConf editor that allowed the user to specify a custom UNIX time format string that the folders their imports are put into is named with.