Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

xfce4-taskmanager: Update to v1.5.7 #4628

Merged
merged 1 commit into from
Dec 20, 2024
Merged

xfce4-taskmanager: Update to v1.5.7 #4628

merged 1 commit into from
Dec 20, 2024

Conversation

malfisya
Copy link
Member

@malfisya malfisya commented Dec 19, 2024 •
edited
Loading

Summary

  • Changes:
    • Update copyright year
    • build: Fix -Wcast-align warning from clang
    • Fix UBSan errors
    • settings-dialog: Fix XtmRefreshRate memory leak
    • Dark mode for process-statusbar
    • Dark mode for process-monitor
    • Allow multiple instances via command line option.
    • Fix blurry app icons when UI scale > 1
    • Use symbolic window picker icon in toolbar
    • Fix broken "show-legend" setting sync
    • Translation Updates: Catalan, Croatian, Estonian, Greek, Hebrew, Italian, Occitan (post 1500), Polish, Spanish, Swedish, Turkish
  • Add appstream metainfo (Part of appstream: Add metainfo to all packages containing a .desktop file #1389)

Test Plan

Install and launch it in the XFCE VM

Checklist

  • Package was built and tested against unstable
  • This change could gainfully be listed in the weekly sync notes once merged

@malfisya malfisya requested a review from EbonJaeger December 19, 2024 23:55
@malfisya malfisya marked this pull request as ready for review December 20, 2024 00:14
EbonJaeger
EbonJaeger reviewed Dec 20, 2024
View reviewed changes
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure how much it matters, but two of the metainfo lines have a space at the end of the sentence.

packages/x/xfce4-taskmanager/files/org.xfce.taskmanager.metainfo.xml Outdated Show resolved Hide resolved
packages/x/xfce4-taskmanager/files/org.xfce.taskmanager.metainfo.xml Outdated Show resolved Hide resolved
@malfisya
xfce4-taskmanager: Update to v1.5.7
08775e7 
**Summary**

- Changes:
   - Update copyright year
   - build: Fix -Wcast-align warning from clang
   - Fix UBSan errors
   - settings-dialog: Fix XtmRefreshRate memory leak
   - Dark mode for process-statusbar
   - Dark mode for process-monitor
   - Allow multiple instances via command line option.
   - Fix blurry app icons when UI scale > 1
   - Use symbolic window picker icon in toolbar
   - Fix broken "show-legend" setting sync
   - Translation Updates: Catalan, Croatian, Estonian,
     Greek, Hebrew, Italian, Occitan (post  1500), Polish,
     Spanish, Swedish, Turkish
- Add appstream metainfo (Part of #1389)
@malfisya
Copy link
Member Author

malfisya commented Dec 20, 2024 •
edited
Loading

It doesn't matter much, but it is nice to be tidy. 😄 Fixed.

@malfisya malfisya requested a review from EbonJaeger December 20, 2024 01:09
EbonJaeger
EbonJaeger approved these changes Dec 20, 2024
View reviewed changes
Copy link
Member

@EbonJaeger EbonJaeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks! Feel free to merge it, I won't be able to tonight, and it would be great to have in for sync tomorrow with all the rest of the XFCE stuff.

@malfisya malfisya merged commit 7828176 into main Dec 20, 2024
1 check passed
@malfisya malfisya deleted the xfce-task branch December 20, 2024 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EbonJaeger EbonJaeger EbonJaeger approved these changes

Assignees
No one assigned
Labels
None yet
Projects
Solus
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@malfisya @EbonJaeger