Blueprint: [labs/select] Provide mechanism for disabling Select handlers

[thomas-mcdonald]

I'm trying to disable a Select input... use case is that one select should be filled out before the second one. There are two pieces here - I can disable the underlying button/child element for the visual side of things but need a way to disable the handlers on the Select component.

[thomas-mcdonald]

Since I need this today, going to try writing a component that switches out the Select w/Button child for just a disabled Button if disabled={true} is passed

[llorca]

@thomas-mcdonald I think a disabled prop for Select would make sense here. Want to submit a PR for that functionality?

[thomas-mcdonald]

What I ended up creating was a frankencomponent which worked in the interim - the disabled select ended up not making UX sense in the app I was working on, so moved away from this being a direct issue (although kept the rough HOC so my Selects take text and automatically get the Button with dropdown caret as per a 'standard' browser select)

Happy to work on a PR next week, I think the only thing I wasn't sure about is naming/responsibilities - disabled suggests to me it will handle the visual aspects of the component being disabled. Maybe disableHandlers? Unsure if there is a one stone way of passing disabled={true} and pushing that through to children.

[JuoCode]

Select has popoverProps and Popover has isDisabled, you should know it.

[brieb]

Yes, as suggested, adding popoverProps={{ isDisabled: true }} should work. Though adding a top-level disabled prop would be clearer.