-
-
Notifications
You must be signed in to change notification settings - Fork 836
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: empty string displayed as SelectDropdown title #3773
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@imorland was his tested locally and with the affected extensions? I haven't tested it myself
Could we also document what this issue was affecting (e.g., an example of an extension where this was a problem)? |
Yes, tested locally - all good |
Co-authored-by: David Wheatley <david@davwheat.dev>
* fix: empty string displayed as SelectDropdown title * chore: remove import * chore: ts-ignore * Update framework/core/js/src/common/components/SelectDropdown.tsx Co-authored-by: David Wheatley <david@davwheat.dev> --------- Co-authored-by: David Wheatley <david@davwheat.dev>
* fix: empty string displayed as SelectDropdown title * chore: remove import * chore: ts-ignore * Update framework/core/js/src/common/components/SelectDropdown.tsx Co-authored-by: David Wheatley <david@davwheat.dev> --------- Co-authored-by: David Wheatley <david@davwheat.dev>
* fix: empty string displayed as SelectDropdown title * chore: remove import * chore: ts-ignore * Update framework/core/js/src/common/components/SelectDropdown.tsx Co-authored-by: David Wheatley <david@davwheat.dev> --------- Co-authored-by: David Wheatley <david@davwheat.dev>
* fix: empty string displayed as SelectDropdown title * chore: remove import * chore: ts-ignore * Update framework/core/js/src/common/components/SelectDropdown.tsx Co-authored-by: David Wheatley <david@davwheat.dev> --------- Co-authored-by: David Wheatley <david@davwheat.dev>
* fix: empty string displayed as SelectDropdown title * chore: remove import * chore: ts-ignore * Update framework/core/js/src/common/components/SelectDropdown.tsx Co-authored-by: David Wheatley <david@davwheat.dev> --------- Co-authored-by: David Wheatley <david@davwheat.dev>
Reverts the change to using
extractText
introduced in #3608Whilst using
extractText
, the button title is rendered as an empty string. This PR reverts that change so that expected behaviour is restored.Changes proposed in this pull request:
Reviewers should focus on:
Screenshot
Necessity
Confirmed
composer test
).Required changes: