@underlayerdev/ui 0.8.0 → 0.8.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -4065,22 +4065,24 @@ class SelectableRowComponent {
|
|
|
4065
4065
|
});
|
|
4066
4066
|
}
|
|
4067
4067
|
/**
|
|
4068
|
-
*
|
|
4069
|
-
*
|
|
4070
|
-
*
|
|
4071
|
-
*
|
|
4072
|
-
*
|
|
4073
|
-
*
|
|
4074
|
-
*
|
|
4075
|
-
*
|
|
4076
|
-
*
|
|
4068
|
+
* While selecting, a tap anywhere in the row toggles it — not just on the
|
|
4069
|
+
* checkbox. This applies on every device, not only once the checkbox has
|
|
4070
|
+
* replaced the media (compact): otherwise the row's content on
|
|
4071
|
+
* fine-pointer devices would only be togglable by precisely hitting the
|
|
4072
|
+
* small hover-revealed checkbox, which is inconsistent with how compact
|
|
4073
|
+
* mode behaves and isn't how Gmail/Photos-style bulk-select works even on
|
|
4074
|
+
* desktop web. It's safe to do unconditionally (regardless of what's
|
|
4075
|
+
* projected) because a consumer whose content contains a routerLink is
|
|
4076
|
+
* already expected to have nulled that binding out while
|
|
4077
|
+
* `selectionActive` — so there's no competing navigation to accidentally
|
|
4078
|
+
* trigger.
|
|
4077
4079
|
*/
|
|
4078
4080
|
onHostClick(event) {
|
|
4079
4081
|
if (this.justLongPressed) {
|
|
4080
4082
|
this.justLongPressed = false;
|
|
4081
4083
|
return;
|
|
4082
4084
|
}
|
|
4083
|
-
if (!this.
|
|
4085
|
+
if (!this.selectionActive())
|
|
4084
4086
|
return;
|
|
4085
4087
|
// The checkbox already has its own (checkedChange) handler — without
|
|
4086
4088
|
// this, a tap on it would toggle twice (once from checkedChange, once
|