Merge pull request #35138 from barredterra/link-filters-ux
fix: link filters UX
This commit is contained in:
commit
7e3bfb3ff4
2 changed files with 9 additions and 1 deletions
|
|
@ -328,6 +328,11 @@ frappe.ui.form.ControlLink = class ControlLink extends frappe.ui.form.ControlDat
|
|||
}
|
||||
}
|
||||
|
||||
if (item.value === "filter_description__link_option") {
|
||||
e.preventDefault();
|
||||
return false;
|
||||
}
|
||||
|
||||
if (item.action) {
|
||||
item.value = "";
|
||||
item.label = "";
|
||||
|
|
@ -441,7 +446,7 @@ frappe.ui.form.ControlLink = class ControlLink extends frappe.ui.form.ControlDat
|
|||
if (filter_string) {
|
||||
r.message.push({
|
||||
html: `<span class="text-muted" style="line-height: 1.5">${filter_string}</span>`,
|
||||
value: "",
|
||||
value: "filter_description__link_option",
|
||||
action: () => {},
|
||||
});
|
||||
}
|
||||
|
|
|
|||
|
|
@ -37,6 +37,9 @@
|
|||
|
||||
& > li,
|
||||
& > [role="option"] {
|
||||
&:has(p[title="filter_description__link_option"]) {
|
||||
cursor: default;
|
||||
}
|
||||
cursor: pointer;
|
||||
@include get_textstyle("sm", "regular");
|
||||
padding: var(--padding-sm);
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue