Skip to content

fix: docs step 6 property binding#60630

Closed
EriksonBahr wants to merge 1 commit intoangular:mainfrom
EriksonBahr:patch-2
Closed

fix: docs step 6 property binding#60630
EriksonBahr wants to merge 1 commit intoangular:mainfrom
EriksonBahr:patch-2

Conversation

@EriksonBahr
Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.dev application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@google-cla
Copy link
Copy Markdown

google-cla bot commented Mar 30, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@pullapprove pullapprove bot requested a review from kirjs March 30, 2025 19:47
@thePunderWoman thePunderWoman added action: review The PR is still awaiting reviews from at least one requested reviewer target: patch This PR is targeted for the next patch release area: docs Related to the documentation merge: fix commit message labels Mar 31, 2025
@ngbot ngbot bot added this to the Backlog milestone Mar 31, 2025
Copy link
Copy Markdown
Contributor

@thePunderWoman thePunderWoman left a comment

Choose a reason for hiding this comment

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

LGTM

@thePunderWoman thePunderWoman added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Mar 31, 2025
@thePunderWoman thePunderWoman removed the request for review from kirjs March 31, 2025 17:02
thePunderWoman pushed a commit that referenced this pull request Mar 31, 2025
@thePunderWoman
Copy link
Copy Markdown
Contributor

This PR was merged into the repository by commit a7d1293.

The changes were merged into the following branches: main, 19.2.x

@csvn
Copy link
Copy Markdown
Contributor

csvn commented Apr 3, 2025

Is this PR really correct? If I try to add <div [contentEditable]="{{isEditable}}"></div> in the tutorial editor, I get an error:

image

I think interpolation and binding are mutually exclusive? Either use [] to bind an expression to a property, or use {{}} to have dynamic parts in a string.

<div contentEditable="{{isEditable}}"></div>
<div [contentEditable]="isEditable"></div>

@JeanMeche
Copy link
Copy Markdown
Member

Yes, a fix already is already pending #60725.

@csvn
Copy link
Copy Markdown
Contributor

csvn commented Apr 3, 2025

@JeanMeche ahh, sorry, didn't see that PR. Thanks for mentioning it 🙂

@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators May 4, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: docs Related to the documentation target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants