Skip to content

Commit 7594ca1

Browse files
crisbetojelbourn
authored andcommitted
feat(sort): remove hammer dependency (#16951)
These changes remove the dependency on Hammer.js from the `matSortHeader`. I decided to remove the `longpress` gesture, rather than implement it with vanilla JS, because it doesn't look like it's worth the trouble. The gesture was meant to show the hover indication if the user holds their pointer on the header, however the touch area is so small that it's most likely that the user won't be able to see the hover indication anyway, because it'll be under their finger. Furthermore, there's nothing indicating to the user that they can long press this element anyway.
1 parent c841289 commit 7594ca1

File tree

1 file changed

+0
-1
lines changed

1 file changed

+0
-1
lines changed

src/material/sort/sort-header.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@ interface MatSortHeaderColumnDef {
7676
host: {
7777
'(click)': '_handleClick()',
7878
'(mouseenter)': '_setIndicatorHintVisible(true)',
79-
'(longpress)': '_setIndicatorHintVisible(true)',
8079
'(mouseleave)': '_setIndicatorHintVisible(false)',
8180
'[attr.aria-sort]': '_getAriaSortAttribute()',
8281
'[class.mat-sort-header-disabled]': '_isDisabled()',

0 commit comments

Comments
 (0)