Skip to content

fix(50224): Intellisense for strings within a type's Union doesn't work properly for JSX - #50231

Merged
Nathan Shively-Sanders (sandersn) merged 2 commits into
microsoft:mainfrom
a-tarasyuk:fix/50224
Aug 23, 2022
Merged

fix(50224): Intellisense for strings within a type's Union doesn't work properly for JSX#50231
Nathan Shively-Sanders (sandersn) merged 2 commits into
microsoft:mainfrom
a-tarasyuk:fix/50224

Conversation

@a-tarasyuk

Copy link
Copy Markdown
Contributor

Fixes #50224

@typescript-bot TypeScript Bot (typescript-bot) added the For Backlog Bug PRs that fix a backlog bug label Aug 9, 2022

@sandersn Nathan Shively-Sanders (sandersn) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The new fallthrough looks wrong to me.

Comment thread src/services/stringCompletions.ts
Comment thread src/services/stringCompletions.ts Outdated
Comment thread tests/cases/fourslash/stringLiteralCompletionsInJsxAttributeInitializer.ts Outdated
@a-tarasyuk

Copy link
Copy Markdown
Contributor Author

Surprisingly, when I re-requested a review from Nathan Shively-Sanders (@sandersn), Github removed Gabriela Araujo Britto (@gabritto) from the reviewers 😟

@sandersn
Nathan Shively-Sanders (sandersn) merged commit 44ce3cf into microsoft:main Aug 23, 2022
@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

For Backlog Bug PRs that fix a backlog bug

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Intellisense for strings within a type's Union doesn't work properly for JSX

3 participants