-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Open
Labels
A-UIGraphical user interfaces, styles, layouts, and widgetsGraphical user interfaces, styles, layouts, and widgetsC-BenchmarksStress tests and benchmarks used to measure how fast things areStress tests and benchmarks used to measure how fast things areD-StraightforwardSimple bug fixes and API improvements, docs, test and examplesSimple bug fixes and API improvements, docs, test and examplesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!This issue is ready for an implementation PR. Go for it!X-UncontroversialThis work is generally agreed uponThis work is generally agreed upon
Description
Yeah I've only had a quick look at this but there already two approvals and this makes sense to me. Not sure about the observer overhead concerns. If we could add some concrete benchmarks in a follow up, like a a many_ui_observers or something, that should help.
Originally posted by @ickshonpe in #22933 (review)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-UIGraphical user interfaces, styles, layouts, and widgetsGraphical user interfaces, styles, layouts, and widgetsC-BenchmarksStress tests and benchmarks used to measure how fast things areStress tests and benchmarks used to measure how fast things areD-StraightforwardSimple bug fixes and API improvements, docs, test and examplesSimple bug fixes and API improvements, docs, test and examplesS-Ready-For-ImplementationThis issue is ready for an implementation PR. Go for it!This issue is ready for an implementation PR. Go for it!X-UncontroversialThis work is generally agreed uponThis work is generally agreed upon
Type
Projects
Status
Needs SME Triage