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

Rendered / Loaded event for Person control #3352

Open
ThomasPe opened this issue Dec 28, 2024 · 1 comment · May be fixed by #3357
Open

Rendered / Loaded event for Person control #3352

ThomasPe opened this issue Dec 28, 2024 · 1 comment · May be fixed by #3357

Comments

@ThomasPe
Copy link

Proposal: Rendered / Loaded event for Person control

Description

Since many controls load data in the background it's not obvious from the outside when a control has completed loading / rendering it's content. So an event on controls like the Person control would be helpful that indicates the control has finished updating.

Rationale

For my usecase, I need to know the longest name (and therefore max width) in a list of multiple Person controls since I need to adjust the width of some static layout that needs a fixed px width. Currently I need to get the width after a timeout and hope all users have been loaded.

Preferred Solution

An event like loaded would be great.

@musale
Copy link
Contributor

musale commented Jan 2, 2025

@ThomasPe this looks like a straightforward feature to add to all components that load data somehow. Would you be interested in contributing to it?

The components will need a custom event fired when data is loaded, updates on documentation to show this new event and an additional story for the use of the event in the playground.

This project is currently in maintenance mode, and the priority is on fixes and security issues. However, we still take any contributions from the community.

@musale musale moved this from Needs Triage 🔍 to Proposed 💡 in Graph Toolkit Jan 2, 2025
@ThomasPe ThomasPe linked a pull request Jan 3, 2025 that will close this issue
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Proposed 💡
Development

Successfully merging a pull request may close this issue.

2 participants