Desktop: Add type 'info' and 'OK' button to update dialog when version up-to-date

[matsest]

This removes a minor issue (not found Issues) where the text in the check for update messagebox is automatically selected. It adds a 'OK' button and makes the messagebox consistent with other boxes (type 'info'). This is only for the dialog when the version is up-to-date.

• Tested on Linux (Ubuntu 18.04 running under Unity)

Issue

Current behavior when pressing check for update:

Text is automatically selected in lack of a button. ESC must be pressed to exit the window.

Fix

With this PR:

OK is automatically selected, icon is shown. ESC or Enter (for 'OK') can be pressed to exit the window.

[matsest]

Should probably be tested on Mac/Windows as well.

In Windows the dialog already looks like this:

due to

buttons String - Array of texts for buttons. On Windows, an empty array will result in one button labeled "OK". (Electron docs)

[tessus]

On macOS it looks like this:

I don't know, if I want to have an info icon shown instead of the Joplin icon.

[matsest]

I'm not sure.. I guess this use of default values makes this happen for both Mac and Windows in different ways.

As long as you state the type (for the logo) and buttons explicitly, it should anyways look similar on all platforms :) I don't have any strong feelings about using the info icon or Joplin icon for this. If you want the Joplin Icon I guess I can change the type line to using the same icon as in the About Joplin-window:

icon: 'build/icons/32x32.png',

Note: The 'info' type was copied from the dialog when an update is available, so IIRC this dialog also shows an info icon.

[laurent22]

What's the standard for message boxes on Linux? Maybe if the Electron team didn't put the OK button is because it's more normal on Linux not to have one?

In any case, if we make this change, we should probably change showMessageBox so that it applies to all message boxes and not just this one. I prefer not to have icons either, because for some reasons they look quite ugly on all platforms (that's why I wasn't using any).

[matsest]

Based on the documentation for Electron I thought that it automatically added 'OK' to Windows, but it indeed seems like this default action happens for MacOS as well - but not for Linux (as shown in my initial screenshot). There's not a coherent standard for Linux AFAIK, it's likely different in different distros and graphical environments. But not having a button does not look good and does not cohere with the general design in the application IMO.

To suppress the default icon action, adding icon: "none" should be sufficient?

When I took another look at the code I see that using showmessagebox directly is not recommended:

joplin/ElectronClient/app/bridge.js

Lines 65 to 67 in 8ff2418

	// Don't use this directly - call one of the showXxxxxxxMessageBox() instead
	showMessageBox_(window, options) {
	const {dialog} = require('electron');

so perhaps showInfoMessageBox from bridge.js should be used instead?

joplin/ElectronClient/app/bridge.js

Lines 90 to 97 in 8ff2418

	showInfoMessageBox(message, options = {}) {
	const result = this.showMessageBox_(this.window(), Object.assign({}, {
	type: 'info',
	message: message,
	buttons: [_('OK')],
	}, options));
	return result === 0;
	}

It's only in checkForUpdates.js this is used directly, so if changes would to be made on other boxes they should be changed in the definitions in bridge.js instead.

[laurent22]

Thanks @matsest, I think this is fine. I might be difficult to use functions from bridge from that script, so let's keep that way.