mirror of
https://github.com/Azure/cosmos-explorer.git
synced 2024-11-25 15:06:55 +00:00
Focus order not logical on invoking entities in table (#1505)
* arialabel has been added to close button of invitational youtube video * heading role has been addedd and tag has been changed to h1 * outline has been restored to choose columns link in entities page * Update QuickstartCarousel.tsx * Update SplashScreen.tsx * Update TableEntity.tsx * outline for edit entity has been added on focus * keyboard accessibility added to rows in table entities * learn more link under analytical store * Column header is populated with text * aria label has been changed for the screen readers to read placeholder text along with label text * Update queryBuilder.less * Update TableEntity.tsx * Update ThroughputInputAutoPilotV3Component.tsx * Update ThroughputInputAutoPilotV3Component.tsx * Update ThroughputInputAutoPilotV3Component.test.tsx.snap * Update ThroughputInput.less * Update PanelComponent.less * Update DataTableBindingManager.ts * Update AddCollectionPanel.tsx * Update AddCollectionPanel.test.tsx.snap * spec.ts files updated for the tests * update to container.spec files * info button in stats tab is made accessible with keyboard * focus order after invoking entities has been made to move logicallt from the entities tab instead of the table getting focus * Update AddCollectionPanel.tsx * Update AddCollectionPanel.test.tsx.snap * Update QueryTabComponent.tsx * Update container.spec.ts * Update container.spec.ts * Update container.spec.ts * Update container32.spec.ts * Update container.spec.ts * Update container.spec.ts * Update DataTableBindingManager.ts
This commit is contained in:
parent
a3bd126a21
commit
e48402ae1b
@ -1,15 +1,14 @@
|
||||
import * as ko from "knockout";
|
||||
import * as _ from "underscore";
|
||||
|
||||
import QueryTablesTab from "../../Tabs/QueryTablesTab";
|
||||
import * as Constants from "../Constants";
|
||||
import * as ViewModels from "../../../Contracts/ViewModels";
|
||||
import * as Entities from "../Entities";
|
||||
import * as Utilities from "../Utilities";
|
||||
import * as DataTableBuilder from "./DataTableBuilder";
|
||||
import DataTableOperationManager from "./DataTableOperationManager";
|
||||
import * as DataTableOperations from "./DataTableOperations";
|
||||
import QueryTablesTab from "../../Tabs/QueryTablesTab";
|
||||
import TableEntityListViewModel from "./TableEntityListViewModel";
|
||||
import * as Utilities from "../Utilities";
|
||||
import * as Entities from "../Entities";
|
||||
|
||||
/**
|
||||
* Custom binding manager of datatable
|
||||
@ -41,7 +40,6 @@ function bindDataTable(element: any, valueAccessor: any, allBindings: any, viewM
|
||||
|
||||
createDataTable(0, tableEntityListViewModel, queryTablesTab); // Fake a DataTable to start.
|
||||
$(window).resize(updateTableScrollableRegionMetrics);
|
||||
operationManager.focusTable(); // Also selects the first row if needed.
|
||||
}
|
||||
|
||||
function onTableColumnChange(enablePrompt: boolean = true, queryTablesTab: QueryTablesTab) {
|
||||
@ -364,10 +362,6 @@ function updateDataTableFocus(queryTablesTabId: string): void {
|
||||
if ($activeElement.is(".sorting_asc") || $activeElement.is(".sorting_desc")) {
|
||||
// If table header is selected, focus is shifted to the selected element as part of accessibility
|
||||
$activeElement && $activeElement.focus();
|
||||
} else {
|
||||
// If some control is active, we don't give focus back to the table,
|
||||
// just select the first row if needed (empty selection).
|
||||
operationManager.selectFirstIfNeeded();
|
||||
}
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user