Ensure that Keyboard shortcuts for save and download actions should not called when their respective UI buttons are disabled. #8803

pull/8868/head
Akshay Joshi 2025-06-17 16:19:41 +05:30
parent 335a39bc75
commit 7946ee78f6
2 changed files with 8 additions and 2 deletions

View File

@ -40,6 +40,7 @@ Bug fixes
| `Issue #8032 <https://github.com/pgadmin-org/pgadmin4/issues/8032>`_ - Fixed an issue where the Schema Diff Tool incorrectly reported differences due to variations in the order of the privileges.
| `Issue #8235 <https://github.com/pgadmin-org/pgadmin4/issues/8235>`_ - Fixed an issue in SQL syntax highlighting where the same color was used for both variable names and datatypes.
| `Issue #8691 <https://github.com/pgadmin-org/pgadmin4/issues/8691>`_ - Fixed an issue in the query tool where using multiple cursors to copy text resulted in only the first line being copied.
| `Issue #8803 <https://github.com/pgadmin-org/pgadmin4/issues/8803>`_ - Ensure that Keyboard shortcuts for save and download actions should not called when their respective UI buttons are disabled.
| `Issue #8808 <https://github.com/pgadmin-org/pgadmin4/issues/8808>`_ - Fixed an issue where data export using a query opened the wrong dialog type.
| `Issue #8809 <https://github.com/pgadmin-org/pgadmin4/issues/8809>`_ - Fixed an issue where data export using a query failed when the query contained a newline character.
| `Issue #8834 <https://github.com/pgadmin-org/pgadmin4/issues/8834>`_ - Fixed an issue where the Columns node was not visible under Catalog Objects.

View File

@ -367,13 +367,18 @@ export function ResultSetToolbar({query, canEdit, totalRowCount, pagination, all
{
shortcut: queryToolPref.save_data,
options: {
callback: ()=>{saveData();}
callback: ()=>{!buttonsDisabled['save-data'] && saveData();}
}
},
{
shortcut: queryToolPref.download_results,
options: {
callback: (e)=>{e.preventDefault(); downloadResult();}
callback: (e)=>{
if (!buttonsDisabled['save-result']) {
e.preventDefault();
downloadResult();
}
}
}
},
], queryToolCtx.mainContainerRef);