angular / angular

Deliver web apps with confidence 🚀
https://angular.dev
MIT License
95.97k stars 25.37k forks source link

docs: The title of the section on the tutorial page does not match the content #58036

Closed AnkitSharma-007 closed 5 days ago

AnkitSharma-007 commented 1 week ago

Describe the problem that you experienced

When navigating to this page: https://angular.dev/tutorials/first-app/11-details-page, I noticed that the title of the first section reads "Create a new service for your app." However, the content of this section doesn't discuss creating a service. Instead, it talks about fetching the dynamic "id" value from the route.

It seems like the title may need updating to reflect the actual content.

Enter the URL of the topic with the problem

https://angular.dev/tutorials/first-app/11-details-page

Describe what you were looking for in the documentation

No response

Describe the actions that led you to experience the problem

No response

Describe what you want to experience that would fix the problem

No response

Add a screenshot if that helps illustrate the problem

image

If this problem caused an exception or error, please paste it here

No response

If the problem is browser-specific, please specify the device, OS, browser, and version

No response

Provide any additional information here in as much as detail as you can

No response

Codebreaker42 commented 1 week ago

I agree with this observation. The current title "Create a new service for your app" doesn't accurately reflect the content of this section.

The actual content focuses on:

  1. Using dynamic routing in Angular
  2. Adding an anchor tag with a routerLink directive
  3. Explaining how to use routerLink for creating dynamic links

A more appropriate title could be: "Implementing Dynamic Routing and Links in Your Angular App"

or

"Using routerLink for Dynamic Navigation in Angular"

This would better prepare readers for the content they're about to engage with and improve the overall flow of the tutorial. Thank you for bringing attention to this discrepancy.

AndrewKushnir commented 5 days ago

Closing this ticket, since the corresponding fix was merged (https://github.com/angular/angular/pull/58060).