Skip to content

Conversation

@oweeye
Copy link

@oweeye oweeye commented Nov 7, 2025

Changes

  • Replace deprecated Indicator.set_icon() with set_icon_full()
  • Use keyword arguments for Gtk.MenuItem() constructors
  • Fixes 12 PyGTKDeprecationWarnings and 1 DeprecationWarning

Details

This PR modernizes the code to be compatible with current PyGObject/GTK3 standards by:

  1. AppIndicator icon: Changed from set_icon() to set_icon_full() with proper description
  2. MenuItem constructors: Updated all 11 menu items to use keyword argument label= instead of positional arguments

Testing

  • Tested with Python 3 and GTK 3.0
  • No deprecation warnings are shown
  • Application runs without errors
  • All menu functionality works as expected

Compatibility

  • Works with both AppIndicator3 and AyatanaAppIndicator3
  • Maintains backward compatibility with existing functionality"

sunweaver and others added 5 commits June 27, 2022 22:25
Features:
- Notes are moved to archive instead of permanent deletion
- Configurable retention period (default: 30 days, 0 = forever)
- Optional delete confirmation dialog (disabled by default)
- Archive viewer with restore and permanent delete options
- Auto-cleanup of old archived notes on startup

Changes:
- stickynotes/info.py: Add default settings for archive
- stickynotes/backend.py: Implement archive storage and management
- stickynotes/gui.py: Add ArchiveDialog and settings UI
- indicator-stickynotes.py: Add 'Archive' menu item

Closes: Note recovery feature request
Add archive functionality for deleted notes
indicator-stickynotes.py: Add support for Ayatana AppIndicator.
- Replace deprecated Indicator.set_icon() with set_icon_full()
- Use keyword arguments for Gtk.MenuItem() constructors
- Fixes 12 PyGTKDeprecationWarnings and 1 DeprecationWarning

This makes the code compatible with modern PyGObject/GTK3 standards.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants