From 5f83045baf70936fd3291784d70819fd442b6249 Mon Sep 17 00:00:00 2001 From: pateljannat Date: Mon, 4 Oct 2021 12:26:12 +0530 Subject: [PATCH 1/7] feat: single thread discussions --- frappe/templates/discussions/comment_box.html | 3 +-- frappe/templates/discussions/discussions.js | 25 ++++++++++++------- .../discussions/discussions_section.html | 17 +++++++------ .../web_template/discussions/discussions.json | 11 ++++++-- 4 files changed, 36 insertions(+), 20 deletions(-) diff --git a/frappe/templates/discussions/comment_box.html b/frappe/templates/discussions/comment_box.html index cbdc3ad3e5..ebaa37f772 100644 --- a/frappe/templates/discussions/comment_box.html +++ b/frappe/templates/discussions/comment_box.html @@ -25,8 +25,7 @@ {{ _("Cancel") }} -
+
{{ _("Post") }}
diff --git a/frappe/templates/discussions/discussions.js b/frappe/templates/discussions/discussions.js index becc50410d..8b36d0846c 100644 --- a/frappe/templates/discussions/discussions.js +++ b/frappe/templates/discussions/discussions.js @@ -87,12 +87,17 @@ const setup_socket_io = () => { }; const publish_message = (data) => { + const doctype = decodeURIComponent($(".discussions-parent .thread-card").attr("data-doctype")); + const docname = decodeURIComponent($(".discussions-parent .thread-card").attr("data-docname")); + const topic = data.topic_info; + + if ($(`.discussion-on-page[data-topic=${topic.name}]`).length) { - if ($(`.discussion-on-page[data-topic=${data.topic_info.name}]`).length) { post_message_cleanup(); - $('
' + data.template).insertBefore(`.discussion-on-page[data-topic=${data.topic_info.name}] .discussion-form`); - } else if ((decodeURIComponent($(".discussions-parent .discussions-card").attr("data-doctype")) == data.topic_info.reference_doctype - && decodeURIComponent($(".discussions-parent .discussions-card").attr("data-docname")) == data.topic_info.reference_docname)) { + $('
' + data.template) + .insertBefore(`.discussion-on-page[data-topic=${topic.name}] .discussion-form`); + + } else if (doctype == topic.reference_doctype && docname == topic.reference_docname) { post_message_cleanup(); data.new_topic_template = style_avatar_frame(data.new_topic_template); @@ -100,16 +105,18 @@ const publish_message = (data) => { $(data.sidebar).insertAfter(`.discussions-sidebar .form-group`); $(`#discussion-group`).prepend(data.new_topic_template); - if (data.topic_info.owner == frappe.session.user) { + if (topic.owner == frappe.session.user) { $(".discussion-on-page").collapse(); $(".sidebar-topic").first().click(); } - } else if (data.topic_info.owner == frappe.session.user) { + + } else if (topic.owner == frappe.session.user + && doctype == topic.reference_docname && docname == topic.reference_docname) { post_message_cleanup(); window.location.reload(); } - update_reply_count(data.topic_info.name); + update_reply_count(topic.name); }; const post_message_cleanup = () => { @@ -191,10 +198,10 @@ const submit_discussion = (e) => { const reply = $(".comment-field:visible").val().trim(); if (reply) { - let doctype = $(e.currentTarget).attr("data-doctype"); + let doctype = $(e.currentTarget).closest(".thread-card").attr("data-doctype"); doctype = doctype ? decodeURIComponent(doctype) : doctype; - let docname = $(e.currentTarget).attr("data-docname"); + let docname = $(e.currentTarget).closest(".thread-card").attr("data-docname"); docname = docname ? decodeURIComponent(docname) : docname; frappe.call({ diff --git a/frappe/templates/discussions/discussions_section.html b/frappe/templates/discussions/discussions_section.html index 65e1730bc7..b9fa80fe4f 100644 --- a/frappe/templates/discussions/discussions_section.html +++ b/frappe/templates/discussions/discussions_section.html @@ -11,10 +11,10 @@ {% endif %} - {% if topics %} -
+
+ {% if topics %}
{% include "frappe/templates/discussions/search.html" %} {% for topic in topics %} @@ -26,11 +26,14 @@
{% include "frappe/templates/discussions/reply_section.html" %}
-
- {% else %} -
-
+ {% elif not topics and single_thread %} +
+ {% include "frappe/templates/discussions/comment_box.html" %} +
+ + {% else %} +
No {{ title }}
There are no {{ title | lower }} for this {{ doctype | lower }}, why don't you start one!
diff --git a/frappe/website/web_template/discussions/discussions.json b/frappe/website/web_template/discussions/discussions.json index 8856787828..c8c90e08aa 100644 --- a/frappe/website/web_template/discussions/discussions.json +++ b/frappe/website/web_template/discussions/discussions.json @@ -22,10 +22,17 @@ "label": "Web Page", "options": "Web Page", "reqd": 1 + }, + { + "__unsaved": 1, + "fieldname": "single_thread", + "fieldtype": "Check", + "label": "Single Thread", + "reqd": 0 } ], "idx": 0, - "modified": "2021-10-01 12:15:28.876920", + "modified": "2021-10-01 15:15:57.366552", "modified_by": "Administrator", "module": "Website", "name": "Discussions", @@ -33,4 +40,4 @@ "standard": 1, "template": "", "type": "Section" -} +} \ No newline at end of file From 4e8f42288fd83f1fefb5e859f08307d832884ab3 Mon Sep 17 00:00:00 2001 From: pateljannat Date: Tue, 5 Oct 2021 12:49:56 +0530 Subject: [PATCH 2/7] feat: single thread discussions --- frappe/templates/discussions/comment_box.html | 48 ++++++++++--------- frappe/templates/discussions/discussions.js | 25 ++++++---- .../discussions/discussions_section.html | 16 +++++-- .../templates/discussions/reply_section.html | 4 -- frappe/templates/styles/discussion_style.css | 2 +- .../discussion_reply/discussion_reply.py | 2 +- 6 files changed, 54 insertions(+), 43 deletions(-) diff --git a/frappe/templates/discussions/comment_box.html b/frappe/templates/discussions/comment_box.html index ebaa37f772..ab4714185a 100644 --- a/frappe/templates/discussions/comment_box.html +++ b/frappe/templates/discussions/comment_box.html @@ -1,31 +1,35 @@
-
-
-
- -
+ {% if not single_thread %} +
+
+
+
+
+ {% endif %} -
-
-
- -
+
+
+
+
+
-
-
- {{ _("Press Cmd+Enter to post your comment") }} -
- - {{ _("Cancel") }} -
- {{ _("Post") }}
+
+
+ {{ _("Press Cmd+Enter to post your comment") }}
- + + {{ _("Cancel") }} +
+ {{ _("Post") }} +
+
+ diff --git a/frappe/templates/discussions/discussions.js b/frappe/templates/discussions/discussions.js index 8b36d0846c..af49dc29d6 100644 --- a/frappe/templates/discussions/discussions.js +++ b/frappe/templates/discussions/discussions.js @@ -87,18 +87,23 @@ const setup_socket_io = () => { }; const publish_message = (data) => { - const doctype = decodeURIComponent($(".discussions-parent .thread-card").attr("data-doctype")); - const docname = decodeURIComponent($(".discussions-parent .thread-card").attr("data-docname")); + const doctype = decodeURIComponent($(".discussions-parent").attr("data-doctype")); + const docname = decodeURIComponent($(".discussions-parent").attr("data-docname")); const topic = data.topic_info; - if ($(`.discussion-on-page[data-topic=${topic.name}]`).length) { + console.log(topic, doctype, topic.reference_doctype, docname, topic.reference_docname, doctype == topic.reference_doctype, docname == topic.reference_docname) + console.log(topic.owner, frappe.session.user, topic.owner == frappe.session.user) + console.log(topic.owner == frappe.session.user && doctype == topic.reference_doctype && docname == topic.reference_docname) + console.log(topic.title) + if ($(`.discussion-on-page[data-topic=${topic.name}]`).length) { + console.log("if") post_message_cleanup(); $('
' + data.template) .insertBefore(`.discussion-on-page[data-topic=${topic.name}] .discussion-form`); - } else if (doctype == topic.reference_doctype && docname == topic.reference_docname) { - + } else if (doctype == topic.reference_doctype && docname == topic.reference_docname && $(".reply-card").length) { + console.log("elif") post_message_cleanup(); data.new_topic_template = style_avatar_frame(data.new_topic_template); @@ -106,13 +111,13 @@ const publish_message = (data) => { $(`#discussion-group`).prepend(data.new_topic_template); if (topic.owner == frappe.session.user) { - $(".discussion-on-page").collapse(); + $(".discussion-on-page") && $(".discussion-on-page").collapse(); $(".sidebar-topic").first().click(); } - } else if (topic.owner == frappe.session.user - && doctype == topic.reference_docname && docname == topic.reference_docname) { + } else if (topic.owner == frappe.session.user && doctype == topic.reference_doctype && docname == topic.reference_docname) { post_message_cleanup(); + console.log("in") window.location.reload(); } @@ -198,10 +203,10 @@ const submit_discussion = (e) => { const reply = $(".comment-field:visible").val().trim(); if (reply) { - let doctype = $(e.currentTarget).closest(".thread-card").attr("data-doctype"); + let doctype = $(e.currentTarget).closest(".discussions-parent").attr("data-doctype"); doctype = doctype ? decodeURIComponent(doctype) : doctype; - let docname = $(e.currentTarget).closest(".thread-card").attr("data-docname"); + let docname = $(e.currentTarget).closest(".discussions-parent").attr("data-docname"); docname = docname ? decodeURIComponent(docname) : docname; frappe.call({ diff --git a/frappe/templates/discussions/discussions_section.html b/frappe/templates/discussions/discussions_section.html index b9fa80fe4f..f62dde112e 100644 --- a/frappe/templates/discussions/discussions_section.html +++ b/frappe/templates/discussions/discussions_section.html @@ -1,9 +1,11 @@ {% set topics = frappe.get_all("Discussion Topic", {"reference_doctype": doctype, "reference_docname": docname}, ["name", "title", "owner", "creation"]) %} -{% include "frappe/templates/discussions/topic_modal.html" %} -
+
+ + {% include "frappe/templates/discussions/topic_modal.html" %} +
{{ _(title) }} {% if topics %} @@ -11,20 +13,24 @@ {% endif %}
- -
+
{% if topics %} + + {% if not single_thread %}
{% include "frappe/templates/discussions/search.html" %} + {% for topic in topics %} {% set replies = frappe.get_all("Discussion Reply", {"topic": topic.name})%} {% include "frappe/templates/discussions/sidebar.html" %} {% endfor %}
+ {% endif %}
+ {% for topic in topics %} {% include "frappe/templates/discussions/reply_section.html" %} + {% endfor %}
{% elif not topics and single_thread %} diff --git a/frappe/templates/discussions/reply_section.html b/frappe/templates/discussions/reply_section.html index e2f86da5f9..0a9d3fdfbe 100644 --- a/frappe/templates/discussions/reply_section.html +++ b/frappe/templates/discussions/reply_section.html @@ -1,8 +1,6 @@ -{% for topic in topics %} {% set replies = frappe.get_all("Discussion Reply", {"topic": topic.name}, ["reply", "owner", "creation"], order_by="creation")%} -{% if replies %}
@@ -34,5 +32,3 @@ {% endif %}
-{% endif %} -{% endfor %} diff --git a/frappe/templates/styles/discussion_style.css b/frappe/templates/styles/discussion_style.css index 7aa6a02d5a..44f3a649fd 100644 --- a/frappe/templates/styles/discussion_style.css +++ b/frappe/templates/styles/discussion_style.css @@ -3,7 +3,7 @@ padding: 1rem; } -.discussions-parent .form-control { +.thread-card .form-control { background-color: #FFFFFF; font-size: inherit; color: inherit; diff --git a/frappe/website/doctype/discussion_reply/discussion_reply.py b/frappe/website/doctype/discussion_reply/discussion_reply.py index 82e379c7d4..9665817fca 100644 --- a/frappe/website/doctype/discussion_reply/discussion_reply.py +++ b/frappe/website/doctype/discussion_reply/discussion_reply.py @@ -27,7 +27,7 @@ class DiscussionReply(Document): }) new_topic_template = frappe.render_template("frappe/templates/discussions/reply_section.html", { - "topics": topic_info + "topic": topic_info[0] }) frappe.publish_realtime( From e7ad5f2c29e7e3207696fbdb7042b9b277eb46a0 Mon Sep 17 00:00:00 2001 From: pateljannat Date: Thu, 7 Oct 2021 09:38:40 +0530 Subject: [PATCH 3/7] fix: single thread without refresh --- frappe/templates/discussions/discussions.js | 21 ++++++++++--------- .../discussions/discussions_section.html | 17 +++++++-------- frappe/templates/discussions/reply_card.html | 4 ++-- .../templates/discussions/reply_section.html | 13 ++++++++++-- .../discussion_reply/discussion_reply.py | 11 +++++----- 5 files changed, 38 insertions(+), 28 deletions(-) diff --git a/frappe/templates/discussions/discussions.js b/frappe/templates/discussions/discussions.js index af49dc29d6..e04c8ae8c0 100644 --- a/frappe/templates/discussions/discussions.js +++ b/frappe/templates/discussions/discussions.js @@ -90,20 +90,16 @@ const publish_message = (data) => { const doctype = decodeURIComponent($(".discussions-parent").attr("data-doctype")); const docname = decodeURIComponent($(".discussions-parent").attr("data-docname")); const topic = data.topic_info; + const single_thread = $(".is-single-thread").length; + const first_topic = !$(".reply-card").length; + const document_match_found = doctype == topic.reference_doctype && docname == topic.reference_docname; - console.log(topic, doctype, topic.reference_doctype, docname, topic.reference_docname, doctype == topic.reference_doctype, docname == topic.reference_docname) - console.log(topic.owner, frappe.session.user, topic.owner == frappe.session.user) - - console.log(topic.owner == frappe.session.user && doctype == topic.reference_doctype && docname == topic.reference_docname) - console.log(topic.title) if ($(`.discussion-on-page[data-topic=${topic.name}]`).length) { - console.log("if") post_message_cleanup(); $('
' + data.template) .insertBefore(`.discussion-on-page[data-topic=${topic.name}] .discussion-form`); - } else if (doctype == topic.reference_doctype && docname == topic.reference_docname && $(".reply-card").length) { - console.log("elif") + } else if (!first_topic && !single_thread && document_match_found) { post_message_cleanup(); data.new_topic_template = style_avatar_frame(data.new_topic_template); @@ -115,9 +111,14 @@ const publish_message = (data) => { $(".sidebar-topic").first().click(); } - } else if (topic.owner == frappe.session.user && doctype == topic.reference_doctype && docname == topic.reference_docname) { + } else if (single_thread && document_match_found) { + post_message_cleanup(); + data.template = style_avatar_frame(data.template); + $(data.template).insertBefore(`.discussion-form`); + $(".discussion-on-page").attr("data-topic", topic.name); + + } else if (topic.owner == frappe.session.user && document_match_found) { post_message_cleanup(); - console.log("in") window.location.reload(); } diff --git a/frappe/templates/discussions/discussions_section.html b/frappe/templates/discussions/discussions_section.html index f62dde112e..57dc59c926 100644 --- a/frappe/templates/discussions/discussions_section.html +++ b/frappe/templates/discussions/discussions_section.html @@ -2,21 +2,22 @@ {"reference_doctype": doctype, "reference_docname": docname}, ["name", "title", "owner", "creation"]) %} -
+
{% include "frappe/templates/discussions/topic_modal.html" %}
{{ _(title) }} - {% if topics %} + {% if topics and not single_thread %} {% include "frappe/templates/discussions/button.html" %} {% endif %}
- {% if topics %} + {% if topics and not single_thread %} + - {% if not single_thread %}
{% include "frappe/templates/discussions/search.html" %} @@ -25,7 +26,6 @@ {% include "frappe/templates/discussions/sidebar.html" %} {% endfor %}
- {% endif %}
{% for topic in topics %} @@ -33,10 +33,9 @@ {% endfor %}
- {% elif not topics and single_thread %} -
- {% include "frappe/templates/discussions/comment_box.html" %} -
+ {% elif single_thread %} + {% set topic = topics[0] if topics | length else None %} + {% include "frappe/templates/discussions/reply_section.html" %} {% else %}
diff --git a/frappe/templates/discussions/reply_card.html b/frappe/templates/discussions/reply_card.html index ed9baf554a..5ff5261472 100644 --- a/frappe/templates/discussions/reply_card.html +++ b/frappe/templates/discussions/reply_card.html @@ -2,10 +2,10 @@
{% set member = frappe.db.get_value("User", reply.owner, ["name", "full_name", "username"], as_dict=True) %}
- {% if loop.index == 1 %} + {% if loop.index == 1 or single_thread %} {{ avatar(reply.owner) }} {% endif %} - + {{ member.full_name }}
{{ frappe.utils.pretty_date(reply.creation) }}
diff --git a/frappe/templates/discussions/reply_section.html b/frappe/templates/discussions/reply_section.html index 0a9d3fdfbe..516acf5e3b 100644 --- a/frappe/templates/discussions/reply_section.html +++ b/frappe/templates/discussions/reply_section.html @@ -1,14 +1,23 @@ +{% if topic %} + {% set replies = frappe.get_all("Discussion Reply", {"topic": topic.name}, ["reply", "owner", "creation"], order_by="creation")%} -
+{% endif %} +
+ + {% if not single_thread %}
{{ _("Back") }}
+ {% endif %} + {% if topic %}
{{ topic.title }}
+ {% endif %} + {% for reply in replies %} {% include "frappe/templates/discussions/reply_card.html" %} diff --git a/frappe/website/doctype/discussion_reply/discussion_reply.py b/frappe/website/doctype/discussion_reply/discussion_reply.py index 9665817fca..d5913e6f29 100644 --- a/frappe/website/doctype/discussion_reply/discussion_reply.py +++ b/frappe/website/doctype/discussion_reply/discussion_reply.py @@ -8,6 +8,10 @@ class DiscussionReply(Document): def after_insert(self): replies = frappe.db.count("Discussion Reply", {"topic": self.topic}) + topic_info = frappe.get_all("Discussion Topic", + {"name": self.topic}, + ["reference_doctype", "reference_docname", "name", "title", "owner", "creation"]) + template = frappe.render_template("frappe/templates/discussions/reply_card.html", { "reply": self, "topic": { @@ -15,13 +19,10 @@ class DiscussionReply(Document): }, "loop": { "index": replies - } + }, + "single_thread": True if not topic_info[0].title else False }) - topic_info = frappe.get_all("Discussion Topic", - {"name": self.topic}, - ["reference_doctype", "reference_docname", "name", "title", "owner", "creation"]) - sidebar = frappe.render_template("frappe/templates/discussions/sidebar.html", { "topic": topic_info[0] }) From bee24912213ae79e303013916f97617a124db456 Mon Sep 17 00:00:00 2001 From: pateljannat Date: Thu, 7 Oct 2021 12:53:19 +0530 Subject: [PATCH 4/7] test: ui test for single thread --- cypress/integration/discussions.js | 16 ++++++++++++++++ frappe/templates/discussions/discussions.js | 4 +++- frappe/templates/discussions/reply_section.html | 2 +- frappe/tests/ui_test_helpers.py | 14 ++++++++------ 4 files changed, 28 insertions(+), 8 deletions(-) diff --git a/cypress/integration/discussions.js b/cypress/integration/discussions.js index 3c055d1923..62cdd9d16f 100644 --- a/cypress/integration/discussions.js +++ b/cypress/integration/discussions.js @@ -57,7 +57,23 @@ context('Discussions', () => { cy.get('.discussion-on-page:visible .comment-field').should('have.value', ''); }; + const single_thread_discussion = () => { + cy.visit('/test-single-thread'); + cy.get('.discussions-sidebar').should('have.length', 0); + cy.get('.reply').should('have.length', 0); + + cy.get('.discussion-on-page .comment-field') + .type('This comment is being made on a single thread discussion.') + .should('have.value', 'This comment is being made on a single thread discussion.'); + + cy.get('.discussion-on-page .submit-discussion').click(); + cy.wait(3000); + cy.get('.discussion-on-page').children(".reply-card").eq(-1).children(".reply-text") + .should('have.text', 'This comment is being made on a single thread discussion.\n'); + } + it('reply through modal', reply_through_modal); it('reply through comment box', reply_through_comment_box); it('cancel and clear comment box', cancel_and_clear_comment_box); + it('single thread discussion', single_thread_discussion); }); diff --git a/frappe/templates/discussions/discussions.js b/frappe/templates/discussions/discussions.js index e04c8ae8c0..19c0f89a49 100644 --- a/frappe/templates/discussions/discussions.js +++ b/frappe/templates/discussions/discussions.js @@ -96,6 +96,7 @@ const publish_message = (data) => { if ($(`.discussion-on-page[data-topic=${topic.name}]`).length) { post_message_cleanup(); + data.template = style_avatar_frame(data.template); $('
' + data.template) .insertBefore(`.discussion-on-page[data-topic=${topic.name}] .discussion-form`); @@ -245,7 +246,8 @@ const get_color_from_palette = (element) => { const style_avatar_frame = (template) => { const $template = $(template); - $template.find(".avatar-frame").css(get_color_from_palette($template.find(".avatar-frame"))); + $template.find(".avatar-frame").length + && $template.find(".avatar-frame").css(get_color_from_palette($template.find(".avatar-frame"))); return $template.prop("outerHTML"); }; diff --git a/frappe/templates/discussions/reply_section.html b/frappe/templates/discussions/reply_section.html index 516acf5e3b..c0be40ab01 100644 --- a/frappe/templates/discussions/reply_section.html +++ b/frappe/templates/discussions/reply_section.html @@ -14,7 +14,7 @@
{% endif %} - {% if topic %} + {% if topic and topic.title %}
{{ topic.title }}
{% endif %} diff --git a/frappe/tests/ui_test_helpers.py b/frappe/tests/ui_test_helpers.py index 82dfe618c7..3e9b3519bc 100644 --- a/frappe/tests/ui_test_helpers.py +++ b/frappe/tests/ui_test_helpers.py @@ -194,16 +194,17 @@ def create_form_tour(): @frappe.whitelist() def create_data_for_discussions(): - web_page = create_web_page() + web_page = create_web_page("Test page for discussions", "test-page-discussions", False) create_topic_and_reply(web_page) + create_web_page("Test single thread discussion", "test-single-thread", True) -def create_web_page(): - web_page = frappe.db.exists("Web Page", {"route": "test-page-discussions"}) +def create_web_page(title, route, single_thread): + web_page = frappe.db.exists("Web Page", {"route": route}) if not web_page: web_page = frappe.get_doc({ "doctype": "Web Page", - "title": "Test page for discussions", - "route": "test-page-discussions", + "title": title, + "route": route, "published": True }) web_page.save() @@ -213,7 +214,8 @@ def create_web_page(): "web_template_values": frappe.as_json({ "title": "Discussions", "cta_title": "New Discussion", - "docname": web_page.name + "docname": web_page.name, + "single_thread": single_thread }) }) web_page.save() From d19864a6a7aceb0b964256a60fdb3ae8bacef8f8 Mon Sep 17 00:00:00 2001 From: pateljannat Date: Thu, 7 Oct 2021 15:18:54 +0530 Subject: [PATCH 5/7] fix: sider --- cypress/integration/discussions.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cypress/integration/discussions.js b/cypress/integration/discussions.js index 62cdd9d16f..a6e0ff9b56 100644 --- a/cypress/integration/discussions.js +++ b/cypress/integration/discussions.js @@ -70,7 +70,7 @@ context('Discussions', () => { cy.wait(3000); cy.get('.discussion-on-page').children(".reply-card").eq(-1).children(".reply-text") .should('have.text', 'This comment is being made on a single thread discussion.\n'); - } + }; it('reply through modal', reply_through_modal); it('reply through comment box', reply_through_comment_box); From 49af7784014933d7a653b98989204e15c806a1f0 Mon Sep 17 00:00:00 2001 From: pateljannat Date: Fri, 8 Oct 2021 18:17:16 +0530 Subject: [PATCH 6/7] style: make divider lider lighter --- frappe/templates/styles/discussion_style.css | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/frappe/templates/styles/discussion_style.css b/frappe/templates/styles/discussion_style.css index 44f3a649fd..593f0958c8 100644 --- a/frappe/templates/styles/discussion_style.css +++ b/frappe/templates/styles/discussion_style.css @@ -247,5 +247,6 @@ .card-divider-dark { border: 1px solid var(--gray-400); - margin-bottom: 16px; + margin-bottom: 1rem; + opacity: 0.5; } From f9104c846f0ac90b8a45ddc98f3f20a20720f4b3 Mon Sep 17 00:00:00 2001 From: pateljannat Date: Mon, 11 Oct 2021 12:05:53 +0530 Subject: [PATCH 7/7] fix: removed opacity and added lighter color to divider --- frappe/templates/styles/discussion_style.css | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/frappe/templates/styles/discussion_style.css b/frappe/templates/styles/discussion_style.css index 593f0958c8..fc9db68aca 100644 --- a/frappe/templates/styles/discussion_style.css +++ b/frappe/templates/styles/discussion_style.css @@ -246,7 +246,6 @@ } .card-divider-dark { - border: 1px solid var(--gray-400); + border: 1px solid var(--gray-300); margin-bottom: 1rem; - opacity: 0.5; }