[shell32] Load file context menu from resources (try 2)
Tobias Burnus
burnus at net-b.de
Mon Jun 20 20:11:02 CDT 2005
Hello,
Alexandre Julliard wrote:
>> Second try (unchanged). Any suggestions how to improve the patch are
>> welcome.
>>
> You shouldn't comment things out like that. Either the info is needed
> and it should be here, or it's not needed and it should be removed
> completely. Commenting it out is only creating confusion.
>
Well, I actually don't know what is supposed to be there; I saw quite
different menus on the different Windows versions. Some seem to have the
"Properties" and "Create Link" (and other stuff more) and others have
not. I will now now blindly assume that that what is in the .c file is
the right thing (TM) and will delete the rest.
Change to previous version (try 1):
- Remove instead of comment out unused menu items in .rc
- shell32_Ja.rc: Copy "Open" string from IDS_OPEN to MENUITEMS
("Exporer" is still missing, but there is no string resource which I can
copy.)
Changelog:
Tobias Burnus <burnus_at_net-b.de <mailto:burnus_at_net-b.de?Subject=Re:%20%5Bshell32%5D%20Load%20file%20context%20menu%20from%20resources%20%5BResent%5D>>
Use resource files for open file dialogs' file context menu
Try 1 was: http://www.winehq.org/hypermail/wine-patches/2005/06/0499.html
Tobias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: shell32-patch2.diff
Type: text/x-patch
Size: 13089 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20050621/57f16c60/shell32-patch2.bin
More information about the wine-patches
mailing list