docs: improve clarity in dependency injection guides#68376
Open
BHUVANSH855 wants to merge 2 commits intoangular:mainfrom
Open
docs: improve clarity in dependency injection guides#68376BHUVANSH855 wants to merge 2 commits intoangular:mainfrom
BHUVANSH855 wants to merge 2 commits intoangular:mainfrom
Conversation
JeanMeche
reviewed
Apr 26, 2026
| # Debugging and troubleshooting dependency injection | ||
|
|
||
| Dependency injection (DI) issues typically stem from configuration mistakes, scope problems, or incorrect usage patterns. This guide helps you identify and resolve common DI problems that developers encounter. | ||
| Dependency injection (DI) issues typically stem from configuration mistakes, scope problems, or incorrect usage patterns. This guide helps you identify and resolve common DI problems. |
Member
There was a problem hiding this comment.
We'd like to keep the end of the sentence.
JeanMeche
reviewed
Apr 26, 2026
| # Hierarchical injectors | ||
|
|
||
| This guide provides in-depth coverage of Angular's hierarchical dependency injection system, including resolution rules, modifiers, and advanced patterns. | ||
| This guide explains Angular's hierarchical dependency injection system, including resolution rules, modifiers, and advanced patterns. |
JeanMeche
reviewed
Apr 26, 2026
|
|
||
| - One of these reference is in the _type position_-- that is, it specifies `LibHeader` as a type: `readonly header: Signal<LibHeader|undefined>`. | ||
| - One of these references is in the _type position_— that is, it specifies `LibHeader` as a type: `readonly header: Signal<LibHeader|undefined>`. | ||
| - The other reference is in the _value position_-- that is, `LibHeader` is the value passed into the `contentChild` function: `contentChild(LibHeader)`. |
Member
There was a problem hiding this comment.
If we change the dash style we should be consistent on it.
Suggested change
| - The other reference is in the _value position_-- that is, `LibHeader` is the value passed into the `contentChild` function: `contentChild(LibHeader)`. | |
| - The other reference is in the _value position_- that is, `LibHeader` is the value passed into the `contentChild` function: `contentChild(LibHeader)`. |
Contributor
Author
|
Thanks for the feedback @JeanMeche I've applied the suggested changes and reverted the sections where needed. Let me know if any further changes are required. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Some dependency injection guides contain verbose phrasing, passive voice, and minor grammar inconsistencies, which can reduce clarity and readability.
Issue Number: N/A
What is the new behavior?
Updated files:
Does this PR introduce a breaking change?
Other information
These changes are limited to documentation improvements only. No functional or behavioral changes are introduced.