Divorce the Edit and Show Tags/Comment menu items

In WinBoard the items in the Edit and View menu for Tags and Comment now
have different IDs, so they can be independently controlled. Only the
items in the View menu are check-marked. The items still do the same.

diff --git a/winboard/resource.h b/winboard/resource.h
index 15835f5..cee0fec 100644 (file)
--- a/winboard/resource.h
+++ b/winboard/resource.h
@@ -51,6 +51,8 @@
 #define IDM_AboutGame                   167\r
 #define IDM_ShowGameList                170\r
 #define IDM_EditTags                    171\r
+#define IDM_Tags                        172\r
+#define IDM_Comment                     173\r
 #define IDM_Debug                       174\r
 #define IDM_AnalysisMode                177\r
 #define IDM_AnalyzeFile                 178\r

diff --git a/winboard/wedittags.c b/winboard/wedittags.c
index e0ea7ae..0cf3ca3 100644 (file)
--- a/winboard/wedittags.c
+++ b/winboard/wedittags.c
@@ -171,7 +171,7 @@ EditTagsDialog(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam)
 VOID TagsPopDown(void)\r
 {\r
   if (editTagsDialog) ShowWindow(editTagsDialog, SW_HIDE);\r
-  CheckMenuItem(GetMenu(hwndMain), IDM_EditTags, MF_UNCHECKED);\r
+  CheckMenuItem(GetMenu(hwndMain), IDM_Tags, MF_UNCHECKED);\r
   editTagsUp = FALSE;\r
 }\r
 \r
@@ -201,7 +201,7 @@ VOID EitherTagsPopUp(char *tags, char *msg, BOOLEAN edit)
   editTagsText = p;\r
   canEditTags = edit;\r
   \r
-  CheckMenuItem(GetMenu(hwndMain), IDM_EditTags, MF_CHECKED);\r
+  CheckMenuItem(GetMenu(hwndMain), IDM_Tags, MF_CHECKED);\r
   if (editTagsDialog) {\r
     SendMessage(editTagsDialog, WM_INITDIALOG, 0, 0);\r
     ShowWindow(editTagsDialog, SW_SHOW);\r

diff --git a/winboard/winboard.c b/winboard/winboard.c
index f05e680..a0e2284 100644 (file)
--- a/winboard/winboard.c
+++ b/winboard/winboard.c
@@ -4920,10 +4920,12 @@ WndProc(HWND hwnd, UINT message, WPARAM wParam, LPARAM lParam)
       break;\r
 \r
     case IDM_EditTags:\r
+    case IDM_Tags:\r
       EditTagsProc();\r
       break;\r
 \r
     case IDM_EditComment:\r
+    case IDM_Comment:\r
       if (commentUp && editComment) {\r
        CommentPopDown();\r
       } else {\r
@@ -6323,7 +6325,7 @@ EitherCommentPopUp(int index, char *title, char *str, BOOLEAN edit)
   FARPROC lpProc;\r
   char *p, *q;\r
 \r
-  CheckMenuItem(GetMenu(hwndMain), IDM_EditComment, edit ? MF_CHECKED : MF_UNCHECKED);\r
+  CheckMenuItem(GetMenu(hwndMain), IDM_Comment, edit ? MF_CHECKED : MF_UNCHECKED);\r
 \r
   if (str == NULL) str = "";\r
   p = (char *) malloc(2 * strlen(str) + 2);\r
@@ -7809,6 +7811,7 @@ Enables ncpEnables[] = {
 \r
 Enables trainingOnEnables[] = {\r
   { IDM_EditComment, MF_BYCOMMAND|MF_GRAYED },\r
+  { IDM_Comment, MF_BYCOMMAND|MF_GRAYED },\r
   { IDM_Pause, MF_BYCOMMAND|MF_GRAYED },\r
   { IDM_Forward, MF_BYCOMMAND|MF_GRAYED },\r
   { IDM_Backward, MF_BYCOMMAND|MF_GRAYED },\r
@@ -7821,6 +7824,7 @@ Enables trainingOnEnables[] = {
 \r
 Enables trainingOffEnables[] = {\r
   { IDM_EditComment, MF_BYCOMMAND|MF_ENABLED },\r
+  { IDM_Comment, MF_BYCOMMAND|MF_ENABLED },\r
   { IDM_Pause, MF_BYCOMMAND|MF_ENABLED },\r
   { IDM_Forward, MF_BYCOMMAND|MF_ENABLED },\r
   { IDM_Backward, MF_BYCOMMAND|MF_ENABLED },\r
@@ -8518,7 +8522,7 @@ CommentPopUp(char *title, char *str)
 VOID\r
 CommentPopDown(void)\r
 {\r
-  CheckMenuItem(GetMenu(hwndMain), IDM_EditComment, MF_UNCHECKED);\r
+  CheckMenuItem(GetMenu(hwndMain), IDM_Comment, MF_UNCHECKED);\r
   if (commentDialog) {\r
     ShowWindow(commentDialog, SW_HIDE);\r
   }\r

diff --git a/winboard/winboard.rc b/winboard/winboard.rc
index 770f58d..235f622 100644 (file)
--- a/winboard/winboard.rc
+++ b/winboard/winboard.rc
@@ -1159,8 +1159,8 @@ BEGIN
         MENUITEM "Open &New Chat Window",       IDM_NewChat\r
         MENUITEM "Type In &Move...\tAlt+I",     IDM_TypeInMove\r
         MENUITEM SEPARATOR\r
-        MENUITEM "&Tags",                       IDM_EditTags\r
-        MENUITEM "&Comments",                   IDM_EditComment\r
+        MENUITEM "&Tags",                       IDM_Tags\r
+        MENUITEM "&Comments",                   IDM_Comment\r
         MENUITEM SEPARATOR\r
         MENUITEM "&Game List Tags...",          IDM_GameListOptions\r
         MENUITEM "&Board...",                   IDM_BoardOptions\r