add comments accessible view

[meganrogge]

fixes #203928

[alexr00]

I haven't tested this, but it looks reasonable. I left a few comments so I disabled auto-merge.

It looks like this is the established pattern for adding accessible views, but right now the accessible view contribution needs to know everything about every accessible view. It might be cleaner to instead have each component contribute an accessible view to prevent accessible view code from needing to know about the whole world.

[meganrogge]

I haven't tested this, but it looks reasonable. I left a few comments so I disabled auto-merge.

It looks like this is the established pattern for adding accessible views, but right now the accessible view contribution needs to know everything about every accessible view. It might be cleaner to instead have each component contribute an accessible view to prevent accessible view code from needing to know about the whole world.

I was thinking the same thing while implementing this. I've created an issue for debt week, thanks.

I will address your other comments in a follow up PR.