Select group label not required, but without label it breaks the UI
See original GitHub issueDescribe the issue. What is the expected and unexpected behavior?
When using select with SelectGroup the label is marked as optional, yet if I don’t pass it it breaks the whole UI with error message Uncaught (in promise) TypeError: Cannot read property 'replace' of undefined
Please provide the steps to reproduce. Feel free to link CodeSandbox or another tool. Remove label in PF examples and it breaks the component.
Is this a bug or enhancement? If this issue is a bug, is this issue blocking you or is there a work-around? Rather critical bug, we can’t properly upgrade to new PF because of this bug.
What is your product and what release version are you targeting? cloud.redhat.com
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (5 by maintainers)
Top Results From Across the Web
Remove Label on Material-UI Select Text Field - Stack Overflow
What I want to do is remove the label entirely and replace that with a blue border. Gender Field, male selected. Here is...
Read more >Labeling Controls | Web Accessibility Initiative (WAI) - W3C
Provide labels to identify all form controls, including text fields, checkboxes, radio buttons, and drop-down menus. In most cases, this is done by...
Read more >Form <input> elements must have labels | Axe Rules
These inputs are hidden and unavailable for user input. They therefore need no label. When adding labels, be sure to avoid the following:...
Read more >Labels - GitLab Docs
The project must have no labels present. To add the default labels to the project: On the top bar, select Main menu >...
Read more >Known issues with sensitivity labels in Office - Microsoft Support
This document details the known issues and workarounds with the sensitivity labeling feature in Office and will be kept updated as new ...
Read more >
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
+1 adding the null check
Making it required would be breaking, and I’m not sure it needs to be required. It definitely needs to not break when left out though
@kmcfaul thank you for the fix!