Merge pull request #35654 from iamejaaz/dont-show-copy
fix(navbar): don't show copy on new doc
This commit is contained in:
commit
3ea0a56048
1 changed files with 16 additions and 11 deletions
|
|
@ -221,8 +221,10 @@ frappe.breadcrumbs = {
|
|||
let form_route = `/desk/${frappe.router.slug(doctype)}/${encodeURIComponent(docname)}`;
|
||||
|
||||
let docname_title;
|
||||
let is_new_doc = false;
|
||||
if (docname.startsWith("new-" + doctype.toLowerCase().replace(/ /g, "-"))) {
|
||||
docname_title = __("New {0}", [__(doctype)]);
|
||||
is_new_doc = true;
|
||||
} else {
|
||||
let title = frappe.model.get_doc_title(doc);
|
||||
docname_title = title || doc.name;
|
||||
|
|
@ -233,22 +235,25 @@ frappe.breadcrumbs = {
|
|||
this.append_breadcrumb_element(form_route, docname_title, "title-text-form");
|
||||
|
||||
if (view === "form") {
|
||||
let last_crumb = this.$breadcrumbs.find("li").last();
|
||||
let last_crumb = this.$breadcrumbs.find(".title-text-form").parent();
|
||||
last_crumb.addClass("disabled");
|
||||
if (frappe.is_mobile()) {
|
||||
last_crumb.addClass("ellipsis");
|
||||
last_crumb.find("a").addClass("ellipsis");
|
||||
}
|
||||
last_crumb.css("cursor", "copy");
|
||||
last_crumb.click((event) => {
|
||||
event.stopImmediatePropagation();
|
||||
frappe.utils.copy_to_clipboard(doc.name);
|
||||
});
|
||||
last_crumb.attr("title", __("Click to copy name"));
|
||||
last_crumb.tooltip({
|
||||
delay: { show: 100, hide: 100 },
|
||||
trigger: "hover",
|
||||
});
|
||||
|
||||
if (!is_new_doc) {
|
||||
last_crumb.css("cursor", "copy");
|
||||
last_crumb.click((event) => {
|
||||
event.stopImmediatePropagation();
|
||||
frappe.utils.copy_to_clipboard(doc.name);
|
||||
});
|
||||
last_crumb.attr("title", __("Click to copy name"));
|
||||
last_crumb.tooltip({
|
||||
delay: { show: 100, hide: 100 },
|
||||
trigger: "hover",
|
||||
});
|
||||
}
|
||||
}
|
||||
},
|
||||
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue