Skip to content

Commit

Permalink
fix(popup): applying focus twice (SAP#10556)
Browse files Browse the repository at this point in the history
Applying focus twice might result into an issue if navigation occurs before promise for second focus is resolved.
  • Loading branch information
nnaydenow authored Jan 20, 2025
1 parent ae56a42 commit 27721a9
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions packages/main/src/Popup.ts
Original file line number Diff line number Diff line change
Expand Up @@ -326,9 +326,6 @@ abstract class Popup extends UI5Element {

await renderFinished();

// initial focus, if focused element is dynamically created
await this.applyInitialFocus();

if (this.isConnected) {
this.fireDecoratorEvent("open");
}
Expand Down

0 comments on commit 27721a9

Please sign in to comment.