From b7f37335db9db86c2428d055c3e0c808036c335c Mon Sep 17 00:00:00 2001 From: Max Gaukler Date: Sat, 24 Dec 2022 14:15:03 +0100 Subject: [PATCH 1/2] Align and distribute: Increase icon size This is a regression bugfix. In older versions (<= 1.0), the align and distribute icons were rendered as 24px wide. With newer Inkscape and GTK, this changed to 16px, which makes the icons difficult to distinguish, especially with the new dark symbolic icon theme. The old size of 24px occured as a "technical accident": The button had the default size of 16px, but because the alignment icons are not available in 16px a larger icon was used. Gtk changed this behavior at some point and started scaling down the icons. With the new symbolic theme, 16px icons also became available. Now, a size of 24px is specifically requested. This restores the previous icon size as it has been with Inkscape 0.92 and 1.0. Related: https://gitlab.com/inkscape/inbox/-/issues/6491 (panel icons are too small for some users) Related: https://gitlab.com/inkscape/inbox/-/issues/7283 (16px is way too small for screens with slightly higher DPI (e.g. 150% of the standard 96DPI) --- share/ui/align-and-distribute.ui | 35 ++++++++++++++++++++++++++++++++ 1 file changed, 35 insertions(+) diff --git a/share/ui/align-and-distribute.ui b/share/ui/align-and-distribute.ui index d3e5fada0b..c0548e20b0 100644 --- a/share/ui/align-and-distribute.ui +++ b/share/ui/align-and-distribute.ui @@ -87,209 +87,244 @@ False align-horizontal-baseline True + 3 True False align-horizontal-center True + 3 True False align-horizontal-left True + 3 True False align-horizontal-left-to-anchor True + 3 True False align-horizontal-node True + 3 True False align-horizontal-right True + 3 True False align-horizontal-right-to-anchor True + 3 True False align-on-canvas + 3 True False align-sel-as-group True + 3 True False align-vertical-baseline True + 3 True False align-vertical-bottom True + 3 True False align-vertical-bottom-to-anchor True + 3 True False align-vertical-center True + 3 True False align-vertical-node True + 3 True False align-vertical-top True + 3 True False align-vertical-top-to-anchor True + 3 True False distribute-graph True + 3 True False distribute-horizontal-baseline True + 3 True False distribute-horizontal-center True + 3 True False distribute-horizontal-gaps True + 3 True False distribute-horizontal-left True + 3 True False distribute-horizontal-node True + 3 True False distribute-horizontal-right True + 3 True False distribute-randomize True + 3 True False distribute-remove-overlaps True + 3 True False distribute-unclump True + 3 True False distribute-vertical-baseline True + 3 True False distribute-vertical-bottom True + 3 True False distribute-vertical-center True + 3 True False distribute-vertical-gaps True + 3 True False distribute-vertical-node True + 3 True False distribute-vertical-top True + 3 True False exchange-positions True + 3 True False exchange-positions-clockwise True + 3 True False exchange-positions-zorder True + 3 True -- GitLab From 63d33aff02601d9b0f98fa01cd1f409a1d4194b3 Mon Sep 17 00:00:00 2001 From: Max Gaukler Date: Sun, 1 Jan 2023 15:44:31 +0100 Subject: [PATCH 2/2] Increase icon size in XML editor and LPE tile --- share/ui/dialog-xml.glade | 10 +++++++++- src/live_effects/lpe-tiling.cpp | 3 ++- 2 files changed, 11 insertions(+), 2 deletions(-) diff --git a/share/ui/dialog-xml.glade b/share/ui/dialog-xml.glade index 2caee9d506..8f85fe4691 100644 --- a/share/ui/dialog-xml.glade +++ b/share/ui/dialog-xml.glade @@ -1,46 +1,54 @@ - + True False xml-element-new + 3 True False xml-text-new + 3 True False xml-node-duplicate + 3 True False xml-node-delete + 3 True False format-indent-less + 3 True False format-indent-more + 3 True False go-up + 3 True False go-down + 3 True diff --git a/src/live_effects/lpe-tiling.cpp b/src/live_effects/lpe-tiling.cpp index 3cfef93a2b..02098a3b9f 100644 --- a/src/live_effects/lpe-tiling.cpp +++ b/src/live_effects/lpe-tiling.cpp @@ -623,7 +623,8 @@ LPETiling::toItem(size_t i, bool reset, bool &write) Gtk::RadioButton* create_radio_button(Gtk::RadioButtonGroup& group, const Glib::ustring& tooltip, const Glib::ustring& icon_name) { auto button = Gtk::make_managed(group, Glib::ustring()); button->set_tooltip_text(tooltip); - button->set_image_from_icon_name(icon_name, Gtk::ICON_SIZE_BUTTON); + // note: icon size LARGE_TOOLBAR (24px) instead of the default 16px because the icons contain small details + button->set_image_from_icon_name(icon_name, Gtk::ICON_SIZE_LARGE_TOOLBAR); button->property_draw_indicator() = false; button->property_always_show_image() = true; button->set_halign(Gtk::ALIGN_CENTER); -- GitLab