Ensure hiding the context menu directly before its menu item actions

Closes af3#4321 (closed)

Issue-Ref: 4321 Issue-Url: af3#4321 (closed)

Signed-off-by: Sebastian Bergemann bergemann@fortiss.org

changed milestone to %MbSE-Training

assigned to @bergemann

requested review from @munaro

added 1 commit

• e5336269 - GREEN

Compare with previous version

Code is GREEN, however the behavior of the dialog window has been affected. See here.

added 1 commit

• 29985a9c - Undo the top flag for reuse library windows due to problems on linux

Compare with previous version

@munaro Commented issue is solved. You can quickly check this and then forward it to Andreas/Simon for merge.

assigned to @munaro and unassigned @bergemann

@bergemann The dialog should check whether the component to reuse still exists when adding it to the library. As the windows is no longer on top, one can delete the component while the dialog is open, causing a NPE.

@bergemann This solution does not work for the reuse library's context menu. There, the context menu stays even after the "Delete Library" entry has been clicked:

assigned to @bergemann and unassigned @munaro

added 1 commit

• 19b44ac5 - Add settings

Compare with previous version

added 12 commits

• 19b44ac5...1826f614 - 11 commits from branch master
• 21e21714 - Merge branch 'master' of https://git.fortiss.org/af3/kernel.git into 4321

Compare with previous version

added 8 commits

• 21e21714...bf673793 - 7 commits from branch master
• 29b84aa5 - Merge branch 'master' of https://git.fortiss.org/af3/kernel.git into 4321

Compare with previous version

added 1 commit

• bc4b16ce - Fix NPE when reuse element is deleted during any reuse action

Compare with previous version

mentioned in issue af3#4326

resolved all threads

resolved all threads

All remarks were addressed. New stuff is YELLOW for re-review @munaro

assigned to @munaro and unassigned @bergemann