Skip to content

Commit add55d8

Browse files
committed
feat(sort): remove hammer dependency
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 dd548be commit add55d8

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)