X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=src%2Fxo-callbacks.c;h=50672f6b14737a2edbf462266894798a12ffbd1d;hb=9f09269d8918dfa930543f4a15de4e7276719e5e;hp=7c520229fbbd68c8137da2525b4d139069569eb5;hpb=881e9a45693f4d724512fdc5297ba312d447851b;p=xournal.git diff --git a/src/xo-callbacks.c b/src/xo-callbacks.c index 7c52022..50672f6 100644 --- a/src/xo-callbacks.c +++ b/src/xo-callbacks.c @@ -1,3 +1,19 @@ +/* + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public + * License as published by the Free Software Foundation; either + * version 2 of the License, or (at your option) any later version. + * + * This software is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . + */ + + #ifdef HAVE_CONFIG_H # include #endif @@ -17,6 +33,7 @@ #include "xo-misc.h" #include "xo-file.h" #include "xo-paint.h" +#include "xo-selection.h" #include "xo-print.h" #include "xo-shapes.h" @@ -25,7 +42,6 @@ on_fileNew_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); if (close_journal()) { new_journal(); ui.zoom = ui.startup_zoom; @@ -47,7 +63,6 @@ on_fileNewBackground_activate (GtkMenuItem *menuitem, gboolean success; end_text(); - reset_focus(); if (!ok_to_close()) return; // user aborted on save confirmation dialog = gtk_file_chooser_dialog_new(_("Open PDF"), GTK_WINDOW (winMain), @@ -121,7 +136,6 @@ on_fileOpen_activate (GtkMenuItem *menuitem, gboolean success; end_text(); - reset_focus(); if (!ok_to_close()) return; // user aborted on save confirmation dialog = gtk_file_chooser_dialog_new(_("Open Journal"), GTK_WINDOW (winMain), @@ -171,7 +185,6 @@ on_fileSave_activate (GtkMenuItem *menuitem, GtkWidget *dialog; end_text(); - reset_focus(); if (ui.filename == NULL) { on_fileSaveAs_activate(menuitem, user_data); return; @@ -204,7 +217,6 @@ on_fileSaveAs_activate (GtkMenuItem *menuitem, struct stat stat_buf; end_text(); - reset_focus(); dialog = gtk_file_chooser_dialog_new(_("Save Journal"), GTK_WINDOW (winMain), GTK_FILE_CHOOSER_ACTION_SAVE, GTK_STOCK_CANCEL, GTK_RESPONSE_CANCEL, GTK_STOCK_SAVE, GTK_RESPONSE_OK, NULL); @@ -226,7 +238,7 @@ on_fileSaveAs_activate (GtkMenuItem *menuitem, } else { curtime = time(NULL); - strftime(stime, 30, "%F-Note-%H-%M.xoj", localtime(&curtime)); + strftime(stime, 30, "%Y-%m-%d-Note-%H-%M.xoj", localtime(&curtime)); if (ui.default_path!=NULL) gtk_file_chooser_set_current_folder(GTK_FILE_CHOOSER (dialog), ui.default_path); gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER (dialog), stime); @@ -308,7 +320,6 @@ on_filePrint_activate (GtkMenuItem *menuitem, char *in_fn, *p; end_text(); - reset_focus(); if (!gtk_check_version(2, 10, 0)) { print = gtk_print_operation_new(); /* @@ -362,7 +373,6 @@ on_filePrintPDF_activate (GtkMenuItem *menuitem, gboolean warn; end_text(); - reset_focus(); dialog = gtk_file_chooser_dialog_new(_("Export to PDF"), GTK_WINDOW (winMain), GTK_FILE_CHOOSER_ACTION_SAVE, GTK_STOCK_CANCEL, GTK_RESPONSE_CANCEL, GTK_STOCK_SAVE, GTK_RESPONSE_OK, NULL); @@ -381,7 +391,7 @@ on_filePrintPDF_activate (GtkMenuItem *menuitem, gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER (dialog), g_basename(in_fn)); } else { curtime = time(NULL); - strftime(stime, 30, "%F-Note-%H-%M.pdf", localtime(&curtime)); + strftime(stime, 30, "%Y-%m-%d-Note-%H-%M.pdf", localtime(&curtime)); if (ui.default_path!=NULL) gtk_file_chooser_set_current_folder(GTK_FILE_CHOOSER (dialog), ui.default_path); gtk_file_chooser_set_current_name(GTK_FILE_CHOOSER (dialog), stime); @@ -437,7 +447,6 @@ on_fileQuit_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); if (ok_to_close()) gtk_main_quit (); } @@ -457,10 +466,10 @@ on_editUndo_activate (GtkMenuItem *menuitem, GnomeCanvasGroup *group; end_text(); - reset_focus(); if (undo == NULL) return; // nothing to undo! reset_selection(); // safer - if (undo->type == ITEM_STROKE || undo->type == ITEM_TEXT) { + reset_recognizer(); // safer + if (undo->type == ITEM_STROKE || undo->type == ITEM_TEXT || undo->type == ITEM_IMAGE) { // we're keeping the stroke info, but deleting the canvas item gtk_object_destroy(GTK_OBJECT(undo->item->canvas_item)); undo->item->canvas_item = NULL; @@ -671,10 +680,10 @@ on_editRedo_activate (GtkMenuItem *menuitem, GnomeCanvasGroup *group; end_text(); - reset_focus(); if (redo == NULL) return; // nothing to redo! reset_selection(); // safer - if (redo->type == ITEM_STROKE || redo->type == ITEM_TEXT) { + reset_recognizer(); // safer + if (redo->type == ITEM_STROKE || redo->type == ITEM_TEXT || redo->type == ITEM_IMAGE) { // re-create the canvas_item make_canvas_item_one(redo->layer->group, redo->item); // reinsert the item on its layer @@ -832,6 +841,12 @@ on_editRedo_activate (GtkMenuItem *menuitem, if (it->type == ITEM_TEXT && it->canvas_item != NULL) gnome_canvas_item_set(it->canvas_item, "fill-color-rgba", it->brush.color_rgba, NULL); + if (it->type == ITEM_IMAGE && it->canvas_item != NULL) { + // remark: a variable-width item might have lost its variable-width + group = (GnomeCanvasGroup *) it->canvas_item->parent; + gtk_object_destroy(GTK_OBJECT(it->canvas_item)); + make_canvas_item_one(group, it); + } } } else if (redo->type == ITEM_TEXT_EDIT) { @@ -873,7 +888,6 @@ on_editCut_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); selection_to_clip(); selection_delete(); } @@ -884,7 +898,6 @@ on_editCopy_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); selection_to_clip(); } @@ -894,7 +907,6 @@ on_editPaste_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); clipboard_paste(); } @@ -904,7 +916,6 @@ on_editDelete_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); selection_delete(); } @@ -917,7 +928,6 @@ on_viewContinuous_activate (GtkMenuItem *menuitem, double yscroll; struct Page *pg; - reset_focus(); if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem))) return; if (ui.view_continuous) return; ui.view_continuous = TRUE; @@ -938,7 +948,6 @@ on_viewOnePage_activate (GtkMenuItem *menuitem, GtkAdjustment *v_adj; double yscroll; - reset_focus(); if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem))) return; if (!ui.view_continuous) return; ui.view_continuous = FALSE; @@ -955,12 +964,12 @@ void on_viewZoomIn_activate (GtkMenuItem *menuitem, gpointer user_data) { - reset_focus(); if (ui.zoom > MAX_ZOOM) return; ui.zoom *= ui.zoom_step_factor; gnome_canvas_set_pixels_per_unit(canvas, ui.zoom); rescale_text_items(); rescale_bg_pixmaps(); + rescale_images(); } @@ -968,12 +977,12 @@ void on_viewZoomOut_activate (GtkMenuItem *menuitem, gpointer user_data) { - reset_focus(); if (ui.zoom < MIN_ZOOM) return; ui.zoom /= ui.zoom_step_factor; gnome_canvas_set_pixels_per_unit(canvas, ui.zoom); rescale_text_items(); rescale_bg_pixmaps(); + rescale_images(); } @@ -981,11 +990,11 @@ void on_viewNormalSize_activate (GtkMenuItem *menuitem, gpointer user_data) { - reset_focus(); ui.zoom = DEFAULT_ZOOM; gnome_canvas_set_pixels_per_unit(canvas, ui.zoom); rescale_text_items(); rescale_bg_pixmaps(); + rescale_images(); } @@ -993,11 +1002,11 @@ void on_viewPageWidth_activate (GtkMenuItem *menuitem, gpointer user_data) { - reset_focus(); ui.zoom = (GTK_WIDGET(canvas))->allocation.width/ui.cur_page->width; gnome_canvas_set_pixels_per_unit(canvas, ui.zoom); rescale_text_items(); rescale_bg_pixmaps(); + rescale_images(); } @@ -1006,7 +1015,6 @@ on_viewFirstPage_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); do_switch_page(0, TRUE, FALSE); } @@ -1016,7 +1024,6 @@ on_viewPreviousPage_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); if (ui.pageno == 0) return; do_switch_page(ui.pageno-1, TRUE, FALSE); } @@ -1027,7 +1034,6 @@ on_viewNextPage_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); if (ui.pageno == journal.npages-1) { // create a page at end on_journalNewPageEnd_activate(menuitem, user_data); return; @@ -1041,7 +1047,6 @@ on_viewLastPage_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); do_switch_page(journal.npages-1, TRUE, FALSE); } @@ -1051,7 +1056,6 @@ on_viewShowLayer_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); if (ui.layerno == ui.cur_page->nlayers-1) return; reset_selection(); ui.layerno++; @@ -1066,7 +1070,6 @@ on_viewHideLayer_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); if (ui.layerno == -1) return; reset_selection(); gnome_canvas_item_hide(GNOME_CANVAS_ITEM(ui.cur_layer->group)); @@ -1084,7 +1087,6 @@ on_journalNewPageBefore_activate (GtkMenuItem *menuitem, struct Page *pg; end_text(); - reset_focus(); reset_selection(); pg = new_page(ui.cur_page); journal.pages = g_list_insert(journal.pages, pg, ui.pageno); @@ -1105,7 +1107,6 @@ on_journalNewPageAfter_activate (GtkMenuItem *menuitem, struct Page *pg; end_text(); - reset_focus(); reset_selection(); pg = new_page(ui.cur_page); journal.pages = g_list_insert(journal.pages, pg, ui.pageno+1); @@ -1126,7 +1127,6 @@ on_journalNewPageEnd_activate (GtkMenuItem *menuitem, struct Page *pg; end_text(); - reset_focus(); reset_selection(); pg = new_page((struct Page *)g_list_last(journal.pages)->data); journal.pages = g_list_append(journal.pages, pg); @@ -1148,9 +1148,9 @@ on_journalDeletePage_activate (GtkMenuItem *menuitem, struct Layer *l; end_text(); - reset_focus(); if (journal.npages == 1) return; reset_selection(); + reset_recognizer(); // safer prepare_new_undo(); undo->type = ITEM_DELETE_PAGE; undo->val = ui.pageno; @@ -1183,7 +1183,6 @@ on_journalNewLayer_activate (GtkMenuItem *menuitem, struct Layer *l; end_text(); - reset_focus(); reset_selection(); l = g_new(struct Layer, 1); l->items = NULL; @@ -1213,9 +1212,9 @@ on_journalDeleteLayer_activate (GtkMenuItem *menuitem, GList *list; end_text(); - reset_focus(); if (ui.cur_layer == NULL) return; reset_selection(); + reset_recognizer(); // safer prepare_new_undo(); undo->type = ITEM_DELETE_LAYER; undo->val = ui.layerno; @@ -1286,7 +1285,6 @@ on_journalPaperSize_activate (GtkMenuItem *menuitem, GList *pglist; end_text(); - reset_focus(); papersize_dialog = create_papersizeDialog(); papersize_width = ui.cur_page->width; papersize_height = ui.cur_page->height; @@ -1337,8 +1335,7 @@ on_papercolorWhite_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); - process_papercolor_activate(menuitem, COLOR_WHITE); + process_papercolor_activate(menuitem, COLOR_WHITE, predef_bgcolors_rgba[COLOR_WHITE]); } @@ -1347,8 +1344,7 @@ on_papercolorYellow_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); - process_papercolor_activate(menuitem, COLOR_YELLOW); + process_papercolor_activate(menuitem, COLOR_YELLOW, predef_bgcolors_rgba[COLOR_YELLOW]); } @@ -1357,8 +1353,7 @@ on_papercolorPink_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); - process_papercolor_activate(menuitem, COLOR_RED); + process_papercolor_activate(menuitem, COLOR_RED, predef_bgcolors_rgba[COLOR_RED]); } @@ -1367,8 +1362,7 @@ on_papercolorOrange_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); - process_papercolor_activate(menuitem, COLOR_ORANGE); + process_papercolor_activate(menuitem, COLOR_ORANGE, predef_bgcolors_rgba[COLOR_ORANGE]); } @@ -1377,8 +1371,7 @@ on_papercolorBlue_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); - process_papercolor_activate(menuitem, COLOR_BLUE); + process_papercolor_activate(menuitem, COLOR_BLUE, predef_bgcolors_rgba[COLOR_BLUE]); } @@ -1387,8 +1380,7 @@ on_papercolorGreen_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); - process_papercolor_activate(menuitem, COLOR_GREEN); + process_papercolor_activate(menuitem, COLOR_GREEN, predef_bgcolors_rgba[COLOR_GREEN]); } @@ -1396,7 +1388,25 @@ void on_papercolorOther_activate (GtkMenuItem *menuitem, gpointer user_data) { - + GtkWidget *dialog; + GtkColorSelection *colorsel; + gint result; + guint rgba; + GdkColor gdkcolor; + + end_text(); + dialog = gtk_color_selection_dialog_new(_("Pick a Paper Color")); + colorsel = GTK_COLOR_SELECTION(GTK_COLOR_SELECTION_DIALOG(dialog)->colorsel); + if (ui.cur_page->bg->type == BG_SOLID) rgba = ui.cur_page->bg->color_rgba; + else rgba = ui.default_page.bg->color_rgba; + rgb_to_gdkcolor(rgba, &gdkcolor); + gtk_color_selection_set_current_color(colorsel, &gdkcolor); + + if (gtk_dialog_run(GTK_DIALOG(dialog)) == GTK_RESPONSE_OK) { + gtk_color_selection_get_current_color(colorsel, &gdkcolor); + process_papercolor_activate(menuitem, COLOR_OTHER, gdkcolor_to_rgba(gdkcolor, 0xffff)); + } + gtk_widget_destroy(dialog); } @@ -1405,7 +1415,6 @@ on_paperstylePlain_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); process_paperstyle_activate(menuitem, RULING_NONE); } @@ -1415,7 +1424,6 @@ on_paperstyleLined_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); process_paperstyle_activate(menuitem, RULING_LINED); } @@ -1425,7 +1433,6 @@ on_paperstyleRuled_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); process_paperstyle_activate(menuitem, RULING_RULED); } @@ -1435,7 +1442,6 @@ on_paperstyleGraph_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); process_paperstyle_activate(menuitem, RULING_GRAPH); } @@ -1454,7 +1460,6 @@ on_journalLoadBackground_activate (GtkMenuItem *menuitem, gboolean attach; end_text(); - reset_focus(); dialog = gtk_file_chooser_dialog_new(_("Open Background"), GTK_WINDOW (winMain), GTK_FILE_CHOOSER_ACTION_OPEN, GTK_STOCK_CANCEL, GTK_RESPONSE_CANCEL, GTK_STOCK_OPEN, GTK_RESPONSE_OK, NULL); @@ -1561,6 +1566,7 @@ on_journalLoadBackground_activate (GtkMenuItem *menuitem, gnome_canvas_set_pixels_per_unit(canvas, ui.zoom); rescale_text_items(); rescale_bg_pixmaps(); + rescale_images(); } do_switch_page(ui.pageno, TRUE, TRUE); } @@ -1572,7 +1578,6 @@ on_journalScreenshot_activate (GtkMenuItem *menuitem, struct Background *bg; end_text(); - reset_focus(); reset_selection(); gtk_window_iconify(GTK_WINDOW(winMain)); // hide ourselves gdk_display_sync(gdk_display_get_default()); @@ -1608,6 +1613,7 @@ on_journalScreenshot_activate (GtkMenuItem *menuitem, gnome_canvas_set_pixels_per_unit(canvas, ui.zoom); rescale_text_items(); rescale_bg_pixmaps(); + rescale_images(); } do_switch_page(ui.pageno, TRUE, TRUE); } @@ -1620,7 +1626,6 @@ on_journalApplyAllPages_activate (GtkMenuItem *menuitem, gboolean active; end_text(); - reset_focus(); active = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); if (active == ui.bg_apply_all_pages) return; ui.bg_apply_all_pages = active; @@ -1674,7 +1679,6 @@ on_toolsPen_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) end_text(); - reset_focus(); reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_PEN; ui.cur_brush = &(ui.brushes[ui.cur_mapping][TOOL_PEN]); @@ -1705,7 +1709,6 @@ on_toolsEraser_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) end_text(); - reset_focus(); reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_ERASER; ui.cur_brush = &(ui.brushes[ui.cur_mapping][TOOL_ERASER]); @@ -1734,7 +1737,6 @@ on_toolsHighlighter_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) end_text(); - reset_focus(); reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_HIGHLIGHTER; ui.cur_brush = &(ui.brushes[ui.cur_mapping][TOOL_HIGHLIGHTER]); @@ -1764,7 +1766,6 @@ on_toolsText_activate (GtkMenuItem *menuitem, if (ui.toolno[ui.cur_mapping] == TOOL_TEXT) return; ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) - reset_focus(); reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_TEXT; ui.cur_brush = &(ui.brushes[ui.cur_mapping][TOOL_PEN]); @@ -1777,10 +1778,55 @@ on_toolsText_activate (GtkMenuItem *menuitem, void -on_toolsSelectRegion_activate (GtkMenuItem *menuitem, +on_toolsImage_activate (GtkMenuItem *menuitem, gpointer user_data) { + if (GTK_OBJECT_TYPE(menuitem) == GTK_TYPE_RADIO_MENU_ITEM) { + if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem))) + return; + } else { + if (!gtk_toggle_tool_button_get_active(GTK_TOGGLE_TOOL_BUTTON (menuitem))) + return; + } + + if (ui.cur_mapping != 0 && !ui.button_switch_mapping) return; // not user-generated + if (ui.toolno[ui.cur_mapping] == TOOL_IMAGE) return; + + ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) + end_text(); + reset_selection(); + ui.toolno[ui.cur_mapping] = TOOL_IMAGE; + update_mapping_linkings(-1); + update_tool_buttons(); + update_tool_menu(); + update_color_menu(); + update_cursor(); +} + +void +on_toolsSelectRegion_activate (GtkMenuItem *menuitem, + gpointer user_data) +{ + if (GTK_OBJECT_TYPE(menuitem) == GTK_TYPE_RADIO_MENU_ITEM) { + if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem))) + return; + } else { + if (!gtk_toggle_tool_button_get_active(GTK_TOGGLE_TOOL_BUTTON (menuitem))) + return; + } + + if (ui.cur_mapping != 0 && !ui.button_switch_mapping) return; // not user-generated + if (ui.toolno[ui.cur_mapping] == TOOL_SELECTREGION) return; + + ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) + end_text(); + ui.toolno[ui.cur_mapping] = TOOL_SELECTREGION; + update_mapping_linkings(-1); + update_tool_buttons(); + update_tool_menu(); + update_color_menu(); + update_cursor(); } @@ -1801,7 +1847,6 @@ on_toolsSelectRectangle_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) end_text(); - reset_focus(); ui.toolno[ui.cur_mapping] = TOOL_SELECTRECT; update_mapping_linkings(-1); update_tool_buttons(); @@ -1828,7 +1873,6 @@ on_toolsVerticalSpace_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; // don't use switch_mapping() (refreshes buttons too soon) end_text(); - reset_focus(); reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_VERTSPACE; update_mapping_linkings(-1); @@ -2005,7 +2049,6 @@ on_eraserStandard_activate (GtkMenuItem *menuitem, { if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem))) return; end_text(); - reset_focus(); ui.brushes[0][TOOL_ERASER].tool_options = TOOLOPT_ERASER_STANDARD; update_mapping_linkings(TOOL_ERASER); } @@ -2017,7 +2060,6 @@ on_eraserWhiteout_activate (GtkMenuItem *menuitem, { if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem))) return; end_text(); - reset_focus(); ui.brushes[0][TOOL_ERASER].tool_options = TOOLOPT_ERASER_WHITEOUT; update_mapping_linkings(TOOL_ERASER); } @@ -2029,7 +2071,6 @@ on_eraserDeleteStrokes_activate (GtkMenuItem *menuitem, { if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem))) return; end_text(); - reset_focus(); ui.brushes[0][TOOL_ERASER].tool_options = TOOLOPT_ERASER_STROKES; update_mapping_linkings(TOOL_ERASER); } @@ -2072,7 +2113,6 @@ on_toolsTextFont_activate (GtkMenuItem *menuitem, g_free(str); if (gtk_dialog_run(GTK_DIALOG(dialog)) != GTK_RESPONSE_OK) { gtk_widget_destroy(dialog); - reset_focus(); return; } str = gtk_font_selection_dialog_get_font_name(GTK_FONT_SELECTION_DIALOG(dialog)); @@ -2086,7 +2126,6 @@ on_toolsDefaultPen_activate (GtkMenuItem *menuitem, { switch_mapping(0); end_text(); - reset_focus(); reset_selection(); g_memmove(&(ui.brushes[0][TOOL_PEN]), ui.default_brushes+TOOL_PEN, sizeof(struct Brush)); ui.toolno[0] = TOOL_PEN; @@ -2106,7 +2145,6 @@ on_toolsDefaultEraser_activate (GtkMenuItem *menuitem, { switch_mapping(0); end_text(); - reset_focus(); reset_selection(); g_memmove(&(ui.brushes[0][TOOL_ERASER]), ui.default_brushes+TOOL_ERASER, sizeof(struct Brush)); ui.toolno[0] = TOOL_ERASER; @@ -2126,7 +2164,6 @@ on_toolsDefaultHighlighter_activate (GtkMenuItem *menuitem, { switch_mapping(0); end_text(); - reset_focus(); reset_selection(); g_memmove(&(ui.brushes[0][TOOL_HIGHLIGHTER]), ui.default_brushes+TOOL_HIGHLIGHTER, sizeof(struct Brush)); ui.toolno[0] = TOOL_HIGHLIGHTER; @@ -2145,7 +2182,6 @@ on_toolsDefaultText_activate (GtkMenuItem *menuitem, { switch_mapping(0); if (ui.toolno[0]!=TOOL_TEXT) end_text(); - reset_focus(); reset_selection(); ui.toolno[0] = TOOL_TEXT; ui.cur_brush = &(ui.brushes[0][TOOL_PEN]); @@ -2194,7 +2230,6 @@ on_toolsSetAsDefault_activate (GtkMenuItem *menuitem, ui.default_font_size = ui.font_size; } end_text(); - reset_focus(); } @@ -2215,7 +2250,6 @@ on_toolsRuler_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; end_text(); - reset_focus(); if (ui.toolno[ui.cur_mapping]!=TOOL_PEN && ui.toolno[ui.cur_mapping]!=TOOL_HIGHLIGHTER) { reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_PEN; @@ -2250,7 +2284,6 @@ on_toolsReco_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; end_text(); - reset_focus(); if (ui.toolno[ui.cur_mapping]!=TOOL_PEN && ui.toolno[ui.cur_mapping]!=TOOL_HIGHLIGHTER) { reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_PEN; @@ -2276,7 +2309,6 @@ on_optionsSavePreferences_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); save_config_to_file(); } @@ -2297,11 +2329,10 @@ on_helpAbout_activate (GtkMenuItem *menuitem, GtkLabel *labelTitle; end_text(); - reset_focus(); aboutDialog = create_aboutDialog (); labelTitle = GTK_LABEL(g_object_get_data(G_OBJECT(aboutDialog), "labelTitle")); gtk_label_set_markup(labelTitle, - "Xournal " VERSION ""); + "Xournal " VERSION_STRING ""); gtk_dialog_run (GTK_DIALOG(aboutDialog)); gtk_widget_destroy(aboutDialog); } @@ -2316,7 +2347,6 @@ on_buttonToolDefault_clicked (GtkToolButton *toolbutton, return; } end_text(); - reset_focus(); switch_mapping(0); if (ui.toolno[0] < NUM_STROKE_TOOLS) { g_memmove(&(ui.brushes[0][ui.toolno[0]]), ui.default_brushes+ui.toolno[0], sizeof(struct Brush)); @@ -2365,8 +2395,6 @@ on_canvas_button_press_event (GtkWidget *widget, gpointer user_data) { double pt[2]; - gboolean page_change; - struct Page *tmppage; GtkWidget *dialog; int mapping; gboolean is_core; @@ -2378,8 +2406,9 @@ on_canvas_button_press_event (GtkWidget *widget, event->device->name, event->x, event->y, event->button, event->state); #endif - if (ui.cur_item_type != ITEM_TEXT) // remove focus from other elements - gtk_widget_grab_focus(GTK_WIDGET(canvas)); + // abort any page changes pending in the spin button, and take the focus + gtk_spin_button_set_value(GTK_SPIN_BUTTON(GET_COMPONENT("spinPageNo")), ui.pageno+1); + reset_focus(); is_core = (event->device == gdk_device_get_core_pointer()); if (!ui.use_xinput && !is_core) return FALSE; @@ -2407,17 +2436,18 @@ on_canvas_button_press_event (GtkWidget *widget, else if (event->button == 5) scroll_event.scroll.direction = GDK_SCROLL_DOWN; else if (event->button == 6) scroll_event.scroll.direction = GDK_SCROLL_LEFT; else scroll_event.scroll.direction = GDK_SCROLL_RIGHT; - printf("sending...\n"); gtk_widget_event(GET_COMPONENT("scrolledwindowMain"), &scroll_event); } return FALSE; } if ((event->state & (GDK_CONTROL_MASK|GDK_MOD1_MASK)) != 0) return FALSE; // no control-clicking or alt-clicking + if (!is_core) gdk_device_get_state(event->device, event->window, event->axes, NULL); + // synaptics touchpads send bogus axis values with ButtonDown if (!is_core) fix_xinput_coords((GdkEvent *)event); - if (!finite(event->x) || !finite(event->y)) return FALSE; // Xorg 7.3 bug + if (!finite_sized(event->x) || !finite_sized(event->y)) return FALSE; // Xorg 7.3 bug if (ui.cur_item_type == ITEM_TEXT) { if (!is_event_within_textview(event)) end_text(); @@ -2451,24 +2481,8 @@ on_canvas_button_press_event (GtkWidget *widget, else mapping = event->button-1; // check whether we're in a page - page_change = FALSE; - tmppage = ui.cur_page; get_pointer_coords((GdkEvent *)event, pt); - while (ui.view_continuous && (pt[1] < - VIEW_CONTINUOUS_SKIP)) { - if (ui.pageno == 0) break; - page_change = TRUE; - ui.pageno--; - tmppage = g_list_nth_data(journal.pages, ui.pageno); - pt[1] += tmppage->height + VIEW_CONTINUOUS_SKIP; - } - while (ui.view_continuous && (pt[1] > tmppage->height + VIEW_CONTINUOUS_SKIP)) { - if (ui.pageno == journal.npages-1) break; - pt[1] -= tmppage->height + VIEW_CONTINUOUS_SKIP; - page_change = TRUE; - ui.pageno++; - tmppage = g_list_nth_data(journal.pages, ui.pageno); - } - if (page_change) do_switch_page(ui.pageno, FALSE, FALSE); + set_current_page(pt); // can't paint on the background... @@ -2487,6 +2501,9 @@ on_canvas_button_press_event (GtkWidget *widget, ui.which_mouse_button = event->button; switch_mapping(mapping); +#ifdef WIN32 + update_cursor(); +#endif // in text tool, clicking in a text area edits it if (ui.toolno[mapping] == TOOL_TEXT) { @@ -2522,6 +2539,9 @@ on_canvas_button_press_event (GtkWidget *widget, do_eraser((GdkEvent *)event, ui.cur_brush->thickness/2, ui.cur_brush->tool_options == TOOLOPT_ERASER_STROKES); } + else if (ui.toolno[mapping] == TOOL_SELECTREGION) { + start_selectregion((GdkEvent *)event); + } else if (ui.toolno[mapping] == TOOL_SELECTRECT) { start_selectrect((GdkEvent *)event); } @@ -2531,6 +2551,9 @@ on_canvas_button_press_event (GtkWidget *widget, else if (ui.toolno[mapping] == TOOL_TEXT) { start_text((GdkEvent *)event, NULL); } + else if (ui.toolno[mapping] == TOOL_IMAGE) { + insert_image((GdkEvent *)event); + } return FALSE; } @@ -2563,6 +2586,9 @@ on_canvas_button_release_event (GtkWidget *widget, else if (ui.cur_item_type == ITEM_ERASURE) { finalize_erasure(); } + else if (ui.cur_item_type == ITEM_SELECTREGION) { + finalize_selectregion(); + } else if (ui.cur_item_type == ITEM_SELECTRECT) { finalize_selectrect(); } @@ -2588,7 +2614,25 @@ on_canvas_enter_notify_event (GtkWidget *widget, GdkEventCrossing *event, gpointer user_data) { + GList *dev_list; + GdkDevice *dev; +#ifdef INPUT_DEBUG + printf("DEBUG: enter notify\n"); +#endif + /* re-enable input devices after they've been emergency-disabled + by leave_notify */ + if (!gtk_check_version(2, 17, 0)) { + gdk_flush(); + gdk_error_trap_push(); + for (dev_list = gdk_devices_list(); dev_list != NULL; dev_list = dev_list->next) { + dev = GDK_DEVICE(dev_list->data); + gdk_device_set_mode(dev, GDK_MODE_SCREEN); + } + ui.is_corestroke = ui.saved_is_corestroke; + gdk_flush(); + gdk_error_trap_pop(); + } return FALSE; } @@ -2597,12 +2641,25 @@ on_canvas_leave_notify_event (GtkWidget *widget, GdkEventCrossing *event, gpointer user_data) { + GList *dev_list; + GdkDevice *dev; + #ifdef INPUT_DEBUG - printf("DEBUG: leave notify\n"); + printf("DEBUG: leave notify (mode=%d, details=%d)\n", event->mode, event->detail); #endif - if (ui.need_emergency_disable_xinput) { - gtk_widget_set_extension_events(GTK_WIDGET (canvas), GDK_EXTENSION_EVENTS_NONE); - ui.need_emergency_disable_xinput = FALSE; + /* emergency disable XInput to avoid segfaults (GTK+ 2.17) or + interface non-responsiveness (GTK+ 2.18) */ + if (!gtk_check_version(2, 17, 0)) { + gdk_flush(); + gdk_error_trap_push(); + for (dev_list = gdk_devices_list(); dev_list != NULL; dev_list = dev_list->next) { + dev = GDK_DEVICE(dev_list->data); + gdk_device_set_mode(dev, GDK_MODE_DISABLED); + } + ui.saved_is_corestroke = ui.is_corestroke; + ui.is_corestroke = TRUE; + gdk_flush(); + gdk_error_trap_pop(); } return FALSE; } @@ -2630,7 +2687,6 @@ on_canvas_key_press_event (GtkWidget *widget, if (event->keyval == GDK_Escape) { if (ui.cur_item_type == ITEM_TEXT) { end_text(); - reset_focus(); return TRUE; } else if (ui.fullscreen) { @@ -2690,12 +2746,12 @@ on_canvas_motion_notify_event (GtkWidget *widget, or if there's a selection (then we might want to change the mouse cursor to indicate the possibility of resizing) */ if (ui.cur_item_type == ITEM_NONE && ui.selection==NULL) return FALSE; - if (ui.cur_item_type == ITEM_TEXT) return FALSE; + if (ui.cur_item_type == ITEM_TEXT || ui.cur_item_type == ITEM_IMAGE) return FALSE; is_core = (event->device == gdk_device_get_core_pointer()); if (!ui.use_xinput && !is_core) return FALSE; if (!is_core) fix_xinput_coords((GdkEvent *)event); - if (!finite(event->x) || !finite(event->y)) return FALSE; // Xorg 7.3 bug + if (!finite_sized(event->x) || !finite_sized(event->y)) return FALSE; // Xorg 7.3 bug if (ui.selection!=NULL && ui.cur_item_type == ITEM_NONE) { get_pointer_coords((GdkEvent *)event, pt); @@ -2725,6 +2781,9 @@ on_canvas_motion_notify_event (GtkWidget *widget, else if (ui.cur_item_type == ITEM_ERASURE) { finalize_erasure(); } + else if (ui.cur_item_type == ITEM_SELECTREGION) { + finalize_selectregion(); + } else if (ui.cur_item_type == ITEM_SELECTRECT) { finalize_selectrect(); } @@ -2734,6 +2793,9 @@ on_canvas_motion_notify_event (GtkWidget *widget, else if (ui.cur_item_type == ITEM_RESIZESEL) { finalize_resizesel(); } + else if (ui.cur_item_type == ITEM_HAND) { + ui.cur_item_type = ITEM_NONE; + } switch_mapping(0); return FALSE; } @@ -2745,6 +2807,9 @@ on_canvas_motion_notify_event (GtkWidget *widget, do_eraser((GdkEvent *)event, ui.cur_brush->thickness/2, ui.cur_brush->tool_options == TOOLOPT_ERASER_STROKES); } + else if (ui.cur_item_type == ITEM_SELECTREGION) { + continue_selectregion((GdkEvent *)event); + } else if (ui.cur_item_type == ITEM_SELECTRECT) { get_pointer_coords((GdkEvent *)event, pt); ui.selection->bbox.right = pt[0]; @@ -2774,7 +2839,6 @@ on_comboLayer_changed (GtkComboBox *combobox, if (ui.in_update_page_stuff) return; // avoid a bad retroaction end_text(); - reset_focus(); val = gtk_combo_box_get_active(combobox); if (val == -1) return; @@ -2803,7 +2867,6 @@ on_winMain_delete_event (GtkWidget *widget, gpointer user_data) { end_text(); - reset_focus(); if (ok_to_close()) gtk_main_quit(); return TRUE; } @@ -2814,7 +2877,6 @@ on_optionsUseXInput_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); ui.allow_xinput = ui.use_xinput = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); @@ -2839,7 +2901,9 @@ on_optionsUseXInput_activate (GtkMenuItem *menuitem, non-responsive). */ +#ifndef WIN32 if (!gtk_check_version(2, 17, 0)) { +#endif /* GTK+ 2.17 and later: everybody shares a single native window, so we'll never get any core events, and we might as well set extension events the way we're supposed to. Doing so helps solve @@ -2847,14 +2911,18 @@ on_optionsUseXInput_activate (GtkMenuItem *menuitem, events in 2.18 */ gtk_widget_set_extension_events(GTK_WIDGET (canvas), ui.use_xinput?GDK_EXTENSION_EVENTS_ALL:GDK_EXTENSION_EVENTS_NONE); +#ifndef WIN32 } else { +#endif /* GTK+ 2.16 and earlier: we only activate extension events on the canvas's parent GdkWindow. This allows us to keep receiving core events. */ gdk_input_set_extension_events(GTK_WIDGET(canvas)->window, GDK_POINTER_MOTION_MASK | GDK_BUTTON_MOTION_MASK | GDK_BUTTON_PRESS_MASK | GDK_BUTTON_RELEASE_MASK, ui.use_xinput?GDK_EXTENSION_EVENTS_ALL:GDK_EXTENSION_EVENTS_NONE); +#ifndef WIN32 } +#endif update_mappings_menu(); } @@ -2890,7 +2958,6 @@ on_vscroll_changed (GtkAdjustment *adjustment, end_text(); do_switch_page(ui.pageno, FALSE, FALSE); } - reset_focus(); return; } @@ -2901,9 +2968,13 @@ on_spinPageNo_value_changed (GtkSpinButton *spinbutton, int val; if (ui.in_update_page_stuff) return; // avoid a bad retroaction - + + /* in preparation for end_text(), send focus to the canvas if it's not ours. + (avoid issues with Gtk trying to send focus to the dead text widget) */ + + if (!GTK_WIDGET_HAS_FOCUS(spinbutton)) + gtk_widget_grab_focus(GTK_WIDGET(canvas)); end_text(); - reset_focus(); val = gtk_spin_button_get_value_as_int(spinbutton) - 1; @@ -2927,7 +2998,6 @@ on_journalDefaultBackground_activate (GtkMenuItem *menuitem, GList *pglist; end_text(); - reset_focus(); reset_selection(); pg = ui.cur_page; @@ -2964,7 +3034,6 @@ on_journalSetAsDefault_activate (GtkMenuItem *menuitem, if (ui.cur_page->bg->type != BG_SOLID) return; end_text(); - reset_focus(); prepare_new_undo(); undo->type = ITEM_NEW_DEFAULT_BG; undo->val_x = ui.default_page.width; @@ -3107,7 +3176,6 @@ on_optionsButtonMappings_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); ui.use_erasertip = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); update_mappings_menu(); @@ -3123,7 +3191,6 @@ on_optionsProgressiveBG_activate (GtkMenuItem *menuitem, active = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); if (ui.progressive_bg == active) return; end_text(); - reset_focus(); ui.progressive_bg = active; if (!ui.progressive_bg) rescale_bg_pixmaps(); } @@ -3138,7 +3205,6 @@ on_mru_activate (GtkMenuItem *menuitem, GtkWidget *dialog; end_text(); - reset_focus(); if (!ok_to_close()) return; // user aborted on save confirmation for (which = 0 ; which < MRU_SIZE; which++) { @@ -3192,6 +3258,14 @@ on_button2Text_activate (GtkMenuItem *menuitem, } +void +on_button2Image_activate (GtkMenuItem *menuitem, + gpointer user_data) +{ + process_mapping_activate(menuitem, 1, TOOL_IMAGE); +} + + void on_button2SelectRegion_activate (GtkMenuItem *menuitem, gpointer user_data) @@ -3224,7 +3298,6 @@ on_button2LinkBrush_activate (GtkMenuItem *menuitem, if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM(menuitem))) return; end_text(); - reset_focus(); ui.linked_brush[1] = BRUSH_LINKED; for (i=0;i= NUM_STROKE_TOOLS) { ui.linked_brush[1] = BRUSH_STATIC; update_mappings_menu_linkings(); @@ -3279,6 +3351,14 @@ on_button3Text_activate (GtkMenuItem *menuitem, } +void +on_button3Image_activate (GtkMenuItem *menuitem, + gpointer user_data) +{ + process_mapping_activate(menuitem, 2, TOOL_IMAGE); +} + + void on_button3SelectRegion_activate (GtkMenuItem *menuitem, gpointer user_data) @@ -3311,7 +3391,6 @@ on_button3LinkBrush_activate (GtkMenuItem *menuitem, if (!gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM(menuitem))) return; end_text(); - reset_focus(); ui.linked_brush[2] = BRUSH_LINKED; for (i=0;i= NUM_STROKE_TOOLS) { ui.linked_brush[2] = BRUSH_STATIC; update_mappings_menu_linkings(); @@ -3347,7 +3425,6 @@ on_viewSetZoom_activate (GtkMenuItem *menuitem, GtkSpinButton *spinZoom; end_text(); - reset_focus(); zoom_dialog = create_zoomDialog(); zoom_percent = 100*ui.zoom / DEFAULT_ZOOM; spinZoom = GTK_SPIN_BUTTON(g_object_get_data(G_OBJECT(zoom_dialog), "spinZoom")); @@ -3375,6 +3452,7 @@ on_viewSetZoom_activate (GtkMenuItem *menuitem, gnome_canvas_set_pixels_per_unit(canvas, ui.zoom); rescale_text_items(); rescale_bg_pixmaps(); + rescale_images(); } } while (response == GTK_RESPONSE_APPLY); @@ -3458,7 +3536,6 @@ on_toolsHand_activate (GtkMenuItem *menuitem, ui.cur_mapping = 0; end_text(); - reset_focus(); reset_selection(); ui.toolno[ui.cur_mapping] = TOOL_HAND; update_mapping_linkings(-1); @@ -3490,19 +3567,15 @@ on_optionsPrintRuling_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); ui.print_ruling = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); } void -on_optionsDiscardCore_activate (GtkMenuItem *menuitem, +on_optionsAutoloadPdfXoj_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); - ui.discard_corepointer = - gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); - update_mappings_menu(); + ui.autoload_pdf_xoj = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); } void @@ -3520,7 +3593,6 @@ on_optionsLeftHanded_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); ui.left_handed = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); gtk_scrolled_window_set_placement(GTK_SCROLLED_WINDOW(GET_COMPONENT("scrolledwindowMain")), ui.left_handed?GTK_CORNER_TOP_RIGHT:GTK_CORNER_TOP_LEFT); @@ -3534,7 +3606,6 @@ on_optionsShortenMenus_activate (GtkMenuItem *menuitem, GtkWidget *w; end_text(); - reset_focus(); ui.shorten_menus = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); /* go over the item list */ @@ -3566,7 +3637,6 @@ on_optionsAutoSavePrefs_activate (GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); ui.auto_save_prefs = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); } @@ -3576,7 +3646,6 @@ on_optionsPressureSensitive_activate (GtkMenuItem *menuitem, { int i; end_text(); - reset_focus(); ui.pressure_sensitivity = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); for (i=0; i<=NUM_BUTTONS; i++) @@ -3603,8 +3672,8 @@ on_optionsButtonsSwitchMappings_activate(GtkMenuItem *menuitem, gpointer user_data) { end_text(); - reset_focus(); switch_mapping(0); ui.button_switch_mapping = gtk_check_menu_item_get_active(GTK_CHECK_MENU_ITEM (menuitem)); } +