Fixed drawAttention() position to comply with user's settings #1612
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This fix is related about issue signalapp#1587 window.drawAttention() repositioned to allow complete control over notifications by settings. It now does not draw attention when the notifications are off.
|
Thanks! |
scottnonnenberg
added a commit
that referenced
this pull request
Oct 30, 2017
Emoji picker (#1608) Prevent drawAttention() when notifications are turned off - thanks @canerelci! (#1612) Support new 833 area code with update to libphonenumber (#1598) Dev: - Support for beta releases installed beside production versions (#1606) - Display of environment and app instance in title bar/about window (#1606) - Fix json parsing exceptions obscuring server errors (#1605)
|
It was my pleasure :) |
1 task
scottnonnenberg
added a commit
that referenced
this pull request
Dec 20, 2017
Note: This release is the same thing as https://github.com/WhisperSystems/Signal-Desktop/releases/tag/v1.1.0-beta.6 Listed below are the changes from the previous production release: https://github.com/WhisperSystems/Signal-Desktop/releases/tag/v1.0.41 Update to electron 1.7.9 (#1736) Support the latest phone number formats via libphonenumber update (#1899) Reduce download size by ~25MB over the previous production build (#1869) Emoji - thanks @liliakai: - Emoji picker (#1608) - Add support for Emoji 5 (#1797) Notifications: - Windows 7: Use an alternate mechanism for notifications (#1812) - Prevent drawAttention() when notifications are turned off - thanks @canerelci! (#1612) Linux: - Support for current (artful) and previous (xenial) ubuntu versions (#1856) - Fix missing application icon on some Linux distributions (#1735) - Fix issue where window would not show new message alerts on some Linux systems - thanks @cornerman (#1820) - Add .deb specific dependencies - thanks @veggiedefender (#1858) The default button is now 'later', not 'restart' in the 'update available' dialog (#1894) Make the window minimum width a little smaller - thanks @emptyflask (#1863) Intl-friendly sort order for contact lists (#1900) Fix issue where update would restore deleted windows shortcut (#1804) Fix issue where .tif file attachments could not be sent or received (#1901) (in testing) Add a tray icon to the application behind command-line argument - thanks @m-pilia (#1676) --use-tray-icon enables the tray icon --start-in-tray enables the tray icon and the application starts minimized in the tray bar (in testing) Support pass-through proxies with HTTPS_PROXY environment variable (#1878) Dev: - Display of environment and app instance in title bar/about window (#1606) - Support for beta releases installed beside production versions (#1606) - Fix json parsing exceptions obscuring server errors (#1605) - Be resilient to thrown non-errors in import process (#1737)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fix is related about issue #1587
window.drawAttention() repositioned to allow complete control over notifications by settings. It now does not draw attention when the notifications are off.
First time contributor checklist
Contributor checklist
Description
In fact, didn't do any test because I didn't set my development environment yet and the fix is too easy.