[shell32] Load file context menu from resources [Resent]

Tobias Burnus burnus at net-b.de
Sun Jun 19 06:35:49 CDT 2005


Second try (unchanged). Any suggestions how to improve the patch are 
welcome.

Tobias

Changed file overview:
- shv_item_cmenu.c: Load from resources (the [currently unused] MENU 
resource already exists)
- shell32_?.rc: Comment out two items (and a separator) and
copy text from IDS_SELECT, needed to match the current context menu
("Select" present, "Properties" and "Create Link" are not)

  shell32_Cs.rc    |    7 ++---
  shell32_De.rc    |   14 +++++-----
  shell32_En.rc    |    7 ++---
  shell32_Es.rc    |    7 ++---
  shell32_Fr.rc    |    7 ++---
  shell32_It.rc    |    9 +++---
  shell32_Ja.rc    |    6 ++--
  shell32_Pl.rc    |    7 ++---
  shell32_Pt.rc    |    7 ++---
  shell32_Uk.rc    |    1
  shv_item_cmenu.c |   75 
+++++++++++++++++++++++++++++++------------------------
  11 files changed, 84 insertions(+), 63 deletions(-)

-------------------

Hello,

this is patch converts ISvItemCm_fnQueryContextMenu of
shell32/shv_item_cmenu.c to use the already existing
MENU_SHV_FILE resource. I used the items of ISvItemCm_fnQueryContextMenu
which means I commented out
the "Properties" and the "Create Link" menu item and added an "Select"
item (I could copy the string from IDS_SELECT).
The Japanese menu will not completely work since some menu items are
missing.
If I understand the current implementation correct, one sometimes gets
"Open" and "Select" menu items which both have the same ID; I don't know
whether this is correct, but this is the current implementation. (Since
they use the same ID, I cannot use MF_BYCOMMAND in swapping Open/Explore
and in conditionally removing Select, which makes the Japanese menu even
more wrong.)

Review note: Some of the logic (LoadMenu, Merging menus) is taken from
shv_bg_cmenu.c's ISVBgCm_fnQueryContextMenu.

Changelog:
      Tobias Burnus <burnus at net-b.de>
      Use resource files for open file dialogs' file context menu

Tobias


-------------- next part --------------
A non-text attachment was scrubbed...
Name: shell32-patch.diff
Type: text/x-patch
Size: 14252 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20050619/9bd55f71/shell32-patch.bin


More information about the wine-patches mailing list