# Midikeyboard component when pressing MenuBarModel Item or any type of menu item note doesn't turn off
**URL:** https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581
**Category:** General JUCE discussion
**Created:** [July 29, 2016, 3:21am UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581 "2016-07-29T03:21:51Z")
**Posts on this page:** 11
**Page:** 1
### Author:  [@marvinh](https://forum.juce.com/u/marvinh)
#### Post date: [July 29, 2016, 3:21am UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/1 "2016-07-29T03:21:51Z")
I have have functions checking whether the focus is gained or lost but it makes no difference the notes hang when pressing menu bar items
---
### Author:  [@fr810](https://forum.juce.com/u/fr810)
#### Post date: [July 29, 2016, 2:19pm UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/2 "2016-07-29T14:19:50Z")
How are you activating the keys? Are you clicking on them or are they coming from a midi device? I’ve tried the latter and can’t seem to reproduce your bug.
---
### Author:  [@marvinh](https://forum.juce.com/u/marvinh)
#### Post date: [July 29, 2016, 3:16pm UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/3 "2016-07-29T15:16:57Z")
Open the plugin host demo hold a note with your computer keyboard and any vst connected and click a menu bar model button let go of the note it hangs
---
### Author:  [@marvinh](https://forum.juce.com/u/marvinh)
#### Post date: [July 29, 2016, 3:38pm UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/4 "2016-07-29T15:38:38Z")
Same thing happens with tracktion
---
### Author:  [@marvinh](https://forum.juce.com/u/marvinh)
#### Post date: [July 29, 2016, 10:46pm UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/5 "2016-07-29T22:46:30Z")
Fabian this is on OS X on Windows the focus lost and focus gain is being called which is a good thing,but not on OS X
---
### Author:  [@fr810](https://forum.juce.com/u/fr810)
#### Post date: [August 2, 2016, 10:38am UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/6 "2016-08-02T10:38:15Z")
OK I can re-produce this now but I’m not really sure how best to fix this (ideas welcome!). If you make the menu bar non-native (as it is on Windows) then the focus will obviously be lost correctly. I can easily have JUCE do the same for native mac menus: simply replace lines [juce\_mac\_MainMenu.mm:731](https://github.com/julianstorer/JUCE/blob/bf582f6c5bd71703517baf541e6a025318c09345/modules/juce_gui_basics/native/juce_mac_MainMenu.mm#L731) to [juce\_mac\_MainMenu.mm:735](https://github.com/julianstorer/JUCE/blob/bf582f6c5bd71703517baf541e6a025318c09345/modules/juce_gui_basics/native/juce_mac_MainMenu.mm#L735) with:
```
if (isTracking)
{
const int numPeers = ComponentPeer::getNumPeers();
for (int i = 0; i < numPeers; ++i)
if (ComponentPeer* peer = ComponentPeer::getPeer (i))
peer->handleFocusLoss();
}
else
{
if (menuHandler->defferedUpdateRequested)
{
menuHandler->defferedUpdateRequested = false;
menuHandler->menuBarItemsChanged (menuHandler->currentModel);
}
}
```
However, this is probably not what most JUCE apps want as the focus of all components will always be lost when using the main menu bar. This is normal on Windows, but on mac it’s normal for the currently focused component to keep it’s focus when accessing the main menu bar. Try this on any native OS X text edit field and access the menu bar.
To be absolutely sure, I implemented a very basic “MidiKeyboardComponent” with native OS X code. Similar to JUCE’s MidiKeyboardComponent, it will reset the state if the focus is lost (see code below, it derives from NSButton). It behaves the same way as JUCE’s MidiKeyboardComponent, i.e. the keys become stuck.
I think we might need to add a new callback to JUCE which tells components if the main menu bar was accessed. What do you think?
```
@implementation NativeFakeMidiKeyboardComponent
- (id) initWithFrame:(NSRect)frameRect
{
self = [super initWithFrame:frameRect];
return self;
}
- (BOOL)canBecomeKeyView
{
return YES;
}
- (void)drawRect:(NSRect)dirtyRect {
[super drawRect:dirtyRect];
}
- (void)keyDown:(NSEvent *)theEvent
{
if ([[theEvent characters] isEqual:@"g"])
[self setFloatValue:1.0f];
else
[super keyDown:theEvent];
}
- (void)keyUp:(NSEvent *)theEvent
{
if ([[theEvent characters] isEqual:@"g"])
[self setFloatValue:0.0f];
else
[super keyUp:theEvent];
}
- (BOOL)becomeFirstResponder
{
return YES;
}
- (BOOL)resignFirstResponder
{
[self setFloatValue:0.0f];
return YES;
}
@end
```
---
### Author:  [@marvinh](https://forum.juce.com/u/marvinh)
#### Post date: [August 2, 2016, 8:18pm UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/7 "2016-08-02T20:18:37Z")
Thanks for info Fabian. What ever you think is the best option.
My solution. I ended up having to make another thread for global key events setting off note on note off cause this got ride of my MenuBarModel issue as well as other focusing issues, but again it’s not full proof, and probably not practical.
I get access violations sometimes for the global KeyPress, and the thread is always on so you will hear notes playing when truly out of focus of the TopLevelWindow at least for Windows haven’t tested this OS X.
Kind of ridiculous, but my thoughts were this way nothing has to have focus and the computer midi will always work when the app is running. Maybe have a button to toggle it on and off. Still working on refining this method.
---
### Author:  [@fr810](https://forum.juce.com/u/fr810)
#### Post date: [August 3, 2016, 8:09am UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/8 "2016-08-03T08:09:29Z")
If you use the code modifications that I’ve suggested above (replacing lines juce\_mac\_MainMenu.mm:731 to juce\_mac\_MainMenu.mm:735), does this work for you in principle? Can you quickly test it without your workarounds? If this works then I’ll add this to JUCE as a flag which you can set for a component, i.e. losesFocusOnMainMenuBarAccess or something similar.
---
### Author:  [@marvinh](https://forum.juce.com/u/marvinh)
#### Post date: [August 3, 2016, 4:38pm UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/9 "2016-08-03T16:38:19Z")
It works it stops the notes! Thanks Fabian!
---
### Author:  [@fr810](https://forum.juce.com/u/fr810)
#### Post date: [August 4, 2016, 11:50am UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/10 "2016-08-04T11:50:14Z")
OK I’ve fixed this on the develop branch. The MIDIKeyboardComponent will now work correctly.
---
### Author:  [@fr810](https://forum.juce.com/u/fr810)
#### Post date: [August 9, 2016, 8:29am UTC](https://forum.juce.com/t/midikeyboard-component-when-pressing-menubarmodel-item-or-any-type-of-menu-item-note-doesnt-turn-off/18581/11 "2016-08-09T08:29:56Z")
Just a heads up. I’ve reworked the fix a bit. Adding a new flag to Component just to fix this problem seemed a bit excessive. I’ve now added a new listener callback to MenuBarModel which you can use when the user accesses the menu bar (MenuBarComponent or the Mac’s main menu bar). Look at the plugin host demo to see how to use it. Sorry for the inconvenience on this.