fix: like in form sidebar
This commit is contained in:
parent
1ed0dba4de
commit
1e235f4e8e
2 changed files with 11 additions and 8 deletions
|
|
@ -54,7 +54,7 @@ frappe.ui.form.Sidebar = class {
|
|||
});
|
||||
|
||||
this.like_icon.on("click", function() {
|
||||
frappe.ui.toggle_like(me.like_icon, me.frm.doctype, me.frm.doc.name, function() {
|
||||
frappe.ui.toggle_like(me.like_wrapper, me.frm.doctype, me.frm.doc.name, function() {
|
||||
me.refresh_like();
|
||||
});
|
||||
});
|
||||
|
|
@ -200,9 +200,9 @@ frappe.ui.form.Sidebar = class {
|
|||
|
||||
make_like() {
|
||||
this.like_wrapper = this.sidebar.find(".liked-by");
|
||||
this.like_icon = this.sidebar.find(".liked-by .octicon-heart");
|
||||
this.like_icon = this.sidebar.find(".liked-by .like-icon");
|
||||
this.like_count = this.sidebar.find(".liked-by .likes-count");
|
||||
frappe.ui.setup_like_popover(this.sidebar.find(".liked-by-parent"), ".liked-by");
|
||||
frappe.ui.setup_like_popover(this.sidebar.find(".liked-by-parent"), ".like-icon");
|
||||
}
|
||||
|
||||
make_follow() {
|
||||
|
|
@ -218,9 +218,10 @@ frappe.ui.form.Sidebar = class {
|
|||
}
|
||||
|
||||
this.like_wrapper.attr("data-liked-by", this.frm.doc._liked_by);
|
||||
|
||||
this.like_icon.toggleClass("text-extra-muted not-liked",
|
||||
!frappe.ui.is_liked(this.frm.doc))
|
||||
const liked = frappe.ui.is_liked(this.frm.doc);
|
||||
this.like_wrapper
|
||||
.toggleClass("not-liked", !liked)
|
||||
.toggleClass("liked", liked)
|
||||
.attr("data-doctype", this.frm.doctype)
|
||||
.attr("data-name", this.frm.doc.name);
|
||||
|
||||
|
|
|
|||
|
|
@ -106,8 +106,10 @@
|
|||
</ul>
|
||||
<ul class="list-unstyled sidebar-menu">
|
||||
<li class="liked-by-parent">
|
||||
<span class="liked-by">
|
||||
<i class="octicon octicon-heart like-action text-extra-muted fa-fw"></i>
|
||||
<span class="liked-by like-action">
|
||||
<svg class="icon icon-sm">
|
||||
<use xlink:href="#icon-heart" class="like-icon"></use>
|
||||
</svg>
|
||||
<span class="likes-count"></span>
|
||||
</span>
|
||||
</li>
|
||||
|
|
|
|||
Loading…
Add table
Reference in a new issue