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.

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

Fix : systray option visible mac #293

Merged

Conversation

louib
Copy link
Member

@louib louib commented Feb 11, 2017

Not all the systray settings were hidden on MacOs.

How Has This Been Tested?

Unit tests.

Screenshots (if appropriate):

Before
screen shot 2017-02-11 at 1 55 16 pm

After
screen shot 2017-02-11 at 1 51 43 pm

Types of changes

  • ✅ Bug fix

Checklist:

  • ✅ I have read the CONTRIBUTING document.
  • ✅ My code follows the code style of this project.
  • ✅ All new and existing tests passed.

@phoerious phoerious merged commit ef53600 into keepassxreboot:release/2.1.2 Feb 11, 2017
@phoerious phoerious added this to the v2.1.2 milestone Feb 11, 2017
@louib louib deleted the fix/systrayOptionVisibleMac branch February 11, 2017 19:51
phoerious added a commit that referenced this pull request Feb 17, 2017
- Ask for save location when creating a new database [#302]
- Remove Libmicrohttpd dependency to clean up the code and ensure better OS X compatibility [#317, #265]
- Prevent Qt from degrading Wifi network performance on certain platforms [#318]
- Visually refine user interface on OS X and other platforms [#299]
- Remove unusable tray icon setting on OS X [#293]
- Fix compositing glitches on Ubuntu and prevent flashing when minimizing to the tray at startup [#307]
- Fix AppImage tray icon on Ubuntu [#277, #273]
- Fix global menu disappearing after restoring KeePassXC from the tray on Ubuntu [#276]
- Fix result order in entry search [#320]
- Enable HiDPI scaling on supported platforms [#315]
- Remove empty directories from installation target [#282]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants