[8.0ß17a] Add to DirHotlist: Submenu selected, not indicated

[Flint]

1. Make sure there is at least one submenu in Directory Hotlist present.
2. Open any dir not present in DirHotlist, press Ctrl+D, select "Add current dir".
3. In the dialog click the checkbox "Add to submenu". When context menu appears, do not select any item but simply press Esc or click anywhere outside the menu (except for the checkbox "Add to submenu").
4. The context menu is closed, no submenu is selected, the checkbox title still says "Add to submenu" without the submenu name. The checkbox itself remains checked however!
5. Click OK -> the new directory will be added to the submenu.

I think, if the context menu is closed without selecting an item, the checkbox should clear automatically, and after clicking OK the item should be added to the root of DirHotlist.

[ghisler(Author)]

Yes, TC is adding the file to the first submenu if the option is checked, but no submenu is chosen. I can change it if you think that this is confusing.

[ghisler(Author)]

I have changed this now in beta 18, please test it!

[umbra]

The checkbox stays checked, even if no submenu is selected.

[Flint]

I have changed this now in beta 18, please test it!

I confirm that if I don't select a submenu, the new item is created in the root, thanks. However, the checkbox remained checked, and it's confusing a bit.

(Actually, when I installed beta 18 I tried this, but when I saw that the checkbox remained checked I thought that the fix was postponed (especially since it was not mentioned in history), so I just pressed Cancel.)

[ghisler(Author)]

Unfortunately there is no notification when the menu is closed without choosing anything, so I have leave it checked.

[Flint]

ghisler(Author)
Hm… How do you show the menu? Do you not use TrackPopupMenu? It allows to distinguish these situations:

Return value

If you specify TPM_RETURNCMD in the uFlags parameter, the return value is the menu-item identifier of the item that the user selected. If the user cancels the menu without making a selection, or if an error occurs, the return value is zero.

[petermad]

Hasn't this thread ended up in the wrong forum section (TC7.56a confirmed bugs) ?

[petermad]

* Bump - this thread still doesn't belong in this section.

[Hacker]

[mod]Moved to Bugs which should be fixed now for lack of better ideas.

Hacker (Moderator)[/mod]

[ghisler(Author)]

The original bug has been fixed, so I'm moving this to fixed bugs. The checkbox behaviour will not be changed.