Re: [Freeciv-Dev] (PR#39913) consistent UI - Government should be Edit
http://bugs.freeciv.org/Ticket/Display.html?id=39913 >
Committed as part of S2_2 revision 14101.
Committed as part of trunk revision 14102.
Index: client/gui-gtk-2.0/gui_main.c
===
--- client/gui-gtk-2.0/gui_main.c (revision 14100)
+++ client/gui-gtk-2.0/gui_main.c (working copy)
@@ -457,7 +457,6 @@
return FALSE;
}
-
if ((ev->state & GDK_SHIFT_MASK)) {
switch (ev->keyval) {
case GDK_Left:
@@ -498,7 +497,24 @@
default:
break;
}
+ } else {
+switch (ev->keyval) {
+
+case GDK_apostrophe:
+ /* FIXME: should find the correct window, even when detached, from any
+ * other window; should scroll to the bottom automatically showing the
+ * latest text from other players; MUST NOT make spurious text windows
+ * at the bottom of other dialogs.
+ */
+ gtk_notebook_set_current_page(GTK_NOTEBOOK(bottom_notebook), 0);
+ gtk_widget_grab_focus(inputline);
+ return TRUE;
+
+default:
+ break;
+};
}
+
/* Return here if observer */
if (client_is_observer()) {
return FALSE;
@@ -1243,7 +1259,7 @@
gtk_widget_set_size_request(sw, 600, 100);
gtk_box_pack_start(GTK_BOX(vbox), sw, TRUE, TRUE, 0);
- label = gtk_label_new_with_mnemonic(_("_Chat"));
+ label = gtk_label_new(_("Chat"));
gtk_notebook_append_page(GTK_NOTEBOOK(bottom_notebook), vbox, label);
text = gtk_text_view_new_with_buffer(message_buffer);
Index: client/gui-gtk-2.0/messagewin.c
===
--- client/gui-gtk-2.0/messagewin.c (revision 14100)
+++ client/gui-gtk-2.0/messagewin.c (working copy)
@@ -187,7 +187,7 @@
gtk_widget_set_sensitive(cmd, FALSE);
cmd = gui_dialog_add_stockbutton(meswin_shell, GTK_STOCK_ZOOM_IN,
- _("_Update City"), CMD_POPCITY);
+ _("Inspect _City"), CMD_POPCITY);
gtk_widget_set_sensitive(cmd, FALSE);
gui_dialog_add_button(meswin_shell, GTK_STOCK_CLOSE, GTK_RESPONSE_CLOSE);
Index: client/gui-gtk-2.0/menu.c
===
--- client/gui-gtk-2.0/menu.c (revision 14100)
+++ client/gui-gtk-2.0/menu.c (working copy)
@@ -727,38 +727,44 @@
NULL, 0,
"" },
{ "/" N_("Game") "/tearoff1",NULL,
NULL, 0,
"" },
- { "/" N_("Game") "/" N_("Local _Options"), NULL,
+ { "/" N_("Game") "/" N_("_Chat"),NULL,
+ NULL, 0,
"" },
+ { "/" N_("Game") "/" N_("_Chat") "/" N_("_Clear Log"), NULL,
+ game_menu_callback, MENU_GAME_CLEAR_OUTPUT
},
+ { "/" N_("Game") "/" N_("_Chat") "/" N_("_Write Log"), NULL,
+ game_menu_callback, MENU_GAME_OUTPUT_LOG
},
+ { "/" N_("Game") "/" N_("_Government"), NULL,
+ NULL, 0,
"" },
+ { "/" N_("Game") "/" N_("_Government") "/" N_("_Tax Rates"),
"t",
+ government_menu_callback, MENU_GOVERNMENT_TAX_RATE
},
+ { "/" N_("Game") "/" N_("_Government") "/" N_("_Revolution..."),
"r",
+ government_menu_callback, MENU_GOVERNMENT_REVOLUTION
},
+ { "/" N_("Game") "/" N_("_Government") "/sep1", NULL,
+ NULL, 0,
"" },
+ { "/" N_("Game") "/" N_("_Options"), NULL,
+ NULL, 0,
"" },
+ { "/" N_("Game") "/" N_("_Options") "/" N_("_Local Client"), NULL,
game_menu_callback, MENU_GAME_OPTIONS
},
- { "/" N_("Game") "/" N_("_Message Options"), NULL,
+ { "/" N_("Game") "/" N_("_Options") "/" N_("_Message"), NULL,
game_menu_callback, MENU_GAME_MSG_OPTIONS
},
- { "/" N_("Game") "/sep1",NULL,
+ { "/" N_("Game") "/" N_("_Options") "/" N_("_Remote Server"),NULL,
+ game_menu_callback, MENU_GAME_SERVER_OPTIONS},
+ { "/" N_("Game") "/" N_("_Options") "/" N_("Save Options _Now"),
NULL,
+ game_menu_callback, MENU_GAME_SAVE_OPTIONS
},
+ { "/" N_("Game") "/" N_("_Options") "/" N_("Save Options on _Exit"), NULL,
+ game_menu_callback, MENU_GAME_SAVE_OPTIONS_ON_EXIT,
"" },
+ { "/" N_("Game") "/sep4",NULL,
NULL, 0,
"" },
#ifdef DEBUG
{ "/" N_("Game") "/" N_("_Reload Tileset"), "r",
game_menu_cal
Re: [Freeciv-Dev] (PR#39913) consistent UI - Government should be Edit
http://bugs.freeciv.org/Ticket/Display.html?id=39913 >
William Allen Simpson wrote:
> I will change Gover_nment to Gov_ernment, so gui-gtk2 matches gui-win32,
> to eliminate the conflict with other _n's. No po changes.
>
Committed S2_1 revision 14083.
Index: client/gui-gtk-2.0/menu.c
===
--- client/gui-gtk-2.0/menu.c (revision 14082)
+++ client/gui-gtk-2.0/menu.c (working copy)
@@ -726,17 +726,17 @@
game_menu_callback, MENU_GAME_QUIT,
"",
GTK_STOCK_QUIT
},
/* Government menu ... */
- { "/" N_("Gover_nment"), NULL,
+ { "/" N_("Gov_ernment"), NULL,
NULL, 0,
"" },
{ "/" N_("Government") "/tearoff1", NULL,
NULL, 0,
"" },
- { "/" N_("Government") "/" N_("_Tax Rates"), "t",
+ { "/" N_("Government") "/" N_("Tax Rates"), "t",
government_menu_callback, MENU_GOVERNMENT_TAX_RATE
},
{ "/" N_("Government") "/sep1", NULL,
NULL, 0,
"" },
- { "/" N_("Government") "/" N_("_Find City"), "f",
+ { "/" N_("Government") "/" N_("Find City"), "f",
government_menu_callback, MENU_GOVERNMENT_FIND_CITY
},
- { "/" N_("Government") "/" N_("_Worklists"), "l",
+ { "/" N_("Government") "/" N_("Worklists"), "l",
government_menu_callback, MENU_GOVERNMENT_WORKLISTS
},
{ "/" N_("Government") "/sep2", NULL,
NULL, 0,
"" },
Index: client/gui-win32/menu.c
===
--- client/gui-win32/menu.c (revision 14082)
+++ client/gui-win32/menu.c (working copy)
@@ -321,8 +321,8 @@
{N_("Gov_ernment"), IDM_SUBMENU},
{N_("Tax Rates") "\tShift+T",IDM_GOVERNMENT_TAX_RATE},
{ "", IDM_SEPARATOR},
- {N_("_Find City")"\tCtl+F", IDM_GOVERNMENT_FIND_CITY},
- {N_("Work_lists")"\tCtl+W", IDM_GOVERNMENT_WORKLISTS},
+ {N_("Find City") "\tCtl+F", IDM_GOVERNMENT_FIND_CITY},
+ {N_("Worklists") "\tCtl+W", IDM_GOVERNMENT_WORKLISTS},
{ "", IDM_SEPARATOR},
{N_("_Change Government"), IDM_SUBMENU},
{N_("_Revolution"), IDM_GOVERNMENT_REVOLUTION},
___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
Re: [Freeciv-Dev] (PR#39913) consistent UI - Government should be Edit
http://bugs.freeciv.org/Ticket/Display.html?id=39913 > Daniel Markstedt wrote: > How about something like Council or Throne Room or Assembly etc. Please > give suggestions! > Civ2 used Kingdom. Anyway, it's a bad idea. Everything related to editing should be under the Edit menu. > Please don't change this in 2.1. OK. I've already written the patch for 2.1, but svn revert works I will change Gover_nment to Gov_ernment, so gui-gtk2 matches gui-win32, to eliminate the conflict with other _n's. No po changes. > Do you mean Revolution? >... > How about putting the Editor functions under Game? No, let's put Revolution under Game, instead. Of course, as I mentioned earlier, it really needs to be activated by clicking on the current government icon in the pane, just like changing tax rates. But they are supposed to have a parallel menu item, too. ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] (PR#39913) consistent UI - Government should be Edit
http://bugs.freeciv.org/Ticket/Display.html?id=39913 > > [wsimpson - Mon Nov 26 13:20:04 2007]: > > As noted elsewhere, the second item in the main menu bar should be > _Edit. > However, as noted yet elsewhere, menu and button labels should preferably be analogies of game world concepts. To "edit" is something I associate with publishing, not the running of a country. We've been trying to move away from the 'Freeciv-looks-like-a-spreadsheet- app' image for a long time now. This would IMO be a move in the wrong direction. How about something like Council or Throne Room or Assembly etc. Please give suggestions! > At some time the the not-too-distant past, somebody changed _Edit to > Gover_nment in GTK2, or Gov_ernment in win32, and who knows what else. > > This is bad from a UI standpoint, or from Human Interface Guidelines. > It > conflicts with several tabs and other screens. > > _Government, G_overnment, Go_vernment, Gove_rnment, Gover_nment, > Govern_ment, > and Governmen_t all conflict. This was a very poor choice for the > main menu. > This is certainly a problem. On the other hand, picking a label shouldn't IMO be based on the keyboard shortcut that could be made for it, but rather the clarity and accuracy of the string. In the worst case, there are always solutions like Government (_X) or whatever. > The _Edit msgid is still present in most 2.1 po file's #~ sections, > and will > be relatively easy to change back. > Please don't change this in 2.1. I wouldn't want such a high-visibility string to change with a maintenance release. It would cause confusion that is out of proportion to the issue it's solving. > There is only one item in the menu that has anything to do with > Government > or changing government, and it has its own key! I'd put it in next to > the > Government icon in the main panel, instead. > Do you mean Revolution? > We should combine the rest with Editor menu items in 2.2. > > > How about putting the Editor functions under Game? Toggle editor mode and Show editor panel could be kept at the top level while the rest could be relegated to a submenu. It wouldn't add too much complexity to the menu IMO. ~Daniel ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
Re: [Freeciv-Dev] (PR#39913) consistent UI - Government should be Edit
http://bugs.freeciv.org/Ticket/Display.html?id=39913 >
This also fixes the bug that the editor main menu was improperly disabled
sometimes. HIG [page 43] explicitly says:
Do not disable menu titles. Allow the user to explore the menu, even
though there might be no available items on it at that time.
With the Editing Mode under Edit, Editing Mode can be disabled instead.
The items below it are disabled until Editing Mode is enabled (as now).
Attempted to fix gui-win32 in passing (without testing), but no others.
Index: client/gui-gtk-2.0/menu.c
===
--- client/gui-gtk-2.0/menu.c (revision 14078)
+++ client/gui-gtk-2.0/menu.c (working copy)
@@ -766,28 +766,39 @@
{ "/" N_("Game") "/" N_("_Quit"),NULL,
game_menu_callback, MENU_GAME_QUIT,
"",
GTK_STOCK_QUIT
},
- /* Government menu ... */
- { "/" N_("Gover_nment"), NULL,
+ /* was Government menu ... */
+ { "/" N_("_Edit"), NULL,
NULL, 0,
"" },
- { "/" N_("Government") "/tearoff1", NULL,
+ { "/" N_("_Edit") "/tearoff1", NULL,
NULL, 0,
"" },
- { "/" N_("Government") "/" N_("_Tax Rates"), "t",
+ { "/" N_("_Edit") "/" N_("Tax Rates"), "t",
government_menu_callback, MENU_GOVERNMENT_TAX_RATE
},
- { "/" N_("Government") "/sep1", NULL,
+ { "/" N_("_Edit") "/sep1", NULL,
NULL, 0,
"" },
- { "/" N_("Government") "/" N_("_Find City"), "f",
+ { "/" N_("_Edit") "/" N_("Find City"), "f",
government_menu_callback, MENU_GOVERNMENT_FIND_CITY
},
- { "/" N_("Government") "/" N_("_Worklists"), "l",
+ { "/" N_("_Edit") "/" N_("Worklists"), "l",
government_menu_callback, MENU_GOVERNMENT_WORKLISTS
},
- { "/" N_("Government") "/sep2", NULL,
+ { "/" N_("_Edit") "/sep2", NULL,
NULL, 0,
"" },
- { "/" N_("Government") "/" N_("_Change Government"), NULL,
+ { "/" N_("_Edit") "/" N_("_Government"), NULL,
NULL, 0,
"" },
- { "/" N_("Government") "/" N_("_Change Government") "/" N_("_Revolution..."),
+ { "/" N_("_Edit") "/" N_("_Government") "/" N_("Revolution..."),
"r",
government_menu_callback, MENU_GOVERNMENT_REVOLUTION
},
- { "/" N_("_Government") "/" N_("_Change Government") "/sep1", NULL,
+ { "/" N_("_Edit") "/" N_("_Government") "/sep1", NULL,
NULL, 0,
"" },
+ /* was Editor menu */
+ { "/" N_("_Edit") "/tearoff2", NULL, NULL, 0, "" },
+ { "/" N_("_Edit") "/" N_("Editing _Mode"), NULL,
+editor_menu_callback, MENU_EDITOR_TOGGLE, "" },
+ { "/" N_("_Edit") "/" N_("Editing _Tools"), NULL,
+editor_menu_callback, MENU_EDITOR_TOOLS },
+ { "/" N_("_Edit") "/" N_("Recalculate _Borders"), NULL,
+editor_menu_callback, MENU_EDITOR_RECALCULATE_BORDERS },
+ { "/" N_("_Edit") "/" N_("Regenerate _Water"), NULL,
+editor_menu_callback, MENU_EDITOR_REGENERATE_WATER },
+
/* View menu ... */
{ "/" N_("_View"), NULL,
NULL, 0,
"" },
@@ -955,18 +966,6 @@
{ "/" N_("Reports") "/" N_("S_paceship"),"F12",
reports_menu_callback, MENU_REPORT_SPACESHIP
},
- /* Editor menu */
- { "/" N_("_Editor"), NULL, NULL, 0, "" },
- { "/" N_("_Editor") "/tearoff1", NULL, NULL, 0, "" },
- { "/" N_("_Editor") "/" N_("_Editing Mode"), NULL,
-editor_menu_callback, MENU_EDITOR_TOGGLE, "" },
- { "/" N_("_Editor") "/" N_("_Tools"), NULL,
-editor_menu_callback, MENU_EDITOR_TOOLS },
- { "/" N_("_Editor") "/" N_("Recalculate _Borders"), NULL,
-editor_menu_callback, MENU_EDITOR_RECALCULATE_BORDERS },
- { "/" N_("_Editor") "/" N_("Regenerate _Water"), NULL,
-editor_menu_callback, MENU_EDITOR_REGENERATE_WATER },
-
/* Help menu ... */
{ "/" N_("_Help"), NULL,
NULL, 0,
"" },
@@ -1268,14 +1267,13 @@
menus_set_sensitive("/_Game/L_eave", aconnection.established);
if (!can_client_change_view()) {
[Freeciv-Dev] (PR#39913) consistent UI - Government should be Edit
http://bugs.freeciv.org/Ticket/Display.html?id=39913 > As noted elsewhere, the second item in the main menu bar should be _Edit. At some time the the not-too-distant past, somebody changed _Edit to Gover_nment in GTK2, or Gov_ernment in win32, and who knows what else. This is bad from a UI standpoint, or from Human Interface Guidelines. It conflicts with several tabs and other screens. _Government, G_overnment, Go_vernment, Gove_rnment, Gover_nment, Govern_ment, and Governmen_t all conflict. This was a very poor choice for the main menu. The _Edit msgid is still present in most 2.1 po file's #~ sections, and will be relatively easy to change back. There is only one item in the menu that has anything to do with Government or changing government, and it has its own key! I'd put it in next to the Government icon in the main panel, instead. We should combine the rest with Editor menu items in 2.2. ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
