Linux: Debian package works (unlike AppImage), but no top-level menu unless reader view open

Question

Linux: Debian package works (unlike AppImage), but no top-level menu unless reader view open

danielweck opened this issue 6 years ago · comments

This in itself is not a big deal, but what is more annoying is that the small "open publication" dialog does not have a "exit" option, so it requires CTRL-C to shutdown!!

Download r2-testapp-js_1.0.6-0_amd64.deb from https://github.com/readium/r2-testapp-js/releases/tag/latest-linux
sudo apt-get install ./r2-testapp-js_1.0.6-0_amd64.deb
r2-testapp-js

Daniel Weck · Answer 1 · Thu Jan 17 2019 02:35:54 GMT+0800 (China Standard Time)

Related issue: #17

Daniel Weck · Answer 2 · Thu Jan 17 2019 02:57:54 GMT+0800 (China Standard Time)

Same problem in Windows.

Daniel Weck · Answer 3 · Thu Jan 17 2019 03:31:27 GMT+0800 (China Standard Time)

Fixed by adding the missing top-level menu, and more importantly by adding a confirmation dialog to exit the app (same logic on MacOS, Linux and Windows to make things simple, although conventionally there is a different behaviour on MacOS because of the persistent window-less top-level menu ... but this is fine for a test app)
55a341d