summaryrefslogtreecommitdiff
AgeCommit message (Collapse)AuthorFilesLines
2024-03-01Do not use MateImageMenuItem where it would have to be subclassedfix_panel--replacelukefromdc1-2/+80
Unlike deprecated GtkImageMenuItem, MateImageMenuItem is declared as "final" and cannot be subclassed. Use GtkImageMenuItem for those two cases until a new major version bump allows fixing this in mate-desktop
2024-02-29Revert "Main menus: fix missing Places, System icons"raveit651-2/+2
- fix crashing panel with --replace option This reverts commit 675f72ff302409dcbfb1991cf56fd09a294039bf. - fixes https://github.com/mate-desktop/mate-panel/issues/1433
2024-02-20release 1.28.0v1.28.0raveit652-1/+5
2024-02-20tx: sync with transifexraveit651-7/+7
2024-02-04release 1.27.3v1.27.3raveit652-19/+52
2024-02-04tx: sync with transifexraveit65147-14288/+18535
2024-02-04Constify some pointer references in localsColomban Wendling1-22/+22
No need to have pointers to mutables when not needed. This does not cover everything at all, and is limited to one file for the moment. origin commit was: https://github.com/mate-desktop/mate-panel/commit/b913762
2024-02-04Reduce scope of more codeColomban Wendling1-7/+7
The computed values are only used in one branch, so compute them there.
2024-02-04Avoid a redundant NULL checkColomban Wendling1-4/+5
It's probably not necessary to perform the NULL check at all as the only code paths that could make `list` NULL are already pretty dramatically broken, but as we have a NULL check move everything relevant inside it.
2024-02-04Reduce scope of variablesColomban Wendling19-70/+55
Mostly found by cppcheck. origin commit was: https://github.com/mate-desktop/mate-panel/commit/96c7ebc
2024-02-04Revert "Add "center-stick" capability applets on the panel"raveit6526-459/+222
This reverts commit 38e00280e9d17282717595a05411736a308804c0.
2024-02-04Revert "Fix center- and right-sticking of expanding applets"raveit651-121/+26
This reverts commit 7b1b00d5ddffc349ca117843bd0d015e9fb08a89.
2024-02-04Revert "Reduce scope of variables"raveit6519-58/+74
This reverts commit 96c7ebc6dc9e8b8327db04a9570054ee78743353.
2024-02-04Revert "Avoid a redundant NULL check"raveit651-5/+4
This reverts commit 17b87e321bfca409c31ce4b4e0e3326eeef92b88.
2024-02-04Revert "Reduce scope of more code"raveit651-7/+7
This reverts commit 655a8f484598a07a210dc56f1662021f8be69737.
2024-02-04Revert "Constify some pointer references in locals"raveit651-30/+30
This reverts commit b91376262abb56d95f9ec86993edaa8e9fb2cc1a.
2024-01-25wayland: keep window list menu off space not used for buttonslukefromdc1-1/+11
*Fix the window list menu replacing the panel context menu between the end of the window list and the next applet
2024-01-25run-dialog: Don't needlessly override DISPLAY environment variableColomban Wendling1-20/+2
This code got introduced with 3c21f66bf30810fcb25dfe8c3c0960df270dc416 trying to mimic `gdk_spawn_on_screen()`, but does not: what `gdk_spawn_on_screen()` did (on X11) was to append the screen number to the display, so that it worked on a multi-screen setup where the app is not on the default X screen. However, the code that got introduced just sets the display, which is gonna be already there anyway (yet possibly loosing the default screen in the process?), and AFAIK GDK dropped support for multi-screen setups anyway (not to be confused with multi-monitor with e.g. Xinerama). We should actually be better of without any of this, as it most likely doesn't do what it is supposed to, nobody needs what it failed to do, and it breaks XWayland on Wayland. Fixes #1387.
2024-01-23ci: use ubuntu jammy as dockerraveit651-1/+1
2024-01-06Update opacity when changed gsettings color valuezhu yaliang1-0/+17
2023-11-22wnck-pager: Fix workspace switcher aspect ratiolukefromdc1-1/+153
2023-11-21Constify some pointer references in localsColomban Wendling1-30/+30
No need to have pointers to mutables when not needed. This does not cover everything at all, and is limited to one file for the moment.
2023-11-21Reduce scope of more codeColomban Wendling1-7/+7
The computed values are only used in one branch, so compute them there.
2023-11-21Remove a redundant NULL checkColomban Wendling1-2/+1
`info` cannot be NULL, and if it was it'd already have crashed above. > mate-panel/panel.c:893:34: warning: Either the condition 'info!=((void*)0)' is redundant or there is possible null pointer dereference: info. [nullPointerRedundantCheck] > parent = gtk_widget_get_parent (info->widget); > ^ > mate-panel/panel.c:895:11: note: Assuming that condition 'info!=((void*)0)' is not redundant > if (info != NULL && > ^ > mate-panel/panel.c:893:34: note: Null pointer dereference > parent = gtk_widget_get_parent (info->widget); > ^
2023-11-21Avoid a redundant NULL checkColomban Wendling1-4/+5
It's probably not necessary to perform the NULL check at all as the only code paths that could make `list` NULL are already pretty dramatically broken, but as we have a NULL check move everything relevant inside it.
2023-11-21Constify a parameterColomban Wendling2-2/+2
2023-11-21Return a value even after an assertion failureColomban Wendling1-1/+1
This is not strictly needed as there's already an assertion (assuming it's not compiled out), but returning a value help analyzers get this right.
2023-11-21Remove redundant shadowing variableColomban Wendling1-1/+0
No need to get the default icon theme again anyway.
2023-11-21Remove unused-but-set variableColomban Wendling1-2/+0
2023-11-21wncklet: Remove double assignmentColomban Wendling1-3/+1
2023-11-21Reduce scope of variablesColomban Wendling19-74/+58
Mostly found by cppcheck.
2023-11-21Remove unused member PanelAction::typeColomban Wendling1-9/+0
> mate-panel/panel-action-button.c:516:26: style: struct member 'PanelAction::type' is never used. [unusedStructMember] > PanelActionButtonType type; > ^
2023-11-21Optimize mate_panel_applet_settings_get_g*list()Colomban Wendling1-10/+10
- Build the list in reverse order, then reverse the result. This is useful because GS?List are list nodes, not containers of nodes, and thus don't contain a pointer to the list's end, meaning to append one has to walk the entire list to find the end each time. To avoid this we use the common idiom of prepending to the list (which is cheap, as it's adding a node before the given one), and then reversing the resulting list to get back the original order. - Avoid unnecessary memory copy by stealing the GStrv's members. We get the array as a copy, so we can simply steal the members and free the container array only, saving a copy for each member.
2023-11-21clock: Simplify sort_locations_by_time()Colomban Wendling1-29/+11
There is no need to return -1 or 1 specifically, anything on the right side of 0 is OK, so simplify using the usual `a - b` implementation for such sort functions. Note that the type is OK here, as the `struct tm` are ints, the same as the function's return value.
2023-11-21Do not perform useless incoherent NULL checkColomban Wendling1-1/+1
cppcheck tells us that we perform NULL checking inconsistently, and reports a potential NULL dereference. Here however the string cannot be NULL, so just drop the unnecessary check. > mate-panel/panel-run-dialog.c:1663:10: warning: Either the condition '!start' is redundant or there is possible null pointer dereference: start. [nullPointerRedundantCheck] > while (*start != '\0' && g_ascii_isspace (*start)) > ^ > mate-panel/panel-run-dialog.c:1679:6: note: Assuming that condition '!start' is not redundant > if (!start || !start [0]) { > ^ > mate-panel/panel-run-dialog.c:1663:10: note: Null pointer dereference > while (*start != '\0' && g_ascii_isspace (*start)) > ^
2023-11-21Reorder array index limit checkColomban Wendling1-3/+1
This is *not* an issue, as it's not really a bounds check but a mere limit, and the array is NULL-terminated. However, it looks odd and cppcheck (unsurprisingly) doesn't understand the details well enough and reports this as a misordered bounds check. > mate-panel/panel-run-dialog.c:166:12: style: Array index 'i' is used before limits check. [arrayIndexThenCheck] > items[i] && i < history_max_size; > ^
2023-10-24status-notifier: Show AttentionIcon when Status is NeedsAttentionColomban Wendling1-9/+30
Fixes #1412.
2023-10-24status-notifier: fix typoAlberts Muktupāvels3-3/+3
Cherry-picked from gnome-panel commit 1cc654e16dc06edccaef8c877db4e0da30ceeeaa.
2023-10-24status-notifier: fix a typoAlberts Muktupāvels1-3/+3
Cherry-picked from gnome-panel commit 55667a735ea33336f5b0912631b184b95ea2140b.
2023-10-12Main menus: fix missing Places, System iconslukefromdc1-2/+2
*After replacing gtk_image_menu_item with mate_image_menu_item *We must also replace GTK_TYPE_IMAGE_MENU_ITEM with MATE_TYPE_IMAGE_MENU_ITEM *This fixes leftover GtkImageMenuItem deprecation warnings and restores missing Places and System icons in main menus
2023-10-09replace deprecated gtk-image-menu-item*raveit657-23/+60
2023-10-05Wayland: Fix force-quit unsupported message dialoglukefromdc1-1/+1
*For some reason "widget" is not being properly passed to this, so use NULL for the parent window
2023-10-05Wayland: Make panel shutdown button work under waylandlukefromdc1-5/+203
*This uses systemd to be able to shutdown when a display manager is running as root *If no session or display manager is running as root, systemd used only to suspend or hibernate *Suspend/hibernate require systemd at this point *If suspend or hibernate not supported error dialog shown if those buttons clicked *If "disable logout" lockdown option set, shut down button cannot be added same as in x11 *If shut down button added to panel and lockout later disabled, same non-responsive button as in x11 *Logind support would be welcome
2023-10-05Force-quit/x11: Fix misalignment of popup with multimonitorslukefromdc1-6/+13
*Compute the size of the monitor we are actually showing the popup on *by getting the monitor it is shown on and running gdk_monitor_get_geometry() *This returns application (scaled) pixels not device pixels
2023-10-02Hidpi x11: center panel force-quit popup windowlukefromdc1-3/+4
2023-09-02tx: update resourcesraveit651-91/+107
2023-08-23release 1.27.2v1.27.2raveit652-1/+25
2023-08-23tx: pull with transifexraveit6520-39/+43
2023-08-12libmate-panel-applet: don't use panel plug to find the screenlukefromdc1-1/+1
* There is only one screen in GTK 3.22 or later * Finding the screen from that used by GtkPanelPlug is out of process only * Any in-process applet calling mate_panel_applet_request_focus needs this to avoid a segfault * One known user: dictionary applet
2023-07-31wayland-tasklist: really set maximum label widthlukefromdc1-1/+4
*Otherwise taskbuttons for programs using URLs or filenames in the window name can balloon across the whole taskbar
generated by cgit v1.2.1 (git 2.18.0) at 2024-11-21 18:14:12 +0000