Focus rings are incompatible with grid layout |
|||
Issue descriptionCalling "FocusRing::Install" on a view that uses a grid layout manager DCHECKs because grid layout expects to manage all subviews of the view.
,
Jul 21
,
Nov 21
**UI mass Triage** Adding labels for expert review. |
|||
►
Sign in to add a comment |
|||
Comment 1 by ellyjo...@chromium.org
, May 8 2018Labels: MacViews-Cleanup M-X
Owner: ellyjo...@chromium.org
Status: Assigned (was: Available)
I think the right path forward here is for FocusRing to not be a View, since FocusRing doesn't fit into the LayoutManager model (in which a LayoutManager is expected to lay out all children of a View) at all. Perhaps we need to introduce a new ancillary concept ("Decoration"?) which has no independent layout, can't receive events, can't have children, &c &c but is just invoked during view painting. I'll take this one on.