more PR feedback

Signed-off-by: Louise Poubel <[email protected]>
gazebosim · Oct 1, 2021 · bb14e9a · bb14e9a
1 parent d03f333
commit bb14e9a
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/GuiEvents.cc b/src/GuiEvents.cc
@@ -120,19 +120,19 @@ class ignition::gui::events::DropOnScene::Implementation
 
 class ignition::gui::events::ScrollOnScene::Implementation
 {
- /// \brief Mouse event
+ /// \brief Mouse event with scroll information.
  public: common::MouseEvent mouse;
 };
 
 class ignition::gui::events::DragOnScene::Implementation
 {
- /// \brief The 2D point over which the user is hovering.
+ /// \brief Mouse event with drag information.
  public: common::MouseEvent mouse;
 };
 
 class ignition::gui::events::MousePressOnScene::Implementation
 {
- /// \brief Mouse event
+ /// \brief Mouse event with press information.
  public: common::MouseEvent mouse;
 };
 

diff --git a/src/plugins/minimal_scene/MinimalScene.cc b/src/plugins/minimal_scene/MinimalScene.cc
@@ -488,7 +488,7 @@ void IgnRenderer::BroadcastKeyRelease()
 /////////////////////////////////////////////////
 void IgnRenderer::BroadcastKeyPress()
 {
- if (this->dataPtr->keyEvent.Type() == common::KeyEvent::PRESS)
+ if (this->dataPtr->keyEvent.Type() != common::KeyEvent::PRESS)
  return;
 
  events::KeyPressOnScene keyPress(this->dataPtr->keyEvent);