Skip to content

Added balloon notifications with tray icon#69

Merged
jushar merged 8 commits into
multitheftauto:masterfrom
Necktrox:feature/balloon-notifications
Sep 17, 2016
Merged

Added balloon notifications with tray icon#69
jushar merged 8 commits into
multitheftauto:masterfrom
Necktrox:feature/balloon-notifications

Conversation

@Necktrox

Copy link
Copy Markdown

Note

This pull request has been re-created from a clean branch from my fork of the repository. The code has been updated to be compatible with the recent directory changes of the project. Furthermore, the notifications are now opt-in and a new function areTrayNotificationsEnabled has been added. There are no language localization string updates in this PR. You can read more in the discussion of the old pull request #35.

Description

This pull requests aims to add notification functionality in the tray icon area on Windows task bar. It implements an unique class to manage creation, deletion of the tray icon and to show balloon notifications. The balloon notifications support 4 types of icons for the message and it's also possible to silence the notification sound.

Features

  • Tray icon
  • Balloon notifications
  • 4 notification icons (default, info, warning, error)
  • Enable/disable notification sound
  • Spam prevention

Preview screenshots

Preview for tray notification
Preview for modified settings
Preview for notification balloon

Notes

Any suggestions are welcome.

@qaisjp qaisjp added the enhancement New feature or request label Sep 14, 2016

@qaisjp qaisjp left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please move CLuaFunctionDefs.Notification.cpp to CLuaUtilDefs.cpp

@jushar jushar merged commit dd40e98 into multitheftauto:master Sep 17, 2016
@Necktrox Necktrox deleted the feature/balloon-notifications branch March 3, 2017 11:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants