Skip to content

fix(ng22): removing mentions of hammermodule#359

Open
ChronosSF wants to merge 7 commits into
vnextfrom
sstoychev/remove-hammer-22
Open

fix(ng22): removing mentions of hammermodule#359
ChronosSF wants to merge 7 commits into
vnextfrom
sstoychev/remove-hammer-22

Conversation

@ChronosSF

Copy link
Copy Markdown
Member

Closes #

Checklist:

  • check topic's TOC/menu and paragraph headings
  • Include TOC topic labels in the topic content when it has a valuable update, is new, or is considered preview / beta
  • link to other topics using ../relative/path.md
  • at the References section at the end of the topic add links to topics, samples, etc
  • reference API documentation instead of adding a section with API

  • use valid component names - [Data] Grid, IgxSelectComponent, <igx-combo>
  • use spell checker tool (VS Code, Grammarly, Microsoft Editor)
  • add inline code blocks for the names of classes / tags / properties
  • add language descriptor for the code blocks
  • check if links function by running the check-api/mdx npm commands
  • check if sample is working and fully visible in the topic
  • check if sample is working and fully visible in the StackBlitz
  • check if code blocks match the code in StackBlitz demo


  • do not resolve requested changes (leave that to the reviewer)

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Ignite UI for Angular documentation to align with Angular 22 by removing guidance and examples that reference HammerModule (which is no longer available from @angular/platform-browser).

Changes:

  • Removed HammerModule mentions and import snippets from multiple component “Getting Started” sections.
  • Updated related “dependency” callouts (e.g., keeping BrowserAnimationsModule where applicable).
  • Revised the update guide’s Angular 9-era Hammer note to reference Angular 22 removal.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
docs/angular/src/content/en/components/treegrid/tree-grid.mdx Removes the HammerModule info aside from the Tree Grid usage section.
docs/angular/src/content/en/components/time-picker.mdx Removes HammerModule from the dependency note and module import example.
docs/angular/src/content/en/components/stepper.mdx Removes HammerModule from standalone import examples.
docs/angular/src/content/en/components/slider/slider.mdx Removes HammerModule warning aside and related module/app.config snippets.
docs/angular/src/content/en/components/paginator.mdx Removes HammerModule from standalone import examples.
docs/angular/src/content/en/components/navdrawer.mdx Removes HammerModule from dependency note and module/standalone examples.
docs/angular/src/content/en/components/month-picker.mdx Removes HammerModule from dependency note and module/standalone examples.
docs/angular/src/content/en/components/list.mdx Removes HammerModule aside and module/standalone import references.
docs/angular/src/content/en/components/hierarchicalgrid/hierarchical-grid.mdx Removes the HammerModule info aside from Getting Started.
docs/angular/src/content/en/components/grid/grid.mdx Removes the HammerModule info aside from Getting Started.
docs/angular/src/content/en/components/general/update-guide.mdx Replaces Angular 9 Hammer guidance with an Angular 22 removal note (needs refinement).
docs/angular/src/content/en/components/date-range-picker.mdx Removes HammerModule from dependency note and module/standalone examples.
docs/angular/src/content/en/components/date-picker.mdx Removes HammerModule from dependency note and module/standalone examples.
docs/angular/src/content/en/components/carousel.mdx Removes HammerModule aside and module/standalone import references.
docs/angular/src/content/en/components/calendar.mdx Removes HammerModule from dependency note and module/standalone examples.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/angular/src/content/en/components/general/update-guide.mdx Outdated
@ChronosSF ChronosSF requested a review from mddragnev June 25, 2026 12:21

@mddragnev mddragnev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

  1. We should probably add some info that we should add hammerjs to the scripts in angular.json in order to have touch interactions of those components.
  2. Please update the jp docs as well

Comment thread docs/angular/src/content/en/components/general/update-guide.mdx Outdated
Comment thread docs/angular/src/content/en/components/general/update-guide.mdx Outdated
@ChronosSF

Copy link
Copy Markdown
Member Author
  1. We should probably add some info that we should add hammerjs to the scripts in angular.json in order to have touch interactions of those components.
  2. Please update the jp docs as well

We have a jp sync pipeline now, although we have some issues keeping it running :/

Y-day we decided to remove hammermodule quietly and instead of suggesting a super temp solution, implement an in-house solution for 22.1 and update the docs then. The task is already logged and we should start working on it soon.

@ChronosSF

ChronosSF commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

@mddragnev , actually scratch the previous. You are right that there is no harm in letting users know how to use the hammer script in the meantime.

Added this under the 21.1 to 22 section.

@ChronosSF ChronosSF requested a review from mddragnev June 25, 2026 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants