DropIt

Lupo73 · **Posted:** Tue Dec 20, 2011 8:09 am

What do you propose? A note in the Guide?

Leif · **Posted:** Tue Dec 20, 2011 9:33 am

[Aside] guinness, please suggest to him that he should fix it instead of e.g. mentioning it in the Guide (who reads guides? :shock:

). In ALL the programs I use it says Del and NEVER Backspace to the right of the Delete option in the menus etc.

Lupo73 · **Posted:** Tue Dec 20, 2011 11:14 am

Ok ok

..I fixed it..now to remove a monitored folder the hotkey is Ctrl+R..

An idea: to make the interface more user-friendly, what do you think if I report the hotkeys in button tooltips? In this way the user can read it simply moving the mouse over a button, without needs to check the Guide..

Leif · **Posted:** Wed Dec 21, 2011 12:20 am

Tooltips are great.

guinness · **Joined:** Mon Aug 27, 2007 2:00 am **Posts:** 3720

I haven't done much research since Ruby pointed out the Registry Keys to see when in Classic Theme Mode, but I came up with this small UDF and it seems to work. You need WinAPIEx.au3 of course >>

Code:

Func _IsClassicTheme() ; By guinness 2011.
   _WinAPI_GetCurrentThemeName()
   Return @error
EndFunc   ;==>_IsClassicTheme

Or how I use it >>

Code:

Func _IsNotClassicTheme() ; By guinness 2011.
   _WinAPI_GetCurrentThemeName()
   Return @error = 0
EndFunc   ;==>_IsClassicTheme

Lupo73 · **Posted:** Thu Dec 22, 2011 6:33 am

Thanks guinness, I'm going to implement it :wink:

A question: does the problem afflict only the main context menu or all menus with images (and tray menu)?

guinness · **Joined:** Mon Aug 27, 2007 2:00 am **Posts:** 3720

As far as I'm aware (without testing) it's only a problem with ContextMenus. Which function are you going to use? Are you going to keep the header i.e. By guinness 2011?

guinness · **Joined:** Mon Aug 27, 2007 2:00 am **Posts:** 3720

A snippet of code from the next ProEject. This is how I'm using the Function I posted above.

Code:

If (_IsWindowsVersion() And _IsNotClassicTheme()) Then ; If Windows is greater than XP and doesn't use Classic Theme Mode.

Lupo73 · **Posted:** Thu Dec 22, 2011 3:06 pm

I kept that header, do you prefer another one?

Thanks about code example.. but I thought to use it adding a space if classic theme is used, why do you use the "if not classic theme"?

guinness · **Joined:** Mon Aug 27, 2007 2:00 am **Posts:** 3720

The space addition doesn't work, treat it as though it was like Windows XP where Context images are disabled. Also I use the NOT for easy reading as well as the NOT version returning True. If I were to use the initial Function I provided I would have to do this >>

Code:

If (_IsWindowsVersion() And _IsClassicTheme() = 0) Then ; Takes a couple of ms longer to parse!

Couple of AutoIt tips: If $sTest Then is quicker than If $sTest = 1 Then...

Everything I do is for a reason, whether it be right or wrong :mrgreen:

Lupo73 · **Posted:** Fri Dec 23, 2011 7:49 am

I'm confused: if the space addition doesn't work, what do you recommend for the theme problem? And does not the tray menu need to be fixed?

guinness · **Joined:** Mon Aug 27, 2007 2:00 am **Posts:** 3720

You know how we check if the user is using Windows XP [_IsWindowsVersion() = 0] it's exactly the same, icons should be disabled (not showed) when using Classic Mode too.

My advice to test by adding whitespace to the start of the string and you'll understand more of why it won't work.

See the separating line between the icon and text, this is the reason why.

Lupo73 · **Posted:** Sat Dec 24, 2011 1:03 am

OK, thanks!

guinness · **Joined:** Mon Aug 27, 2007 2:00 am **Posts:** 3720

Does it make sense though?

Lupo73 · **Posted:** Sat Dec 24, 2011 4:58 am

I could directly modify __SetItemImage() and don't add icons in that case.. do you think is a good solution?

DropIt

Who is online