Skip to content

docs: fix documentation links and formatting#67661

Closed
ibrahim-hussien-dev wants to merge 1 commit intoangular:mainfrom
ibrahim-hussien-dev:docs/multiple-doc-improvements
Closed

docs: fix documentation links and formatting#67661
ibrahim-hussien-dev wants to merge 1 commit intoangular:mainfrom
ibrahim-hussien-dev:docs/multiple-doc-improvements

Conversation

@ibrahim-hussien-dev
Copy link
Contributor

This PR fixes several small documentation improvements across Angular.

Changes

  1. binding.md - Fixed broken markdown links (added missing leading /)

    • Fixed component link: (guide/components)\ → (/guide/components)\
    • Fixed directive link: (guide/directives)\ → (/guide/directives)\
    • Fixed accessibility guide link: (best-practices/a11y#...)\ → (/best-practices/a11y#...)\
  2. binding.md - Updated NOTE formatting

    • Changed from 'NOTE:' to '> Note:' for consistency with Angular documentation style
  3. ng-container.md - Added missing Dependency Injection guide link

    • Changed from plain text to markdown link: [Dependency Injection guide](/guide/di)\

All changes are minimal, safe documentation-only improvements. No functional code changes.

@pullapprove pullapprove bot requested a review from devversion March 12, 2026 22:20
@angular-robot angular-robot bot added the area: docs Related to the documentation label Mar 12, 2026
@ngbot ngbot bot added this to the Backlog milestone Mar 12, 2026
@google-cla
Copy link

google-cla bot commented Mar 12, 2026

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.

@ibrahim-hussien-dev ibrahim-hussien-dev force-pushed the docs/multiple-doc-improvements branch from 0d39bac to c4b5047 Compare March 12, 2026 22:33
- Fix broken markdown links in binding.md (missing leading /)
- Add missing link to Dependency Injection guide in ng-container.md
- Update NOTE formatting to markdown blockquote style for consistency
@ibrahim-hussien-dev ibrahim-hussien-dev force-pushed the docs/multiple-doc-improvements branch from c4b5047 to 8eca291 Compare March 12, 2026 22:34
If an element has multiple bindings for the same CSS class, Angular resolves collisions by following its style precedence order.

NOTE: Class bindings do not support space-separated class names in a single key. They also don't support mutations on objects as the reference of the binding remains the same. If you need one or the other, use the [ngClass](/api/common/NgClass) directive.
> **Note:** Class bindings do not support space-separated class names in a single key. They also don't support mutations on objects as the reference of the binding remains the same. If you need one or the other, use the [ngClass](/api/common/NgClass) directive.
Copy link
Contributor

Choose a reason for hiding this comment

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

This doesn't seem necessary; it currently works as NOTE: without any problems

Angular supports binding dynamic values into object properties and HTML attributes with square brackets.

You can bind to properties on an HTML element's DOM instance, a [component](guide/components) instance, or a [directive](guide/directives) instance.
You can bind to properties on an HTML element's DOM instance, a [component](/guide/components) instance, or a [directive](/guide/directives) instance.
Copy link
Contributor

Choose a reason for hiding this comment

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

The / is also not necessary; it is validated in both /guide/components and guide/components.

Copy link
Contributor

Choose a reason for hiding this comment

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

This is also duplicated with #67662

@JeanMeche
Copy link
Member

#67668

@JeanMeche JeanMeche closed this Mar 12, 2026
@ibrahim-hussien-dev ibrahim-hussien-dev deleted the docs/multiple-doc-improvements branch March 12, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: docs Related to the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants