Show better instructions to IE9+ users

src/plugin/PluginClass.cpp

Index: src/plugin/PluginClass.cpp
===================================================================
--- a/src/plugin/PluginClass.cpp
+++ b/src/plugin/PluginClass.cpp
@@ -574,8 +574,11 @@
           // Do we have enough rights to enable a status bar?
           if (regRes != 0)
           {
+            LPCWSTR browserVersion = CPluginSystem::GetInstance()->GetBrowserVersion();
+            int versionNumber = wcstol(browserVersion, 0, 10);

[Felix Dahlke, 2013/06/07 06:28:24]

@Oleksandr:
Can I rely on this being an integer? It's "9" for me (although I have IE10), can
this be a non-integer (e.g. a full semantic version string) in other IE
versions?

Is there a better way to check if we're IE9 or newer?

[Oleksandr, 2013/06/07 07:16:46]

This is guaranteed to be either 1 number or an empty string. Not a full semantic
version. 

You get version 9 for IE10, since we lack a special handling for IE10. (The
registry value should be srcVersion there
http://support.microsoft.com/kb/969393/en-us). Other then that you should have a
number here with a correct IE version.

I think this is the best way to check of we're on IE9 or later.

That being said, I don't think the menu bar was only hidden in IE9. IE8 had it
hidden by default as well. In IE7 it can be hidden as well, in fact. Not 100%
sure about IE6, but I think you can hide it there as well (not by default
though). 


In other words, I don't think we need a version check here at all. Just modify
the message for all versions - it should work.
On 2013/06/07 06:28:24, Felix H. Dahlke wrote:

[Felix Dahlke, 2013/06/07 08:02:47]

Hm, I thought it was shown by default in IE8. But I had to google around for
screenshots to figure that out, I wouldn't bet on that :)

The more important question is: Will right clicking the title bar always work? I
can only check with IE6 and IE10, but in IE6 at least it doesn't work.

[Wladimir Palant, 2013/06/07 08:48:29]

I looked through the IE versions:

* Custom title bar context menu was introduced in IE9, before that you would
right-click the menu bar to get that menu.
* The title/menu bar context menu contains a "Status Bar" entry in all IE
versions but IE6.
* IE6-8 have both menu and status bar displayed by default.
* The important menu entry is View / Status Bar in IE6 and IE7. In IE8 and IE9
it was moved to View / Toolbars / Status Bar (presumably still there in IE10 but
I don't have it).

We shouldn't worry about people hiding the menu bar if it isn't hidden by
default - people who do that will be able to get it back. Also, we shouldn't
tell people to use the menu if the menu is hidden by default.

IMHO we can have a single message: "Right-click the menu bar/title bar of the
Internet Explorer window and enable the status bar." It will work for all
versions with the exception of IE6. Given that pretty much all IE6 users have
the status bar enabled anyway, it should be acceptable to not display any hint
there.

Side-note: You should fix the GetBrowserVersion() function to return a number
rather than convert the string in the caller.

+            LPCSTR messageKey = versionNumber >= 9 ? "ERROR_CAN_NOT_ENABLE_STATUS_BAR_IE9" : "ERROR_CAN_NOT_ENABLE_STATUS_BAR";
             // We use the tab window here and in the next few calls, since the browser window may still not be available
-            LRESULT res = MessageBox((HWND)m_hTabWnd, dictionary->Lookup("ERROR_CAN_NOT_ENABLE_STATUS_BAR"),
+            LRESULT res = MessageBox((HWND)m_hTabWnd, dictionary->Lookup(messageKey),
               dictionary->Lookup("ERROR_CAN_NOT_ENABLE_STATUS_BAR_TITLE"), MB_OK);
             return;
           }