Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reduce the number of events sent to the host to track focus ownership #172

Closed
danielwiehl opened this issue Sep 28, 2022 · 1 comment
Closed
Assignees
Labels
comp: microfrontend-platform performance Related to the performance of the platform.
Milestone

Comments

@danielwiehl
Copy link
Collaborator

Describe the bug

A microfrontend sends a large number of events so that the host can track focus ownership, among other things. For example, every click event is sent to the host, causing a significant communication overhead. Also, pausing execution using breakpoints can lead to errors since not receiving message confirmation in time.

Environment

  • SCION Microfrontend Platform version: 1.0.0-rc.6
@danielwiehl
Copy link
Collaborator Author

Also, if loaded in the context of an activator, consider installing only relevant platform subscriptions, e.g., no subscriptions for tracking the focus owner, ...

@danielwiehl danielwiehl self-assigned this Nov 11, 2022
@danielwiehl danielwiehl added this to the v1.0.0 milestone Nov 12, 2022
@danielwiehl danielwiehl added comp: microfrontend-platform performance Related to the performance of the platform. labels Nov 12, 2022
danielwiehl added a commit that referenced this issue Nov 17, 2022
danielwiehl added a commit that referenced this issue Nov 17, 2022
danielwiehl added a commit that referenced this issue Nov 17, 2022
This commit reduces:
- number of events for the platform to track focus ownership
- number of mouse events dispatched to the active outlet
- disables event handling in activator microfrontends

closes #172
danielwiehl added a commit that referenced this issue Nov 18, 2022
This commit reduces:
- number of events for the platform to track focus ownership
- number of mouse events dispatched to the active outlet
- disables event handling in activator microfrontends

closes #172
danielwiehl added a commit that referenced this issue Nov 18, 2022
Optimizations include:
- Reducing the number of events for the platform to track focus across microfrontends.
- Dispatching of synthetic mouse events only from inactive microfrontends.
- Delivery of synthetic mouse events to the active microfrontend only.
- Disabling event listeners that are not needed in activators.
- Listening to keyboard events only in contexts where keystrokes are registered.

closes #172
@danielwiehl danielwiehl mentioned this issue Nov 18, 2022
17 tasks
danielwiehl added a commit that referenced this issue Nov 18, 2022
Optimizations include:
- Reducing the number of events for the platform to track focus across microfrontends.
- Dispatching of synthetic mouse events only from inactive microfrontends.
- Delivery of synthetic mouse events to the active microfrontend only.
- Disabling event listeners that are not needed in activators.
- Listening to keyboard events only in contexts where keystrokes are registered.

closes #172
danielwiehl added a commit that referenced this issue Nov 21, 2022
Optimizations include:
- Reducing the number of events for the platform to track focus across microfrontends.
- Dispatching of synthetic mouse events only from inactive microfrontends.
- Delivery of synthetic mouse events to the active microfrontend only.
- Disabling event listeners that are not needed in activators.
- Listening to keyboard events only in contexts where keystrokes are registered.

closes #172
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
comp: microfrontend-platform performance Related to the performance of the platform.
Projects
None yet
Development

No branches or pull requests

1 participant