Skip to content

fix(UIShell): various fixes to avoid null pointer assertions#14144

Merged
tay1orjones merged 2 commits into
carbon-design-system:mainfrom
tay1orjones:fix-uishell-bugs
Jul 6, 2023
Merged

fix(UIShell): various fixes to avoid null pointer assertions#14144
tay1orjones merged 2 commits into
carbon-design-system:mainfrom
tay1orjones:fix-uishell-bugs

Conversation

@tay1orjones

Copy link
Copy Markdown
Member

Surfaced in office hours this morning, a few bugs relating to UIShell and TextArea

Changelog

Changed

  • protect against null pointer exceptions with optional chaining
  • only pass handleSwitcherItemFocus to components that accept that prop

@netlify

netlify Bot commented Jul 6, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-components-react ready!

Name Link
🔨 Latest commit d12fb8c
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/64a6eb73c5c0c100075b63bb
😎 Deploy Preview https://deploy-preview-14144--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify

netlify Bot commented Jul 6, 2023

Copy link
Copy Markdown

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit d12fb8c
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/64a6eb73b3778900088a2027
😎 Deploy Preview https://deploy-preview-14144--carbon-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@francinelucca francinelucca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is it expected for the switcher not to show any options here? looks like the stories changed a bit so not sure if that's intended
image
I'm also not able to open the switcher on this one
image

@tay1orjones

Copy link
Copy Markdown
Member Author

@francinelucca sorry I forgot the return statements, should be fixed now 👍

@tay1orjones tay1orjones requested a review from francinelucca July 6, 2023 16:31

@francinelucca francinelucca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@tw15egan tw15egan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍 ✅

@tay1orjones tay1orjones merged commit 330b4bf into carbon-design-system:main Jul 6, 2023
@tay1orjones tay1orjones self-assigned this Jul 7, 2023
francinelucca pushed a commit to francinelucca/carbon that referenced this pull request Jul 10, 2023
…design-system#14144)

* fix(UIShell): various fixes to avoid null pointer assertions

* fix(UIShell): various fixes to avoid null pointer assertions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants