vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #38743 from sokumon/toggle-chart

Browse source
This commit is contained in:
Soham Kulkarni 2026-04-22 12:12:26 +05:30 • committed by GitHub
commit c7bfffd7e0
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
3 changed files with 7 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
frappe/public/js/frappe/ui/field_group.js
View file

@ -143,6 +143,9 @@ frappe.ui.FieldGroup = class FieldGroup extends frappe.ui.form.Layout {
let f = this.fields_dict[key];
if (f.get_value) {
let v = f.get_value();
if (!v && f.df.include_default) {
v = f.df.default;
}
if (f.df.reqd && is_null(typeof v === "string" ? strip_html(v) : v))
errors.push(__(f.df.label));

4
frappe/public/js/frappe/ui/group_by/group_by.js
View file

@ -431,7 +431,9 @@ frappe.ui.GroupBy = class {
this.get_group_by_field_label(),
])
: __("Add Group");
if (group_by_applied) {
this.group_by_button.find(".button-label").css("gap", "4px");
}
this.group_by_button
.toggleClass("btn-default", !group_by_applied)
.toggleClass("btn-primary-light", group_by_applied);

1
frappe/public/js/frappe/views/reports/report_view.js
View file

@ -579,6 +579,7 @@ frappe.views.ReportView = class ReportView extends frappe.views.ListView {
options: y_fields,
description: __("Showing only Numeric fields from Report"),
default: defaults.y_axes ? defaults.y_axes.join(", ") : null,
include_default: true,
},
{
label: __("Chart Type"),