Skip to content

Commit

Permalink
Bug fix, ensures search bar is cleared and hidden on tab change and c…
Browse files Browse the repository at this point in the history
…onnections removed (#132)
  • Loading branch information
GrantComm authored Sep 11, 2024
1 parent 1678664 commit 368a497
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
1 change: 1 addition & 0 deletions ui/command_tab_view.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -147,6 +147,7 @@ void CommandTabView::OnSearchBarVisibilityChange(bool isHidden)
if (isHidden)
{
m_search_bar->clearSearch();
m_search_bar->hide();
m_search_trigger_button->show();
DisconnectSearchBar();
}
Expand Down
8 changes: 8 additions & 0 deletions ui/main_window.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1012,6 +1012,8 @@ void MainWindow::CreateShortcuts()
m_event_search_bar->clearSearch();
m_event_search_bar->hide();
m_search_trigger_button->show();

DisconnectSearchBar();
});

// Overview Shortcut
Expand Down Expand Up @@ -1178,6 +1180,7 @@ void MainWindow::OnCommandBufferSearchBarVisibilityChange(bool isHidden)
if (isHidden)
{
m_event_search_bar->clearSearch();
m_event_search_bar->hide();
m_search_trigger_button->show();

DisconnectSearchBar();
Expand All @@ -1195,6 +1198,11 @@ void MainWindow::OnTabViewChange()
m_event_search_bar->clearSearch();
m_event_search_bar->hide();
m_search_trigger_button->show();
DisconnectSearchBar();
}
else if (currentIndex != m_command_view_tab_index)
{
m_command_tab_view->OnSearchBarVisibilityChange(true);
}
}

Expand Down

0 comments on commit 368a497

Please sign in to comment.