Skip to content

Conversation

mistrykaran91
Copy link
Contributor

Currently, docs is not grouped based on Components and Directives. This fix will separate those two sections.

Fixes #24093

@mistrykaran91 mistrykaran91 requested a review from a team as a code owner March 19, 2025 17:08
@mistrykaran91 mistrykaran91 requested a review from a team as a code owner March 20, 2025 09:31
@mistrykaran91 mistrykaran91 requested review from crisbeto and wagnermaciel and removed request for a team March 20, 2025 09:31
@angular-robot angular-robot bot added area: docs Related to the documentation area: build & ci Related the build and CI infrastructure of the project area: cdk/overlay area: material/timepicker area: cdk/tree labels Mar 20, 2025
@mistrykaran91 mistrykaran91 force-pushed the fix/section-issue-between-directives-and-component branch from 55724c1 to 239922b Compare March 20, 2025 09:34
@angular-robot angular-robot bot removed area: docs Related to the documentation area: build & ci Related the build and CI infrastructure of the project labels Mar 20, 2025
@mistrykaran91 mistrykaran91 force-pushed the fix/section-issue-between-directives-and-component branch 3 times, most recently from dd2f276 to 573efbe Compare March 20, 2025 09:50
@mistrykaran91 mistrykaran91 force-pushed the fix/section-issue-between-directives-and-component branch from 573efbe to 4f56add Compare March 20, 2025 09:57
@mistrykaran91
Copy link
Contributor Author

Not sure why other labels are added. I just rebased it with the main branch and pushed it. So it might be because of previous changes it added that. Should I another MR by closing this one and branching out from main again ?

Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

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

LGTM

@mistrykaran91
Copy link
Contributor Author

mistrykaran91 commented Mar 22, 2025

@josephperrott, There's one more change in thescss file related to updating the font-size that I have made it.

… handling in dgeni

Currently, docs is not grouped based on Components and Directives. This fix will separate those two sections.

Fixes angular#24093
@mistrykaran91 mistrykaran91 force-pushed the fix/section-issue-between-directives-and-component branch from 379bdf8 to 8cd3206 Compare March 29, 2025 05:29
@mistrykaran91
Copy link
Contributor Author

mistrykaran91 commented Mar 29, 2025

@josephperrott @crisbeto, any idea when this can be merged ?

Copy link
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

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

LGTM

@mmalerba mmalerba added the action: merge The PR is ready for merge by the caretaker label Apr 4, 2025
@mmalerba mmalerba merged commit 3a1a0b3 into angular:main Apr 4, 2025
24 of 25 checks passed
mistrykaran91 added a commit to mistrykaran91/components that referenced this pull request Apr 7, 2025
… handling in dgeni (angular#30661)

Currently, docs is not grouped based on Components and Directives. This fix will separate those two sections.

Fixes angular#24093
@mistrykaran91 mistrykaran91 deleted the fix/section-issue-between-directives-and-component branch April 15, 2025 13:16
@angular-automatic-lock-bot
Copy link

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 16, 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: material.angular.io

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs-bug(icon): MatIcon appears in the Directives section

3 participants