Skip to content

fix: validate prop binding with exp binding #713

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

marufrasully
Copy link
Contributor

Issue:
Currently if expression binding is used along with property binding info, property binding info was ignored (no LSP support).
Code example

<Button id="test" text="{= ${/actionButtonsInfo/midColumn/closeColumn} !== null } {parts: [' '] }"></Text>

This PR solves that issue.

Copy link

changeset-bot bot commented May 16, 2024

🦋 Changeset detected

Latest commit: 263c32b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages
Name Type
@ui5-language-assistant/vscode-ui5-language-assistant-bas-ext Patch
vscode-ui5-language-assistant Patch
@ui5-language-assistant/binding-parser Patch
@ui5-language-assistant/binding Patch
@ui5-language-assistant/xml-views-definition Patch
@ui5-language-assistant/language-server Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant