From c92b5d01660d47f3c145f9f5333ec7c58d019871 Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Mon, 24 Aug 2020 19:12:26 +0530 Subject: [PATCH 001/121] fix: Handle duration fieldtype during export --- frappe/core/doctype/data_export/exporter.py | 4 ++- frappe/desk/query_report.py | 31 +++++++++++++++++++-- frappe/desk/reportview.py | 27 +++++++++++++++++- 3 files changed, 58 insertions(+), 4 deletions(-) diff --git a/frappe/core/doctype/data_export/exporter.py b/frappe/core/doctype/data_export/exporter.py index e4d2ff2af6..bec8cde7ea 100644 --- a/frappe/core/doctype/data_export/exporter.py +++ b/frappe/core/doctype/data_export/exporter.py @@ -8,7 +8,7 @@ from frappe import _ import frappe.permissions import re, csv, os from frappe.utils.csvutils import UnicodeWriter -from frappe.utils import cstr, formatdate, format_datetime, parse_json, cint +from frappe.utils import cstr, formatdate, format_datetime, parse_json, cint, format_duration from frappe.core.doctype.data_import_legacy.importer import get_data_keys from six import string_types from frappe.core.doctype.access_log.access_log import make_access_log @@ -330,6 +330,8 @@ class DataExporter: value = formatdate(value) elif fieldtype == "Datetime": value = format_datetime(value) + elif fieldtype == "Duration": + value = format_duration(value, df.hide_days) row[_column_start_end.start + i + 1] = value diff --git a/frappe/desk/query_report.py b/frappe/desk/query_report.py index a1cfd02132..8e1c98c2af 100644 --- a/frappe/desk/query_report.py +++ b/frappe/desk/query_report.py @@ -8,14 +8,13 @@ import os, json from frappe import _ from frappe.modules import scrub, get_module_path -from frappe.utils import flt, cint, get_html_format, get_url_to_form +from frappe.utils import flt, cint, get_html_format, get_url_to_form, gzip_decompress, format_duration from frappe.model.utils import render_include from frappe.translate import send_translations import frappe.desk.reportview from frappe.permissions import get_role_permissions from six import string_types, iteritems from datetime import timedelta -from frappe.utils import gzip_decompress from frappe.core.utils import ljust_list def get_report_doc(report_name): @@ -83,6 +82,8 @@ def generate_report_result(report, filters=None, user=None, custom_columns=None) if cint(report.add_total_row) and result and not skip_total_row: result = add_total_row(result, columns) + result = handle_duration_fieldtype_values(columns, result) + return { "result": result, "columns": columns, @@ -266,6 +267,32 @@ def get_columns_from_dict(columns, result): return reordered_result + +def handle_duration_fieldtype_values(columns, result, meta=None): + for i, col in enumerate(columns): + fieldtype, fieldname = None, None + if isinstance(col, string_types): + col = col.split(":") + if len(col) > 1: + if col[1]: + fieldtype = col[1] + if "/" in fieldtype: + fieldtype, options = fieldtype.split("/") + else: + fieldtype = "Data" + else: + fieldtype = col.get("fieldtype") + fieldname = col.get("fieldname") + + if fieldtype == "Duration": + for entry in range(0, len(result)): + val_in_seconds = result[entry][i] + if val_in_seconds: + duration_val = format_duration(val_in_seconds) + result[entry][i] = duration_val + return result + + def get_prepared_report_result(report, filters, dn="", user=None): latest_report_data = {} doc = None diff --git a/frappe/desk/reportview.py b/frappe/desk/reportview.py index 6102be61ce..340e447f19 100644 --- a/frappe/desk/reportview.py +++ b/frappe/desk/reportview.py @@ -11,7 +11,7 @@ from frappe.model.db_query import DatabaseQuery from frappe import _ from six import string_types, StringIO from frappe.core.doctype.access_log.access_log import make_access_log -from frappe.utils import cstr +from frappe.utils import cstr, format_duration @frappe.whitelist() @@ -166,6 +166,8 @@ def export_query(): for i, row in enumerate(ret): data.append([i+1] + list(row)) + data = handle_duration_fieldtype_values(doctype, data, db_query.fields) + if file_format_type == "CSV": # convert to csv @@ -235,6 +237,29 @@ def get_labels(fields, doctype): return labels +def handle_duration_fieldtype_values(doctype, data, fields): + for field in fields: + key = field.split(" as ")[0] + + if key.startswith(('count(', 'sum(', 'avg(')): continue + + if "." in key: + parenttype, fieldname = key.split(".")[0][4:-1], key.split(".")[1].strip("`") + else: + parenttype = doctype + fieldname = field.strip("`") + + df = frappe.get_meta(parenttype).get_field(fieldname) + + if df and df.fieldtype == 'Duration': + index = fields.index(field) + 1 + for i in range(1, len(data)): + val_in_seconds = data[i][index] + if val_in_seconds: + duration_val = format_duration(val_in_seconds, df.hide_days) + data[i][index] = duration_val + return data + @frappe.whitelist() def delete_items(): """delete selected items""" From c0b4532ea510c3e905ab3052d63d867badf87edd Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Mon, 24 Aug 2020 19:12:59 +0530 Subject: [PATCH 002/121] fix: Handle duration fieldtype for Data Import --- frappe/core/doctype/data_import/exporter.py | 5 +++ frappe/core/doctype/data_import/importer.py | 4 ++- .../doctype/data_import_legacy/importer.py | 5 +-- frappe/utils/data.py | 33 +++++++++++++++++++ 4 files changed, 44 insertions(+), 3 deletions(-) diff --git a/frappe/core/doctype/data_import/exporter.py b/frappe/core/doctype/data_import/exporter.py index 3eef6ce016..2ef206f56a 100644 --- a/frappe/core/doctype/data_import/exporter.py +++ b/frappe/core/doctype/data_import/exporter.py @@ -8,6 +8,7 @@ from frappe.model import ( no_value_fields, table_fields as table_fieldtypes, ) +from frappe.utils import flt, format_duration from frappe.utils.csvutils import build_csv_response from frappe.utils.xlsxutils import build_xlsx_response @@ -148,6 +149,10 @@ class Exporter: continue row[i] = doc.get(df.fieldname, "") + if df.fieldtype == "Duration": + value = flt(doc.get(df.fieldname, 0)) + row[i] = format_duration(value, df.hide_days) + return rows def get_data_as_docs(self): diff --git a/frappe/core/doctype/data_import/importer.py b/frappe/core/doctype/data_import/importer.py index 2c10c6b0a5..301c356e45 100644 --- a/frappe/core/doctype/data_import/importer.py +++ b/frappe/core/doctype/data_import/importer.py @@ -9,7 +9,7 @@ import timeit import json from datetime import datetime, date from frappe import _ -from frappe.utils import cint, flt, update_progress_bar, cstr +from frappe.utils import cint, flt, update_progress_bar, cstr, duration_to_seconds from frappe.utils.csvutils import read_csv_content, get_csv_content_from_google_sheets from frappe.utils.xlsxutils import ( read_xlsx_file_from_attached_file, @@ -692,6 +692,8 @@ class Row: value = flt(value) elif df.fieldtype in ["Date", "Datetime"]: value = self.get_date(value, col) + elif df.fieldtype == "Duration": + value = duration_to_seconds(value) return value diff --git a/frappe/core/doctype/data_import_legacy/importer.py b/frappe/core/doctype/data_import_legacy/importer.py index 5bd0daf32b..f7f196da61 100644 --- a/frappe/core/doctype/data_import_legacy/importer.py +++ b/frappe/core/doctype/data_import_legacy/importer.py @@ -15,7 +15,7 @@ from frappe import _ from frappe.utils.csvutils import getlink from frappe.utils.dateutils import parse_date -from frappe.utils import cint, cstr, flt, getdate, get_datetime, get_url, get_absolute_url +from frappe.utils import cint, cstr, flt, getdate, get_datetime, get_url, get_absolute_url, duration_to_seconds from six import string_types @@ -164,7 +164,8 @@ def upload(rows = None, submit_after_import=None, ignore_encoding_errors=False, d[fieldname] = get_datetime(_date + " " + _time) else: d[fieldname] = None - + elif df.fieldtype == "Duration": + d[fieldname] = duration_to_seconds(cstr(d[fieldname])) elif fieldtype in ("Image", "Attach Image", "Attach"): # added file to attachments list attachments.append(d[fieldname]) diff --git a/frappe/utils/data.py b/frappe/utils/data.py index fd5c838b57..2a5ab8d5a3 100644 --- a/frappe/utils/data.py +++ b/frappe/utils/data.py @@ -346,6 +346,11 @@ def format_datetime(datetime_string, format_string=None): return formatted_datetime def format_duration(seconds, hide_days=False): + """Converts the given duration value in float(seconds) to duration format + + example: converts 12885 to '3h 34m 45s' where 12885 = seconds in float + """ + total_duration = { 'days': math.floor(seconds / (3600 * 24)), 'hours': math.floor(seconds % (3600 * 24) / 3600), @@ -373,6 +378,34 @@ def format_duration(seconds, hide_days=False): return duration +def duration_to_seconds(duration): + """Converts the given duration formatted value to duration value in seconds + + example: converts '3h 34m 45s' to 12885 (value in seconds) + """ + value = 0 + if 'd' in duration: + val = duration.split('d') + days = val[0] + value += cint(days) * 24 * 60 * 60 + duration = val[1] + if 'h' in duration: + val = duration.split('h') + hours = val[0] + value += cint(hours) * 60 * 60 + duration = val[1] + if 'm' in duration: + val = duration.split('m') + mins = val[0] + value += cint(mins) * 60 + duration = val[1] + if 's' in duration: + val = duration.split('s') + secs = val[0] + value += cint(secs) + + return value + def get_weekdays(): return ['Monday', 'Tuesday', 'Wednesday', 'Thursday', 'Friday', 'Saturday', 'Sunday'] From 22a8cc2ddaee2db053dfc7255dc92d72a9454269 Mon Sep 17 00:00:00 2001 From: Gavin D'souza Date: Wed, 2 Sep 2020 16:31:02 +0530 Subject: [PATCH 003/121] fix: Uninstall App even if it doesn't exist on bench --- frappe/commands/site.py | 2 +- frappe/installer.py | 26 ++++++++++++++------------ 2 files changed, 15 insertions(+), 13 deletions(-) diff --git a/frappe/commands/site.py b/frappe/commands/site.py index d343d10126..f85a9b2474 100755 --- a/frappe/commands/site.py +++ b/frappe/commands/site.py @@ -452,7 +452,7 @@ def uninstall(context, app, dry_run, yes, no_backup, force): try: frappe.init(site=site) frappe.connect() - remove_app(app_name=app, dry_run=dry_run, yes=yes, no_backup=no_backup, force=force) + remove_app(app_name=app, dry_run=dry_run, yes=yes, no_backup=no_backup, force=force, verbose=context.verbose) finally: frappe.destroy() if not context.sites: diff --git a/frappe/installer.py b/frappe/installer.py index 4994646890..53a8d878c7 100755 --- a/frappe/installer.py +++ b/frappe/installer.py @@ -8,7 +8,7 @@ from __future__ import unicode_literals, print_function from six.moves import input -import os, json, subprocess, shutil +import os, json, subprocess, shutil, sys import click import frappe import frappe.database @@ -119,9 +119,12 @@ def remove_from_installed_apps(app_name): if frappe.flags.in_install: post_install() -def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False): +def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False, verbose=True): """Remove app and all linked to the app's module with the app from a site.""" + if not (verbose or dry_run): + sys.stdout = open(os.devnull, "w") + # dont allow uninstall app if not installed unless forced if not force: if app_name not in frappe.get_installed_apps(): @@ -143,11 +146,12 @@ def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False) frappe.flags.in_uninstall = True drop_doctypes = [] - # remove modules, doctypes, roles - for module_name in frappe.get_module_list(app_name): - for doctype in frappe.get_list("DocType", filters={"module": module_name}, - fields=["name", "issingle"]): - print("removing DocType {0}...".format(doctype.name)) + modules = (x.name for x in frappe.get_all("Module Def", filters={"app_name": app_name})) + for module_name in modules: + print("Deleting Module '{0}'".format(module_name)) + + for doctype in frappe.get_list("DocType", filters={"module": module_name}, fields=["name", "issingle"]): + print("* removing DocType '{0}'...".format(doctype.name)) if not dry_run: frappe.delete_doc("DocType", doctype.name) @@ -155,24 +159,22 @@ def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False) if not doctype.issingle: drop_doctypes.append(doctype.name) - linked_doctypes = frappe.get_all("DocField", filters={"fieldtype": "Link", "options": "Module Def"}, fields=['parent']) ordered_doctypes = ["Desk Page", "Report", "Page", "Web Form"] doctypes_with_linked_modules = ordered_doctypes + [doctype.parent for doctype in linked_doctypes if doctype.parent not in ordered_doctypes] for doctype in doctypes_with_linked_modules: for record in frappe.get_list(doctype, filters={"module": module_name}): - print("removing {0} {1}...".format(doctype, record.name)) + print("* removing {0} '{1}'...".format(doctype, record.name)) if not dry_run: frappe.delete_doc(doctype, record.name) - print("removing Module {0}...".format(module_name)) + print("* removing Module Def '{0}'...".format(module_name)) if not dry_run: frappe.delete_doc("Module Def", module_name) - remove_from_installed_apps(app_name) - if not dry_run: + remove_from_installed_apps(app_name) # drop tables after a commit frappe.db.commit() From dbea31943a6f58582c116423f4a170f0b0935f94 Mon Sep 17 00:00:00 2001 From: Gavin D'souza Date: Wed, 2 Sep 2020 17:08:00 +0530 Subject: [PATCH 004/121] fix: Add verbosity --- frappe/commands/site.py | 5 +++-- frappe/installer.py | 7 ++++--- 2 files changed, 7 insertions(+), 5 deletions(-) diff --git a/frappe/commands/site.py b/frappe/commands/site.py index f85a9b2474..9de06b3723 100755 --- a/frappe/commands/site.py +++ b/frappe/commands/site.py @@ -444,15 +444,16 @@ def remove_from_installed_apps(context, app): @click.option('--dry-run', help='List all doctypes that will be deleted', is_flag=True, default=False) @click.option('--no-backup', help='Do not backup the site', is_flag=True, default=False) @click.option('--force', help='Force remove app from site', is_flag=True, default=False) +@click.option('--verbose', help='Add verbosity', is_flag=True, default=False) @pass_context -def uninstall(context, app, dry_run, yes, no_backup, force): +def uninstall(context, app, dry_run, yes, no_backup, force, verbose): "Remove app and linked modules from site" from frappe.installer import remove_app for site in context.sites: try: frappe.init(site=site) frappe.connect() - remove_app(app_name=app, dry_run=dry_run, yes=yes, no_backup=no_backup, force=force, verbose=context.verbose) + remove_app(app_name=app, dry_run=dry_run, yes=yes, no_backup=no_backup, force=force, verbose=context.verbose or verbose) finally: frappe.destroy() if not context.sites: diff --git a/frappe/installer.py b/frappe/installer.py index 53a8d878c7..d0b6db77d4 100755 --- a/frappe/installer.py +++ b/frappe/installer.py @@ -122,9 +122,6 @@ def remove_from_installed_apps(app_name): def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False, verbose=True): """Remove app and all linked to the app's module with the app from a site.""" - if not (verbose or dry_run): - sys.stdout = open(os.devnull, "w") - # dont allow uninstall app if not installed unless forced if not force: if app_name not in frappe.get_installed_apps(): @@ -138,6 +135,9 @@ def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False, if not confirm: return + if not (verbose or dry_run): + sys.stdout = open(os.devnull, "w") + if not no_backup: from frappe.utils.backups import scheduled_backup print("Backing up...") @@ -181,6 +181,7 @@ def remove_app(app_name, dry_run=False, yes=False, no_backup=False, force=False, for doctype in set(drop_doctypes): frappe.db.sql("drop table `tab{0}`".format(doctype)) + sys.stdout = sys.__stdout__ click.secho("Uninstalled App {0} from Site {1}".format(app_name, frappe.local.site), fg="green") frappe.flags.in_uninstall = False From 5c01c7145a75f84474e05ea826dceacd905bea70 Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Wed, 2 Sep 2020 17:38:21 +0530 Subject: [PATCH 005/121] fix: handle total for duration fieldtype --- frappe/desk/query_report.py | 32 ++------------------------ frappe/public/js/frappe/model/model.js | 2 +- 2 files changed, 3 insertions(+), 31 deletions(-) diff --git a/frappe/desk/query_report.py b/frappe/desk/query_report.py index 8e1c98c2af..fa1df349c3 100644 --- a/frappe/desk/query_report.py +++ b/frappe/desk/query_report.py @@ -8,7 +8,7 @@ import os, json from frappe import _ from frappe.modules import scrub, get_module_path -from frappe.utils import flt, cint, get_html_format, get_url_to_form, gzip_decompress, format_duration +from frappe.utils import flt, cint, get_html_format, get_url_to_form, gzip_decompress, format_duration, cstr from frappe.model.utils import render_include from frappe.translate import send_translations import frappe.desk.reportview @@ -82,8 +82,6 @@ def generate_report_result(report, filters=None, user=None, custom_columns=None) if cint(report.add_total_row) and result and not skip_total_row: result = add_total_row(result, columns) - result = handle_duration_fieldtype_values(columns, result) - return { "result": result, "columns": columns, @@ -267,32 +265,6 @@ def get_columns_from_dict(columns, result): return reordered_result - -def handle_duration_fieldtype_values(columns, result, meta=None): - for i, col in enumerate(columns): - fieldtype, fieldname = None, None - if isinstance(col, string_types): - col = col.split(":") - if len(col) > 1: - if col[1]: - fieldtype = col[1] - if "/" in fieldtype: - fieldtype, options = fieldtype.split("/") - else: - fieldtype = "Data" - else: - fieldtype = col.get("fieldtype") - fieldname = col.get("fieldname") - - if fieldtype == "Duration": - for entry in range(0, len(result)): - val_in_seconds = result[entry][i] - if val_in_seconds: - duration_val = format_duration(val_in_seconds) - result[entry][i] = duration_val - return result - - def get_prepared_report_result(report, filters, dn="", user=None): latest_report_data = {} doc = None @@ -454,7 +426,7 @@ def add_total_row(result, columns, meta = None): if i >= len(row): continue cell = row.get(fieldname) if isinstance(row, dict) else row[i] - if fieldtype in ["Currency", "Int", "Float", "Percent"] and flt(cell): + if fieldtype in ["Currency", "Int", "Float", "Percent", "Duration"] and flt(cell): total_row[i] = flt(total_row[i]) + flt(cell) if fieldtype == "Percent" and i not in has_percent: diff --git a/frappe/public/js/frappe/model/model.js b/frappe/public/js/frappe/model/model.js index 663850d08c..308d9bd5f8 100644 --- a/frappe/public/js/frappe/model/model.js +++ b/frappe/public/js/frappe/model/model.js @@ -31,7 +31,7 @@ $.extend(frappe.model, { {fieldname:'docstatus', fieldtype:'Int', label:__('Document Status')}, ], - numeric_fieldtypes: ["Int", "Float", "Currency", "Percent"], + numeric_fieldtypes: ["Int", "Float", "Currency", "Percent", "Duration"], std_fields_table: [ {fieldname:'parent', fieldtype:'Data', label:__('Parent')}, From 0888fb48a73d42cd3f2eaae2eb77bbd4a9947ebe Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Wed, 2 Sep 2020 19:01:42 +0530 Subject: [PATCH 006/121] feat: validate duration format in data import --- frappe/core/doctype/data_import/importer.py | 14 ++++++++++++++ 1 file changed, 14 insertions(+) diff --git a/frappe/core/doctype/data_import/importer.py b/frappe/core/doctype/data_import/importer.py index 301c356e45..5271690527 100644 --- a/frappe/core/doctype/data_import/importer.py +++ b/frappe/core/doctype/data_import/importer.py @@ -664,6 +664,20 @@ class Row: } ) return + elif df.fieldtype == "Duration": + import re + is_valid_duration = re.match("^(?:(\d+d)?((^|\s)\d+h)?((^|\s)\d+m)?((^|\s)\d+s)?)$", value) + if not is_valid_duration: + self.warnings.append( + { + "row": self.row_number, + "col": col.column_number, + "field": df_as_json(df), + "message": _("Value {0} must be in the valid duration format: d h m s").format( + frappe.bold(value) + ) + } + ) return value From 32e864bb6c55d5f84f3cf7be56294510e8d12b30 Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Wed, 2 Sep 2020 19:21:56 +0530 Subject: [PATCH 007/121] test: duration fieldtype import --- .../data_import/fixtures/sample_import_file.csv | 10 +++++----- frappe/core/doctype/data_import/test_importer.py | 7 ++++++- 2 files changed, 11 insertions(+), 6 deletions(-) diff --git a/frappe/core/doctype/data_import/fixtures/sample_import_file.csv b/frappe/core/doctype/data_import/fixtures/sample_import_file.csv index ef5b96df58..693f400878 100644 --- a/frappe/core/doctype/data_import/fixtures/sample_import_file.csv +++ b/frappe/core/doctype/data_import/fixtures/sample_import_file.csv @@ -1,5 +1,5 @@ -Title ,Description ,Number ,another_number ,ID (Table Field 1) ,Child Title (Table Field 1) ,Child Description (Table Field 1) ,Child 2 Title (Table Field 2) ,Child 2 Date (Table Field 2) ,Child 2 Number (Table Field 2) ,Child Title (Table Field 1 Again) ,Child Date (Table Field 1 Again) ,Child Number (Table Field 1 Again) ,table_field_1_again.child_another_number -Test ,test description ,1 ,2 ,"" ,child title ,child description ,child title ,14-08-2019 ,4 ,child title again ,22-09-2020 ,5 , 7 - , , , , ,child title 2 ,child description 2 ,title child ,30-10-2019 ,5 ,child title again 2 ,22-09-2021 , , -Test 2 ,test description 2 ,1 ,2 , ,child mandatory title , ,title child man , , ,child mandatory again , , , -Test 3 ,test description 3 ,4 ,5 ,"" ,child title asdf ,child description asdf ,child title asdf adsf ,15-08-2019 ,6 ,child title again asdf ,22-09-2022 ,9 , 71 +Title ,Description ,Number ,Duration,another_number ,ID (Table Field 1),Child Title (Table Field 1),Child Description (Table Field 1),Child 2 Title (Table Field 2),Child 2 Date (Table Field 2),Child 2 Number (Table Field 2),Child Title (Table Field 1 Again),Child Date (Table Field 1 Again),Child Number (Table Field 1 Again),table_field_1_again.child_another_number +Test ,test description ,1,3h,2, ,child title ,child description ,child title ,14-08-2019,4,child title again ,22-09-2020,5,7 + , , ,, , ,child title 2,child description 2,title child ,30-10-2019,5,child title again 2,22-09-2021, , +Test 2,test description 2,1,4d 3h,2, ,child mandatory title , ,title child man , , ,child mandatory again , , , +Test 3,test description 3,4,5d 5h 45m,5, ,child title asdf ,child description asdf ,child title asdf adsf ,15-08-2019,6,child title again asdf ,22-09-2022,9,71 \ No newline at end of file diff --git a/frappe/core/doctype/data_import/test_importer.py b/frappe/core/doctype/data_import/test_importer.py index bdadad7890..3b573e64a1 100644 --- a/frappe/core/doctype/data_import/test_importer.py +++ b/frappe/core/doctype/data_import/test_importer.py @@ -5,7 +5,7 @@ from __future__ import unicode_literals import unittest import frappe -from frappe.utils import getdate +from frappe.utils import getdate, format_duration doctype_name = 'DocType for Import' @@ -24,6 +24,7 @@ class TestImporter(unittest.TestCase): self.assertEqual(doc1.description, 'test description') self.assertEqual(doc1.number, 1) + self.assertEqual(format_duration(doc1.duration), '3h') self.assertEqual(doc1.table_field_1[0].child_title, 'child title') self.assertEqual(doc1.table_field_1[0].child_description, 'child description') @@ -40,7 +41,10 @@ class TestImporter(unittest.TestCase): self.assertEqual(doc1.table_field_1_again[1].child_date, getdate('2021-09-22')) self.assertEqual(doc2.description, 'test description 2') + self.assertEqual(format_duration(doc2.duration), '4d 3h') + self.assertEqual(doc3.another_number, 5) + self.assertEqual(format_duration(doc3.duration), '5d 5h 45m') def test_data_import_preview(self): import_file = get_import_file('sample_import_file') @@ -146,6 +150,7 @@ def create_doctype_if_not_exists(doctype_name, force=False): {'label': 'Title', 'fieldname': 'title', 'reqd': 1, 'fieldtype': 'Data'}, {'label': 'Description', 'fieldname': 'description', 'fieldtype': 'Small Text'}, {'label': 'Date', 'fieldname': 'date', 'fieldtype': 'Date'}, + {'label': 'Duration', 'fieldname': 'duration', 'fieldtype': 'Duration'}, {'label': 'Number', 'fieldname': 'number', 'fieldtype': 'Int'}, {'label': 'Number', 'fieldname': 'another_number', 'fieldtype': 'Int'}, {'label': 'Table Field 1', 'fieldname': 'table_field_1', 'fieldtype': 'Table', 'options': table_1_name}, From 5f51c201773bee983c6f36869eefcfed120e4677 Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Thu, 3 Sep 2020 13:36:30 +0530 Subject: [PATCH 008/121] fix(reports): handle duration fieldtype during export --- .../doctype/report_column/report_column.json | 4 +-- frappe/desk/query_report.py | 29 +++++++++++++++++++ frappe/public/js/frappe/utils/utils.js | 8 ++++- .../js/frappe/views/reports/query_report.js | 3 ++ 4 files changed, 41 insertions(+), 3 deletions(-) diff --git a/frappe/core/doctype/report_column/report_column.json b/frappe/core/doctype/report_column/report_column.json index 53b5dff9b6..2e6a22d29a 100644 --- a/frappe/core/doctype/report_column/report_column.json +++ b/frappe/core/doctype/report_column/report_column.json @@ -31,7 +31,7 @@ "fieldtype": "Select", "in_list_view": 1, "label": "Fieldtype", - "options": "Check\nCurrency\nData\nDate\nDatetime\nDynamic Link\nFloat\nFold\nInt\nLink\nSelect\nTime", + "options": "Check\nCurrency\nData\nDate\nDatetime\nDuration\nDynamic Link\nFloat\nFold\nInt\nLink\nSelect\nTime", "reqd": 1 }, { @@ -48,7 +48,7 @@ "index_web_pages_for_search": 1, "istable": 1, "links": [], - "modified": "2020-08-17 14:32:17.174796", + "modified": "2020-09-03 10:52:03.895817", "modified_by": "Administrator", "module": "Core", "name": "Report Column", diff --git a/frappe/desk/query_report.py b/frappe/desk/query_report.py index fa1df349c3..dc42228f81 100644 --- a/frappe/desk/query_report.py +++ b/frappe/desk/query_report.py @@ -359,6 +359,7 @@ def export_query(): columns = get_columns_dict(data.columns) from frappe.utils.xlsxutils import make_xlsx + data['result'] = handle_duration_fieldtype_values(data.get('result'), data.get('columns')) xlsx_data = build_xlsx_data(columns, data, visible_idx, include_indentation) xlsx_file = make_xlsx(xlsx_data, "Query Report") @@ -366,6 +367,30 @@ def export_query(): frappe.response['filecontent'] = xlsx_file.getvalue() frappe.response['type'] = 'binary' +def handle_duration_fieldtype_values(result, columns): + for i, col in enumerate(columns): + fieldtype, fieldname = None, None + if isinstance(col, string_types): + col = col.split(":") + if len(col) > 1: + if col[1]: + fieldtype = col[1] + if "/" in fieldtype: + fieldtype, options = fieldtype.split("/") + else: + fieldtype = "Data" + else: + fieldtype = col.get("fieldtype") + fieldname = col.get("fieldname") + + if fieldtype == "Duration": + for entry in range(0, len(result)): + val_in_seconds = result[entry][i] + if val_in_seconds: + duration_val = format_duration(val_in_seconds) + result[entry][i] = duration_val + + return result def build_xlsx_data(columns, data, visible_idx, include_indentation): result = [[]] @@ -385,7 +410,11 @@ def build_xlsx_data(columns, data, visible_idx, include_indentation): for idx in range(len(data.columns)): label = columns[idx]["label"] fieldname = columns[idx]["fieldname"] + fieldtype = columns[idx]["fieldtype"] cell_value = row.get(fieldname, row.get(label, "")) + if fieldtype == "Duration": + cell_value = format_duration(value) + if cint(include_indentation) and 'indent' in row and idx == 0: cell_value = (' ' * cint(row['indent'])) + cell_value row_data.append(cell_value) diff --git a/frappe/public/js/frappe/utils/utils.js b/frappe/public/js/frappe/utils/utils.js index 38c22c9c9f..d64be06869 100644 --- a/frappe/public/js/frappe/utils/utils.js +++ b/frappe/public/js/frappe/utils/utils.js @@ -824,8 +824,14 @@ Object.assign(frappe.utils, { }; }, - get_formatted_duration(value, duration_options) { + get_formatted_duration(value, duration_options=null) { let duration = ''; + if (!duration_options) { + duration_options = { + hide_days: 0, + hide_seconds: 0 + } + } if (value) { let total_duration = frappe.utils.seconds_to_duration(value, duration_options); diff --git a/frappe/public/js/frappe/views/reports/query_report.js b/frappe/public/js/frappe/views/reports/query_report.js index 1bec65e460..0817d8cfa5 100644 --- a/frappe/public/js/frappe/views/reports/query_report.js +++ b/frappe/public/js/frappe/views/reports/query_report.js @@ -1322,6 +1322,9 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList { return row .slice(standard_column_count) .map((cell, i) => { + if (cell.column.fieldtype === "Duration") { + cell.content = frappe.utils.get_formatted_duration(cell.content) + } if (include_indentation && i===0) { cell.content = ' '.repeat(row.meta.indent) + (cell.content || ''); } From 14af05037abe8e24c66a1907eda37f15d09e2890 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 13:15:00 +0530 Subject: [PATCH 009/121] feat: Section with Collapsible Content --- frappe/public/scss/page-builder.scss | 38 ++++++++++++++ .../section_with_collapsible_content.html | 21 ++++++++ .../section_with_collapsible_content.json | 51 +++++++++++++++++++ 3 files changed, 110 insertions(+) create mode 100644 frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.html create mode 100644 frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.json diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index f6446a9ba9..28db0b5a85 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -409,3 +409,41 @@ } } } + + +/* Section with Collapsible Content */ + +.collapsible-items { + max-width: 46rem; +} + +.collapsible-item { + padding: 1.75rem 0; + + &:not(:last-child) { + border-bottom: 1px solid $border-color; + } +} + +.collapsible-item a { + text-decoration: none; +} + +.collapsible-item h3 { + margin-bottom: 0; +} + +.collapsible-content { + margin-top: 1rem; + margin-bottom: 0; + color: $gray-700; +} + +.section-with-collapsible-content.align-center { + .section-title, .section-description { + text-align: center; + } + .section-description, .collapsible-items { + margin: 0 auto; + } +} diff --git a/frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.html b/frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.html new file mode 100644 index 0000000000..2b86e7f992 --- /dev/null +++ b/frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.html @@ -0,0 +1,21 @@ +
+

{{ title }}

+ {%- if subtitle -%} +

{{ subtitle }}

+ {%- endif -%} + +
+ {%- for item in items -%} +
+ {%- set collapse_id = 'id-' + frappe.utils.generate_hash('Collapse', 12) -%} + +
+ {{ frappe.utils.md_to_html(item.content) }} +
+
+ {%- endfor -%} +
+
diff --git a/frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.json b/frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.json new file mode 100644 index 0000000000..f35b8d793e --- /dev/null +++ b/frappe/website/web_template/section_with_collapsible_content/section_with_collapsible_content.json @@ -0,0 +1,51 @@ +{ + "creation": "2020-08-07 16:27:38.265089", + "docstatus": 0, + "doctype": "Web Template", + "fields": [ + { + "fieldname": "title", + "fieldtype": "Data", + "label": "Title", + "reqd": 0 + }, + { + "fieldname": "subtitle", + "fieldtype": "Data", + "label": "Subtitle", + "reqd": 0 + }, + { + "fieldname": "align", + "fieldtype": "Select", + "label": "Align", + "options": "Left\nCenter", + "reqd": 0 + }, + { + "fieldname": "items", + "fieldtype": "Table Break", + "label": "Items", + "reqd": 0 + }, + { + "fieldname": "title", + "fieldtype": "Data", + "label": "Title", + "reqd": 0 + }, + { + "fieldname": "content", + "fieldtype": "Markdown Editor", + "label": "Content", + "reqd": 0 + } + ], + "idx": 0, + "modified": "2020-08-13 15:51:23.728803", + "modified_by": "Administrator", + "name": "Section with Collapsible Content", + "owner": "Administrator", + "standard": 1, + "template": "" +} \ No newline at end of file From c0734921de5d30e335c9dedd6282dbcfff321abe Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 13:23:03 +0530 Subject: [PATCH 010/121] feat: Section with Image align center --- frappe/public/scss/page-builder.scss | 14 +++++++++++-- .../section_with_image.html | 16 +++++++------- .../section_with_image.json | 21 ++++++++++++++----- 3 files changed, 37 insertions(+), 14 deletions(-) diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index 28db0b5a85..81c06420bc 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -1,6 +1,6 @@ .hero-content { .btn-primary { - margin-top: 1rem; + margin-top: 1rem; margin-right: 0.5rem; @include media-breakpoint-up(lg) { @@ -35,6 +35,15 @@ } } +.section-with-image.align-center { + text-align: center; + + .section-description, .section-image { + margin-left: auto; + margin-right: auto; + } +} + .section-image { margin-top: 2rem; border-radius: 0.75rem; @@ -444,6 +453,7 @@ text-align: center; } .section-description, .collapsible-items { - margin: 0 auto; + margin-left: auto; + margin-right: auto; } } diff --git a/frappe/website/web_template/section_with_image/section_with_image.html b/frappe/website/web_template/section_with_image/section_with_image.html index ffa47d089e..cfd98064ac 100644 --- a/frappe/website/web_template/section_with_image/section_with_image.html +++ b/frappe/website/web_template/section_with_image/section_with_image.html @@ -1,8 +1,10 @@ -

{{ title }}

-

{{ subtitle }}

+
+

{{ title }}

+

{{ subtitle }}

-{{ frappe.render_template('templates/includes/image_with_blur.html', { - "src": image, - "alt": image_description, - "class": "section-image" -}) }} + {{ frappe.render_template('templates/includes/image_with_blur.html', { + "src": image, + "alt": image_description, + "class": "section-image" + }) }} +
diff --git a/frappe/website/web_template/section_with_image/section_with_image.json b/frappe/website/web_template/section_with_image/section_with_image.json index 5f610e5e2f..46169a8cc3 100644 --- a/frappe/website/web_template/section_with_image/section_with_image.json +++ b/frappe/website/web_template/section_with_image/section_with_image.json @@ -6,26 +6,37 @@ { "fieldname": "title", "fieldtype": "Data", - "label": "Title" + "label": "Title", + "reqd": 0 }, { "fieldname": "subtitle", "fieldtype": "Small Text", - "label": "Subtitle" + "label": "Subtitle", + "reqd": 0 }, { "fieldname": "image", "fieldtype": "Attach Image", - "label": "Image" + "label": "Image", + "reqd": 0 }, { "fieldname": "image_description", "fieldtype": "Data", - "label": "Image Description" + "label": "Image Description", + "reqd": 0 + }, + { + "fieldname": "align", + "fieldtype": "Select", + "label": "Align", + "options": "Left\nCenter", + "reqd": 0 } ], "idx": 0, - "modified": "2020-04-17 19:31:33.474017", + "modified": "2020-08-06 16:08:12.005764", "modified_by": "Administrator", "name": "Section with Image", "owner": "Administrator", From 40a0c69255b7a3db0f5f1a8807ac90e576aaffb8 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:00:26 +0530 Subject: [PATCH 011/121] feat: Footer - Split footer in files - Footer grouped links styling - Put footer logo and extension in one row - Delete unused footer_items.html - Uncheck Right when adding Footer Items in Website Settings --- frappe/public/scss/footer.scss | 77 +++++++++++++++++++ frappe/public/scss/website.scss | 63 +-------------- frappe/templates/includes/footer/footer.html | 44 ++--------- .../includes/footer/footer_grouped_links.html | 56 ++++++++------ .../includes/footer/footer_info.html | 19 +++++ .../includes/footer/footer_items.html | 28 ------- .../includes/footer/footer_links.html | 8 +- .../footer/footer_logo_extension.html | 16 ++++ .../website_settings/website_settings.js | 6 +- 9 files changed, 158 insertions(+), 159 deletions(-) create mode 100644 frappe/public/scss/footer.scss create mode 100644 frappe/templates/includes/footer/footer_info.html delete mode 100644 frappe/templates/includes/footer/footer_items.html create mode 100644 frappe/templates/includes/footer/footer_logo_extension.html diff --git a/frappe/public/scss/footer.scss b/frappe/public/scss/footer.scss new file mode 100644 index 0000000000..9214907fbb --- /dev/null +++ b/frappe/public/scss/footer.scss @@ -0,0 +1,77 @@ +.web-footer { + padding: 5rem 0; + min-height: 140px; +} + +.footer-logo { + min-width: 5rem; + height: 1.5rem; + object-fit: contain; + object-position: left; +} + +.footer-child-item { + margin-top: 0.5rem; +} + +.footer-link, .footer-child-item a { + font-size: $font-size-sm; + font-weight: 500; + color: $gray-700; + + &:hover { + color: $primary; + text-decoration: none; + } +} + +.footer-col-left, .footer-col-right { + padding-top: 0.8rem; + padding-bottom: 1rem; + line-height: 2; + + &:empty { + padding: 0; + } +} + +.footer-col-right { + @include media-breakpoint-up(sm) { + text-align: right; + } +} + +.footer-col-left .footer-link { + margin-right: 1rem; +} + +.footer-col-right .footer-link { + margin-right: 1rem; + @include media-breakpoint-up(sm) { + margin-right: 0; + margin-left: 1rem; + } +} + +.footer-group { + margin-top: 2rem; +} + +.footer-group-label { + color: $text-muted; + font-size: $font-size-sm; + margin-bottom: 0.5rem; +} + +.footer-group-links { + display: flex; + flex-direction: column; + flex-wrap: wrap; + max-height: 10rem; +} + +.footer-info { + border-top: 1px solid $border-color; + color: $text-muted; + font-size: $font-size-sm; +} diff --git a/frappe/public/scss/website.scss b/frappe/public/scss/website.scss index e64c090ea8..59bbe4f19d 100644 --- a/frappe/public/scss/website.scss +++ b/frappe/public/scss/website.scss @@ -12,6 +12,7 @@ @import 'portal'; @import 'search'; @import 'doc'; +@import 'footer'; .ql-editor.read-mode { padding: 0; @@ -162,68 +163,6 @@ a.card { color: #d1d8dd !important; } -// footer - -.web-footer { - padding: 5rem 0; - min-height: 140px; -} - -.footer-logo { - width: 5rem; - height: 2rem; - object-fit: contain; - object-position: left; -} - -.footer-link, .footer-child-item a { - font-weight: 500; - color: $gray-700; - - &:hover { - color: $primary; - text-decoration: none; - } -} - -.footer-col-left, .footer-col-right { - padding-top: 0.8rem; - padding-bottom: 1rem; - line-height: 2; -} - -.footer-col-right { - @include media-breakpoint-up(sm) { - text-align: right; - } -} - -.footer-col-left .footer-link { - margin-right: 1rem; -} - -.footer-col-right .footer-link { - margin-right: 1rem; - @include media-breakpoint-up(sm) { - margin-right: 0; - margin-left: 1rem; - } -} - -.footer-group-label { - color: $text-muted; -} - -.footer-parent-item { - margin-bottom: 0.5rem; -} - -.footer-info { - border-top: 1px solid $border-color; - color: $text-muted; - font-size: $font-size-sm; -} - .no-underline { text-decoration: none !important; } diff --git a/frappe/templates/includes/footer/footer.html b/frappe/templates/includes/footer/footer.html index 671e928d32..2016c7e3d9 100644 --- a/frappe/templates/includes/footer/footer.html +++ b/frappe/templates/includes/footer/footer.html @@ -1,46 +1,12 @@
- {%- if footer_logo -%} -
- -
- {%- endif -%} -
-
- {% if footer_items -%} -
- {% include ["templates/includes/footer/footer_grouped_links.html", "templates/includes/footer/footer_items.html"] %} -
- {% endif %} -
+ {% include "templates/includes/footer/footer_logo_extension.html" %} -
- {% block extension %} - {% include "templates/includes/footer/footer_extension.html" %} - {% endblock %} -
-
+ {% if footer_items -%} + {% include "templates/includes/footer/footer_grouped_links.html" %} + {% endif %} {% include "templates/includes/footer/footer_links.html" %} - -
-
-
- {% if copyright %} - © {{ copyright }} - {% endif %} - {% if footer_address %} - {% if copyright %}
{% endif %} - {{ footer_address }} - {% endif %} -
- {# powered #} -
- {% block powered %} - {% include "templates/includes/footer/footer_powered.html" %} - {% endblock %} -
-
-
+ {% include "templates/includes/footer/footer_info.html" %}
diff --git a/frappe/templates/includes/footer/footer_grouped_links.html b/frappe/templates/includes/footer/footer_grouped_links.html index 6e20c51279..22cdb10824 100644 --- a/frappe/templates/includes/footer/footer_grouped_links.html +++ b/frappe/templates/includes/footer/footer_grouped_links.html @@ -1,28 +1,34 @@ -{% for page in footer_items if page.child_items %} -
-
- -
- {%- if page.icon -%} - {{ page.label }} - {%- else -%} - {{ page.label }} - {%- endif -%} +
+
+ {% for page in footer_items if page.child_items %} + {# 2 columns to every 5 links, so 5 links get 2 columns, 5-10 links get 4 columns, and so on #} + {%- set cols = frappe.utils.ceil((page.child_items | len) / 5) * 2 -%} +
+
+ +
+ {%- if page.icon -%} + {{ page.label }} + {%- else -%} + {{ page.label }} + {%- endif -%} +
+ +
- - +
+ {% endfor %}
-{% endfor %} diff --git a/frappe/templates/includes/footer/footer_info.html b/frappe/templates/includes/footer/footer_info.html new file mode 100644 index 0000000000..a186247c9a --- /dev/null +++ b/frappe/templates/includes/footer/footer_info.html @@ -0,0 +1,19 @@ +
+
+
+ {% if copyright %} + © {{ copyright }} + {% endif %} + {% if footer_address %} + {% if copyright %}
{% endif %} + {{ footer_address }} + {% endif %} +
+ {# powered #} +
+ {% block powered %} + {% include "templates/includes/footer/footer_powered.html" %} + {% endblock %} +
+
+
diff --git a/frappe/templates/includes/footer/footer_items.html b/frappe/templates/includes/footer/footer_items.html deleted file mode 100644 index 352bde6e27..0000000000 --- a/frappe/templates/includes/footer/footer_items.html +++ /dev/null @@ -1,28 +0,0 @@ -{% for page in footer_items %} -{% if not page.parent_label %} -
- {% if not page.parent_label -%} -
- - {%- if page.child_items -%} -
{{ page.label }}
- - - {%- else -%} -
{{ page.label }}
- - {%- endif -%} -
- {%- endif -%} -
- {% endif %} -{% endfor %} diff --git a/frappe/templates/includes/footer/footer_links.html b/frappe/templates/includes/footer/footer_links.html index e8bfdadb7f..a5939a9635 100644 --- a/frappe/templates/includes/footer/footer_links.html +++ b/frappe/templates/includes/footer/footer_links.html @@ -10,15 +10,15 @@
- {% for item in footer_items if item.label and not (item.parent_label or item.child_items) and not item.right %} + {%- for item in footer_items if item.label and not (item.parent_label or item.child_items) and not item.right %} {{ footer_link(item) }} - {% endfor %} + {% endfor -%}
- {% for item in footer_items if item.label and not (item.parent_label or item.child_items) and item.right %} + {%- for item in footer_items if item.label and not (item.parent_label or item.child_items) and item.right %} {{ footer_link(item) }} - {% endfor %} + {% endfor -%}
diff --git a/frappe/templates/includes/footer/footer_logo_extension.html b/frappe/templates/includes/footer/footer_logo_extension.html new file mode 100644 index 0000000000..17f3218c45 --- /dev/null +++ b/frappe/templates/includes/footer/footer_logo_extension.html @@ -0,0 +1,16 @@ +
+
+
+ {%- if footer_logo -%} +
+ +
+ {%- endif -%} +
+
+ {% block extension %} + {% include "templates/includes/footer/footer_extension.html" %} + {% endblock %} +
+
+
diff --git a/frappe/website/doctype/website_settings/website_settings.js b/frappe/website/doctype/website_settings/website_settings.js index 1575802f5f..0acaa9577f 100644 --- a/frappe/website/doctype/website_settings/website_settings.js +++ b/frappe/website/doctype/website_settings/website_settings.js @@ -83,6 +83,10 @@ frappe.ui.form.on('Website Settings', { }); frappe.ui.form.on('Top Bar Item', { + footer_items_add(frm, cdt, cdn) { + frappe.model.set_value(cdt, cdn, 'right', 0); + }, + parent_label: function(frm, doctype, name) { frm.events.set_parent_options(frm, doctype, name); }, @@ -107,4 +111,4 @@ frappe.tour['Website Settings'] = [ title: __("Disable Signup for your site"), description: __("Check this if you don't want users to sign up for an account on your site. Users won't get desk access unless you explicitly provide it."), } -]; \ No newline at end of file +]; From a1cfcbfb64bc2a41b09c7f3c8af61513bc0beab7 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:00:52 +0530 Subject: [PATCH 012/121] fix: Default styling for h3 --- frappe/public/scss/base.scss | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/frappe/public/scss/base.scss b/frappe/public/scss/base.scss index 0b01a83b02..4190780ece 100644 --- a/frappe/public/scss/base.scss +++ b/frappe/public/scss/base.scss @@ -33,7 +33,7 @@ h1 { h2 { font-size: $font-size-xl; - font-weight: bold; + font-weight: 700; margin-bottom: 0.75rem; @include media-breakpoint-up(sm) { @@ -44,3 +44,15 @@ h2 { } } +h3 { + font-size: $font-size-base; + font-weight: 600; + margin-bottom: 0.5rem; + + @include media-breakpoint-up(sm) { + font-size: $font-size-lg; + } + @include media-breakpoint-up(md) { + font-size: $font-size-xl; + } +} From 6fa69eb23299a64bf21212493f7372e21807830b Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:01:25 +0530 Subject: [PATCH 013/121] fix: overflow auto in grid field --- frappe/public/less/form_grid.less | 1 + 1 file changed, 1 insertion(+) diff --git a/frappe/public/less/form_grid.less b/frappe/public/less/form_grid.less index d9e7d8bceb..8642b327a5 100644 --- a/frappe/public/less/form_grid.less +++ b/frappe/public/less/form_grid.less @@ -73,6 +73,7 @@ padding: 10px 15px; max-height: 200px; border-right: 1px solid @border-color; + overflow: auto; } .editable-form .grid-static-col.bold { From fbbfce51fb7c46b38670ce02c449bda134fadf31 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:01:40 +0530 Subject: [PATCH 014/121] fix: Remove margin bottom of last child in md --- frappe/public/scss/markdown.scss | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/frappe/public/scss/markdown.scss b/frappe/public/scss/markdown.scss index 1cb78dcc62..4b0c20cbc4 100644 --- a/frappe/public/scss/markdown.scss +++ b/frappe/public/scss/markdown.scss @@ -10,6 +10,10 @@ margin-top: 0; } + > :last-child { + margin-bottom: 0; + } + ul, ol { padding-left: 2.5rem; From 8c905437e67001b9b0a5bf8fb3332595cafcf9dd Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:02:02 +0530 Subject: [PATCH 015/121] fix: Add template validation in web_blocks --- frappe/utils/jinja.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/frappe/utils/jinja.py b/frappe/utils/jinja.py index 8653cdc30a..d73c4e0fa4 100644 --- a/frappe/utils/jinja.py +++ b/frappe/utils/jinja.py @@ -197,10 +197,13 @@ def web_blocks(blocks): web_blocks = [] for block in blocks: + if not block.get('template'): + frappe.throw('Web Template is not specified') + doc = { 'doctype': 'Web Page Block', 'web_template': block['template'], - 'web_template_values': block['values'], + 'web_template_values': block.get('values', {}), 'add_top_padding': 1, 'add_bottom_padding': 1, 'add_container': 1, From db69a4eee5e37d3b3f9aa3429d7d1700d70a1087 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:02:29 +0530 Subject: [PATCH 016/121] fix: Syntax highlighting for Custom SCSS field --- frappe/website/doctype/website_theme/website_theme.json | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/frappe/website/doctype/website_theme/website_theme.json b/frappe/website/doctype/website_theme/website_theme.json index 7af6c91281..49d01606cd 100644 --- a/frappe/website/doctype/website_theme/website_theme.json +++ b/frappe/website/doctype/website_theme/website_theme.json @@ -126,7 +126,8 @@ { "fieldname": "custom_scss", "fieldtype": "Code", - "label": "Custom SCSS" + "label": "Custom SCSS", + "options": "SCSS" }, { "fieldname": "light_color", @@ -170,7 +171,7 @@ } ], "links": [], - "modified": "2020-05-16 18:36:22.203519", + "modified": "2020-08-16 23:12:02.863833", "modified_by": "Administrator", "module": "Website", "name": "Website Theme", From f53fab89182babd287d7fcd34bbb8427b4553475 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:03:12 +0530 Subject: [PATCH 017/121] fix: Fields is not mandatory in Web Template --- frappe/website/doctype/web_template/web_template.json | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/frappe/website/doctype/web_template/web_template.json b/frappe/website/doctype/web_template/web_template.json index 18b36b9b81..4ec2a38575 100644 --- a/frappe/website/doctype/web_template/web_template.json +++ b/frappe/website/doctype/web_template/web_template.json @@ -23,8 +23,7 @@ "fieldname": "fields", "fieldtype": "Table", "label": "Fields", - "options": "Web Template Field", - "reqd": 1 + "options": "Web Template Field" }, { "default": "0", @@ -34,7 +33,7 @@ } ], "links": [], - "modified": "2020-05-15 17:50:51.856135", + "modified": "2020-08-17 13:00:11.863241", "modified_by": "Administrator", "module": "Website", "name": "Web Template", From 9afc6a7b9bd9ac16467cdcfba103475b7c6abb86 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:03:49 +0530 Subject: [PATCH 018/121] feat: Support for Table field via Table Break in Web Template --- frappe/website/doctype/web_page/web_page.js | 55 ++++++++++++++++--- .../web_template_field.json | 4 +- 2 files changed, 50 insertions(+), 9 deletions(-) diff --git a/frappe/website/doctype/web_page/web_page.js b/frappe/website/doctype/web_page/web_page.js index b2e06efc79..2964a29989 100644 --- a/frappe/website/doctype/web_page/web_page.js +++ b/frappe/website/doctype/web_page/web_page.js @@ -49,15 +49,57 @@ frappe.ui.form.on('Web Page', { frappe.ui.form.on("Web Page Block", { edit_values(frm, cdt, cdn) { let row = frm.selected_doc; + let values = JSON.parse(row.web_template_values || "{}"); + + function get_fields(doc) { + let normal_fields = []; + let table_fields = []; + + let current_table = null; + for (let df of doc.fields) { + if (current_table) { + current_table.fields = current_table.fields || []; + + if (df.fieldtype != 'Table Break') { + current_table.fields.push(df); + } else { + table_fields.push(df); + current_table = df; + } + } else if (df.fieldtype != 'Table Break') { + normal_fields.push(df); + } else { + table_fields.push(df); + current_table = df; + } + } + + let fields = [ + ...normal_fields, + ...table_fields.map(tf => { + let data = values[tf.fieldname] || []; + return { + label: tf.label, + fieldname: tf.fieldname, + fieldtype: 'Table', + fields: tf.fields.map((df, i) => ({ + ...df, + in_list_view: i <= 1, + columns: tf.fields.length == 1 ? 10 : 5 + })), + data, + get_data: () => data + }; + }) + ]; + + return fields; + } + frappe.model.with_doc("Web Template", row.web_template).then((doc) => { let d = new frappe.ui.Dialog({ title: __("Edit Values"), - fields: doc.fields.map((df) => { - if (df.fieldtype == "Section Break") { - df.collapsible = 1; - } - return df; - }), + fields: get_fields(doc), primary_action(values) { frappe.model.set_value( cdt, @@ -68,7 +110,6 @@ frappe.ui.form.on("Web Page Block", { d.hide(); }, }); - let values = JSON.parse(row.web_template_values || "{}"); d.set_values(values); d.show(); diff --git a/frappe/website/doctype/web_template_field/web_template_field.json b/frappe/website/doctype/web_template_field/web_template_field.json index 900655e207..147cc47da2 100644 --- a/frappe/website/doctype/web_template_field/web_template_field.json +++ b/frappe/website/doctype/web_template_field/web_template_field.json @@ -31,7 +31,7 @@ "fieldtype": "Select", "in_list_view": 1, "label": "Fieldtype", - "options": "Attach Image\nCheck\nData\nInt\nSelect\nSmall Text\nText\nMarkdown Editor\nSection Break\nColumn Break", + "options": "Attach Image\nCheck\nData\nInt\nSelect\nSmall Text\nText\nMarkdown Editor\nSection Break\nColumn Break\nTable Break", "reqd": 1 }, { @@ -48,7 +48,7 @@ ], "istable": 1, "links": [], - "modified": "2020-04-29 14:53:23.192395", + "modified": "2020-08-13 14:21:04.331509", "modified_by": "Administrator", "module": "Website", "name": "Web Template Field", From 6138e0b4bb3eb31dd0b65bd4a1fd4d774d4ae99a Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 17:04:15 +0530 Subject: [PATCH 019/121] fix: empty init file --- .../web_template/section_with_collapsible_content/__init__.py | 0 1 file changed, 0 insertions(+), 0 deletions(-) create mode 100644 frappe/website/web_template/section_with_collapsible_content/__init__.py diff --git a/frappe/website/web_template/section_with_collapsible_content/__init__.py b/frappe/website/web_template/section_with_collapsible_content/__init__.py new file mode 100644 index 0000000000..e69de29bb2 From e010947e389d125d9450a487e3d148e4a601e434 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Mon, 17 Aug 2020 21:31:50 +0530 Subject: [PATCH 020/121] fix: revert overflow auto --- frappe/public/less/form_grid.less | 1 - 1 file changed, 1 deletion(-) diff --git a/frappe/public/less/form_grid.less b/frappe/public/less/form_grid.less index 8642b327a5..d9e7d8bceb 100644 --- a/frappe/public/less/form_grid.less +++ b/frappe/public/less/form_grid.less @@ -73,7 +73,6 @@ padding: 10px 15px; max-height: 200px; border-right: 1px solid @border-color; - overflow: auto; } .editable-form .grid-static-col.bold { From 03aed4d7f17dc7535a7c32da1ac1d0fa903dd389 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 12:57:31 +0530 Subject: [PATCH 021/121] fix: Apply vertical margin only in sidebar layout --- frappe/public/scss/website.scss | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/public/scss/website.scss b/frappe/public/scss/website.scss index 59bbe4f19d..2a46f3650f 100644 --- a/frappe/public/scss/website.scss +++ b/frappe/public/scss/website.scss @@ -128,7 +128,7 @@ color: $light; } -.page-content-wrapper { +.main-column .page-content-wrapper { margin: 2rem 0; } From 6c46cbd4a31bd788784a55fc6ea9ae0e872f2e75 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 12:58:46 +0530 Subject: [PATCH 022/121] fix: Hero image layout reflow on lazy load Before this change the hero image would cause a layout reflow resulting a layout jump when the image is loaded --- frappe/public/scss/page-builder.scss | 1 + 1 file changed, 1 insertion(+) diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index 81c06420bc..506e281af7 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -97,6 +97,7 @@ .hero-image { width: auto; display: none; + flex: 1; object-fit: contain; max-height: 36rem; From b7facef52dcf72fac31f1d3c41ece5f4ae83bf33 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 14:14:56 +0530 Subject: [PATCH 023/121] feat: Section with Features --- frappe/public/scss/page-builder.scss | 103 ++++++++++++++++++ .../section_with_features/__init__.py | 0 .../section_with_features.html | 27 +++++ .../section_with_features.json | 62 +++++++++++ 4 files changed, 192 insertions(+) create mode 100644 frappe/website/web_template/section_with_features/__init__.py create mode 100644 frappe/website/web_template/section_with_features/section_with_features.html create mode 100644 frappe/website/web_template/section_with_features/section_with_features.json diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index 506e281af7..55c67e6311 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -458,3 +458,106 @@ margin-right: auto; } } + +/* Section with Features */ + +.section-features { + display: grid; + + &[data-columns="2"] { + margin-top: 3.75rem; + grid-template-columns: repeat(2, 1fr); + gap: 6rem; + + .feature-title { + font-size: $font-size-xl; + font-weight: bold; + + @include media-breakpoint-up(md) { + font-size: $font-size-2xl; + } + } + + .feature-content { + font-size: $font-size-base; + margin-top: 1.75rem; + margin-bottom: 1.75rem; + + @include media-breakpoint-up(xl) { + font-size: $font-size-lg; + } + } + + .feature-icon { + margin-bottom: 2rem; + width: 3.375rem; + height: 3.375rem; + object-fit: contain; + } + } + + &[data-columns="3"] { + margin-top: 3rem; + grid-template-columns: repeat(3, 1fr); + gap: 4.875rem; + + .feature-title { + font-size: $font-size-lg; + font-weight: 600; + + @include media-breakpoint-up(md) { + font-size: $font-size-xl; + } + } + .feature-content { + font-size: $font-size-base; + margin-top: 1rem; + margin-bottom: 1rem; + } + + .feature-icon { + margin-bottom: 1.75rem; + width: 2.5rem; + height: 2.5rem; + object-fit: contain; + } + } + + &[data-columns="4"] { + margin-top: 2.5rem; + grid-template-columns: repeat(4, 1fr); + gap: 3.75rem; + + .feature-title { + font-size: $font-size-base; + font-weight: 600; + } + + .feature-content { + font-size: $font-size-sm; + margin-top: 0.875rem; + margin-bottom: 0.875rem; + } + + .feature-url { + font-size: $font-size-sm; + } + + .feature-icon { + margin-bottom: 1.5rem; + width: 2.375rem; + height: 2.375rem; + object-fit: contain; + } + } +} + +.section-feature { + display: flex; + flex-direction: column; +} + +.feature-url { + display: inline-block; + margin-top: auto; +} diff --git a/frappe/website/web_template/section_with_features/__init__.py b/frappe/website/web_template/section_with_features/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/frappe/website/web_template/section_with_features/section_with_features.html b/frappe/website/web_template/section_with_features/section_with_features.html new file mode 100644 index 0000000000..e0de409425 --- /dev/null +++ b/frappe/website/web_template/section_with_features/section_with_features.html @@ -0,0 +1,27 @@ +
+ {%- if title -%} +

{{ title }}

+ {%- endif -%} + {%- if subtitle -%} +

{{ subtitle }}

+ {%- endif -%} + +
+ {%- for feature in features -%} +
+ {%- if feature.icon -%} + Icon for {{ feature.title }} + {%- endif -%} + {%- if feature.title -%} +

{{ feature.title }}

+ {%- endif -%} + {%- if feature.content -%} +

{{ feature.content }}

+ {%- endif -%} + {%- if feature.url -%} + Learn more → + {%- endif -%} +
+ {%- endfor -%} +
+
diff --git a/frappe/website/web_template/section_with_features/section_with_features.json b/frappe/website/web_template/section_with_features/section_with_features.json new file mode 100644 index 0000000000..5aa06bffc0 --- /dev/null +++ b/frappe/website/web_template/section_with_features/section_with_features.json @@ -0,0 +1,62 @@ +{ + "creation": "2020-08-18 13:10:11.476496", + "docstatus": 0, + "doctype": "Web Template", + "fields": [ + { + "fieldname": "title", + "fieldtype": "Data", + "label": "Title", + "reqd": 0 + }, + { + "fieldname": "subtitle", + "fieldtype": "Data", + "label": "Subtitle", + "reqd": 0 + }, + { + "fieldname": "columns", + "fieldtype": "Select", + "label": "Columns", + "options": "2\n3\n4", + "reqd": 0 + }, + { + "fieldname": "features", + "fieldtype": "Table Break", + "label": "Features", + "reqd": 0 + }, + { + "fieldname": "title", + "fieldtype": "Data", + "label": "Title", + "reqd": 0 + }, + { + "fieldname": "content", + "fieldtype": "Small Text", + "label": "Content", + "reqd": 0 + }, + { + "fieldname": "url", + "fieldtype": "Data", + "label": "URL", + "reqd": 0 + }, + { + "fieldname": "icon", + "fieldtype": "Attach Image", + "label": "Icon", + "reqd": 0 + } + ], + "idx": 0, + "modified": "2020-08-18 13:48:43.138779", + "modified_by": "Administrator", + "name": "Section with Features", + "owner": "Administrator", + "standard": 1 +} \ No newline at end of file From 3b434251d0ca6b03f27f22f5d3cc6847a840779e Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 16:36:40 +0530 Subject: [PATCH 024/121] feat: Hero align center style --- frappe/public/scss/page-builder.scss | 11 ++++++ frappe/website/web_template/hero/hero.html | 40 +++++++++++----------- 2 files changed, 31 insertions(+), 20 deletions(-) diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index 55c67e6311..354a7371f4 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -25,6 +25,17 @@ } } +.hero.align-center { + h1, .hero-subtitle, .hero-buttons { + text-align: center; + } + + .hero-subtitle { + margin-left: auto; + margin-right: auto; + } +} + .section-description { max-width: 56rem; margin-top: 0.5rem; diff --git a/frappe/website/web_template/hero/hero.html b/frappe/website/web_template/hero/hero.html index 7e2938387d..ca97438873 100644 --- a/frappe/website/web_template/hero/hero.html +++ b/frappe/website/web_template/hero/hero.html @@ -1,24 +1,24 @@ -
-

- {{ title }} -

- {%- if subtitle -%} -

- {{ subtitle }} -

- {%- endif -%} - {%- if primary_action or secondary_action -%} -
- {%- if primary_action -%} - - {{ primary_action_label }} - +
+
+

{{ title }}

+ {%- if subtitle -%} +

+ {{ subtitle }} +

{%- endif -%} - {%- if secondary_action -%} - - {{ secondary_action_label }} - + {%- if primary_action or secondary_action -%} +
+ {%- if primary_action -%} + + {{ primary_action_label }} + + {%- endif -%} + {%- if secondary_action -%} + + {{ secondary_action_label }} + + {%- endif -%} +
{%- endif -%}
- {%- endif -%}
From a73a54723b2c308813c6b538d4ea8bd2e97a9ba0 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 16:37:03 +0530 Subject: [PATCH 025/121] fix: Footer styles --- frappe/public/scss/footer.scss | 5 ++++ .../includes/footer/footer_grouped_links.html | 24 +++++++++---------- 2 files changed, 16 insertions(+), 13 deletions(-) diff --git a/frappe/public/scss/footer.scss b/frappe/public/scss/footer.scss index 9214907fbb..a12fb10033 100644 --- a/frappe/public/scss/footer.scss +++ b/frappe/public/scss/footer.scss @@ -63,11 +63,16 @@ margin-bottom: 0.5rem; } +.footer-grouped-links { + margin-bottom: 2rem; +} + .footer-group-links { display: flex; flex-direction: column; flex-wrap: wrap; max-height: 10rem; + margin-bottom: 0; } .footer-info { diff --git a/frappe/templates/includes/footer/footer_grouped_links.html b/frappe/templates/includes/footer/footer_grouped_links.html index 22cdb10824..0383409090 100644 --- a/frappe/templates/includes/footer/footer_grouped_links.html +++ b/frappe/templates/includes/footer/footer_grouped_links.html @@ -1,21 +1,19 @@
- {% for page in footer_items if page.child_items %} + {% for group in footer_items if group.child_items %} {# 2 columns to every 5 links, so 5 links get 2 columns, 5-10 links get 4 columns, and so on #} - {%- set cols = frappe.utils.ceil((page.child_items | len) / 5) * 2 -%} + {%- set cols = frappe.utils.ceil((group.child_items | len) / 5) * 2 -%}
-
- -
- {%- if page.icon -%} - {{ page.label }} - {%- else -%} - {{ page.label }} - {%- endif -%} -
- +
+
+ {%- if group.icon -%} + {{ group.label }} + {%- else -%} + {{ group.label }} + {%- endif -%} +
+ + From 3a5927d372493859455e68a112bc247d726e147a Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 18:38:25 +0530 Subject: [PATCH 027/121] fix: Responsive styling for Section with Features --- frappe/public/scss/page-builder.scss | 84 ++++++++++++++++--- .../section_with_features.html | 28 ++++--- 2 files changed, 88 insertions(+), 24 deletions(-) diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index 100a0f0be4..b63d1e3e92 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -494,9 +494,17 @@ display: grid; &[data-columns="2"] { - margin-top: 3.75rem; - grid-template-columns: repeat(2, 1fr); - gap: 6rem; + grid-template-columns: repeat(1, 1fr); + gap: 2.5rem; + + @include media-breakpoint-up(sm) { + gap: 3rem; + } + + @include media-breakpoint-up(md) { + grid-template-columns: repeat(2, 1fr); + gap: 6rem; + } .feature-title { font-size: $font-size-xl; @@ -510,13 +518,16 @@ .feature-content { font-size: $font-size-base; margin-top: 1.75rem; - margin-bottom: 1.75rem; @include media-breakpoint-up(xl) { font-size: $font-size-lg; } } + .feature-url { + margin-top: 1.75rem; + } + .feature-icon { margin-bottom: 2rem; width: 3.375rem; @@ -526,9 +537,18 @@ } &[data-columns="3"] { - margin-top: 3rem; - grid-template-columns: repeat(3, 1fr); - gap: 4.875rem; + grid-template-columns: repeat(1, 1fr); + gap: 2rem; + + @include media-breakpoint-up(sm) { + grid-template-columns: repeat(2, 1fr); + gap: 2.5rem; + } + + @include media-breakpoint-up(md) { + grid-template-columns: repeat(3, 1fr); + gap: 4.875rem; + } .feature-title { font-size: $font-size-lg; @@ -538,10 +558,14 @@ font-size: $font-size-xl; } } + .feature-content { font-size: $font-size-base; margin-top: 1rem; - margin-bottom: 1rem; + } + + .feature-url { + margin-top: 1rem; } .feature-icon { @@ -553,9 +577,23 @@ } &[data-columns="4"] { - margin-top: 2.5rem; - grid-template-columns: repeat(4, 1fr); - gap: 3.75rem; + grid-template-columns: repeat(1, 1fr); + gap: 2rem; + + @include media-breakpoint-up(sm) { + grid-template-columns: repeat(2, 1fr); + gap: 2.5rem; + } + + @include media-breakpoint-up(md) { + grid-template-columns: repeat(3, 1fr); + gap: 3rem; + } + + @include media-breakpoint-up(lg) { + grid-template-columns: repeat(4, 1fr); + gap: 3.75rem; + } .feature-title { font-size: $font-size-base; @@ -565,7 +603,10 @@ .feature-content { font-size: $font-size-sm; margin-top: 0.875rem; - margin-bottom: 0.875rem; + } + + .feature-url { + margin-top: 0.875rem; } .feature-url { @@ -581,9 +622,28 @@ } } +.section-title + .section-features, .section-description + .section-features { + &[data-columns="2"] { + margin-top: 3.75rem; + } + + &[data-columns="3"] { + margin-top: 3rem; + } + + &[data-columns="4"] { + margin-top: 2.5rem; + } +} + .section-feature { display: flex; flex-direction: column; + justify-content: space-between; +} + +.feature-title, .feature-content { + margin-bottom: 0; } .feature-url { diff --git a/frappe/website/web_template/section_with_features/section_with_features.html b/frappe/website/web_template/section_with_features/section_with_features.html index e0de409425..0b487a0568 100644 --- a/frappe/website/web_template/section_with_features/section_with_features.html +++ b/frappe/website/web_template/section_with_features/section_with_features.html @@ -9,18 +9,22 @@
{%- for feature in features -%}
- {%- if feature.icon -%} - Icon for {{ feature.title }} - {%- endif -%} - {%- if feature.title -%} -

{{ feature.title }}

- {%- endif -%} - {%- if feature.content -%} -

{{ feature.content }}

- {%- endif -%} - {%- if feature.url -%} - Learn more → - {%- endif -%} +
+ {%- if feature.icon -%} + Icon for {{ feature.title }} + {%- endif -%} + {%- if feature.title -%} +

{{ feature.title }}

+ {%- endif -%} + {%- if feature.content -%} +

{{ feature.content }}

+ {%- endif -%} +
+
+ {%- if feature.url -%} + Learn more → + {%- endif -%} +
{%- endfor -%}
From 629c4afca058bff8c8a59be38cb27581f1061180 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 18:38:52 +0530 Subject: [PATCH 028/121] feat: Section with Embed --- frappe/public/scss/page-builder.scss | 7 +++++ .../section_with_embed/__init__.py | 0 .../section_with_embed.html | 10 ++++++ .../section_with_embed.json | 31 +++++++++++++++++++ 4 files changed, 48 insertions(+) create mode 100644 frappe/website/web_template/section_with_embed/__init__.py create mode 100644 frappe/website/web_template/section_with_embed/section_with_embed.html create mode 100644 frappe/website/web_template/section_with_embed/section_with_embed.json diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index b63d1e3e92..505806f770 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -650,3 +650,10 @@ display: inline-block; margin-top: auto; } + + +/* Section with Embed */ + +.section-with-embed .embed-container { + margin-top: 2rem; +} diff --git a/frappe/website/web_template/section_with_embed/__init__.py b/frappe/website/web_template/section_with_embed/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/frappe/website/web_template/section_with_embed/section_with_embed.html b/frappe/website/web_template/section_with_embed/section_with_embed.html new file mode 100644 index 0000000000..e9205eb100 --- /dev/null +++ b/frappe/website/web_template/section_with_embed/section_with_embed.html @@ -0,0 +1,10 @@ +
+

{{ title }}

+ {%- if subtitle -%} +

{{ subtitle }}

+ {%- endif -%} + +
+ {{ frappe.utils.md_to_html(embed_html) }} +
+
diff --git a/frappe/website/web_template/section_with_embed/section_with_embed.json b/frappe/website/web_template/section_with_embed/section_with_embed.json new file mode 100644 index 0000000000..b3564c5a04 --- /dev/null +++ b/frappe/website/web_template/section_with_embed/section_with_embed.json @@ -0,0 +1,31 @@ +{ + "creation": "2020-08-18 17:45:13.606363", + "docstatus": 0, + "doctype": "Web Template", + "fields": [ + { + "fieldname": "title", + "fieldtype": "Data", + "label": "Title", + "reqd": 0 + }, + { + "fieldname": "subtitle", + "fieldtype": "Data", + "label": "Subtitle", + "reqd": 0 + }, + { + "fieldname": "embed_html", + "fieldtype": "Markdown Editor", + "label": "Embed HTML", + "reqd": 0 + } + ], + "idx": 0, + "modified": "2020-08-18 17:45:13.606363", + "modified_by": "Administrator", + "name": "Section with Embed", + "owner": "Administrator", + "standard": 1 +} \ No newline at end of file From 5305873a376bda2e8f27bd15cbafa88e1c00f9d3 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 18:40:22 +0530 Subject: [PATCH 029/121] fix: Set common css variables on root --- frappe/public/scss/css_variables.scss | 28 +++++++++++++++++++++++++++ frappe/public/scss/website.scss | 1 + 2 files changed, 29 insertions(+) create mode 100644 frappe/public/scss/css_variables.scss diff --git a/frappe/public/scss/css_variables.scss b/frappe/public/scss/css_variables.scss new file mode 100644 index 0000000000..8b1be1e479 --- /dev/null +++ b/frappe/public/scss/css_variables.scss @@ -0,0 +1,28 @@ +:root { + --gray-50: #{$gray-50}; + --gray-100: #{$gray-100}; + --gray-200: #{$gray-200}; + --gray-300: #{$gray-300}; + --gray-400: #{$gray-400}; + --gray-500: #{$gray-500}; + --gray-600: #{$gray-600}; + --gray-700: #{$gray-700}; + --gray-800: #{$gray-800}; + --gray-900: #{$gray-900}; + + --black: #{$black}; + --primary: #{$primary}; + --primary-light: #{$primary-light}; + --light: #{$light}; + + --font-size-xs: #{$font-size-xs}; + --font-size-sm: #{$font-size-sm}; + --font-size-base: #{$font-size-base}; + --font-size-lg: #{$font-size-lg}; + --font-size-xl: #{$font-size-xl}; + --font-size-2xl: #{$font-size-2xl}; + --font-size-3xl: #{$font-size-3xl}; + --font-size-4xl: #{$font-size-4xl}; + --font-size-5xl: #{$font-size-5xl}; + --font-size-6xl: #{$font-size-6xl}; +} diff --git a/frappe/public/scss/website.scss b/frappe/public/scss/website.scss index 2a46f3650f..0fc98e468f 100644 --- a/frappe/public/scss/website.scss +++ b/frappe/public/scss/website.scss @@ -1,5 +1,6 @@ @import '~quill/dist/quill.core'; @import 'variables'; +@import 'css_variables'; @import 'frappe/public/css/font-awesome'; @import '~bootstrap/scss/bootstrap'; @import 'base'; From 4889ce13ce3cef65b3a4f95314d07477ea64c099 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Tue, 18 Aug 2020 18:40:42 +0530 Subject: [PATCH 030/121] fix: Conditionally render subtitle --- .../website/web_template/section_with_cta/section_with_cta.html | 2 ++ 1 file changed, 2 insertions(+) diff --git a/frappe/website/web_template/section_with_cta/section_with_cta.html b/frappe/website/web_template/section_with_cta/section_with_cta.html index 313f49dde3..820fb726aa 100644 --- a/frappe/website/web_template/section_with_cta/section_with_cta.html +++ b/frappe/website/web_template/section_with_cta/section_with_cta.html @@ -1,7 +1,9 @@

{{ title }}

+ {%- if subtitle -%}

{{ subtitle }}

+ {%- endif -%}
{{ cta_label }}
From 65616031e46e2eb749943ebc6a682a6eea12466c Mon Sep 17 00:00:00 2001 From: Rucha Mahabal Date: Thu, 3 Sep 2020 13:59:08 +0530 Subject: [PATCH 031/121] fix: sider issues --- frappe/core/doctype/data_import_legacy/importer.py | 2 +- frappe/desk/query_report.py | 8 ++------ frappe/public/js/frappe/utils/utils.js | 2 +- frappe/public/js/frappe/views/reports/query_report.js | 2 +- 4 files changed, 5 insertions(+), 9 deletions(-) diff --git a/frappe/core/doctype/data_import_legacy/importer.py b/frappe/core/doctype/data_import_legacy/importer.py index f7f196da61..35569c7186 100644 --- a/frappe/core/doctype/data_import_legacy/importer.py +++ b/frappe/core/doctype/data_import_legacy/importer.py @@ -164,7 +164,7 @@ def upload(rows = None, submit_after_import=None, ignore_encoding_errors=False, d[fieldname] = get_datetime(_date + " " + _time) else: d[fieldname] = None - elif df.fieldtype == "Duration": + elif fieldtype == "Duration": d[fieldname] = duration_to_seconds(cstr(d[fieldname])) elif fieldtype in ("Image", "Attach Image", "Attach"): # added file to attachments list diff --git a/frappe/desk/query_report.py b/frappe/desk/query_report.py index dc42228f81..183d1a1371 100644 --- a/frappe/desk/query_report.py +++ b/frappe/desk/query_report.py @@ -8,7 +8,7 @@ import os, json from frappe import _ from frappe.modules import scrub, get_module_path -from frappe.utils import flt, cint, get_html_format, get_url_to_form, gzip_decompress, format_duration, cstr +from frappe.utils import flt, cint, get_html_format, get_url_to_form, gzip_decompress, format_duration from frappe.model.utils import render_include from frappe.translate import send_translations import frappe.desk.reportview @@ -369,7 +369,7 @@ def export_query(): def handle_duration_fieldtype_values(result, columns): for i, col in enumerate(columns): - fieldtype, fieldname = None, None + fieldtype = None if isinstance(col, string_types): col = col.split(":") if len(col) > 1: @@ -381,7 +381,6 @@ def handle_duration_fieldtype_values(result, columns): fieldtype = "Data" else: fieldtype = col.get("fieldtype") - fieldname = col.get("fieldname") if fieldtype == "Duration": for entry in range(0, len(result)): @@ -410,10 +409,7 @@ def build_xlsx_data(columns, data, visible_idx, include_indentation): for idx in range(len(data.columns)): label = columns[idx]["label"] fieldname = columns[idx]["fieldname"] - fieldtype = columns[idx]["fieldtype"] cell_value = row.get(fieldname, row.get(label, "")) - if fieldtype == "Duration": - cell_value = format_duration(value) if cint(include_indentation) and 'indent' in row and idx == 0: cell_value = (' ' * cint(row['indent'])) + cell_value diff --git a/frappe/public/js/frappe/utils/utils.js b/frappe/public/js/frappe/utils/utils.js index d64be06869..b8eeefb046 100644 --- a/frappe/public/js/frappe/utils/utils.js +++ b/frappe/public/js/frappe/utils/utils.js @@ -830,7 +830,7 @@ Object.assign(frappe.utils, { duration_options = { hide_days: 0, hide_seconds: 0 - } + }; } if (value) { let total_duration = frappe.utils.seconds_to_duration(value, duration_options); diff --git a/frappe/public/js/frappe/views/reports/query_report.js b/frappe/public/js/frappe/views/reports/query_report.js index 0817d8cfa5..3d7631371c 100644 --- a/frappe/public/js/frappe/views/reports/query_report.js +++ b/frappe/public/js/frappe/views/reports/query_report.js @@ -1323,7 +1323,7 @@ frappe.views.QueryReport = class QueryReport extends frappe.views.BaseList { .slice(standard_column_count) .map((cell, i) => { if (cell.column.fieldtype === "Duration") { - cell.content = frappe.utils.get_formatted_duration(cell.content) + cell.content = frappe.utils.get_formatted_duration(cell.content); } if (include_indentation && i===0) { cell.content = ' '.repeat(row.meta.indent) + (cell.content || ''); From 6692211705056b52c0f1c5ece3c339ed2df93b2f Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Fri, 21 Aug 2020 14:17:00 +0530 Subject: [PATCH 032/121] feat: Customizable navbar and footer with Web Template --- frappe/hooks.py | 5 ++ frappe/patches.txt | 1 + frappe/public/js/frappe/utils/web_template.js | 71 ++++++++++++++++++ frappe/templates/base.html | 16 +++- frappe/website/doctype/web_page/web_page.js | 75 +------------------ .../doctype/web_template/web_template.json | 9 ++- .../website_settings/website_settings.js | 37 +++++++++ .../website_settings/website_settings.json | 51 ++++++++++++- .../website_settings/website_settings.py | 3 +- .../web_template/standard_footer/__init__.py | 0 .../standard_footer/standard_footer.html | 1 + .../standard_footer/standard_footer.json | 13 ++++ .../web_template/standard_navbar/__init__.py | 0 .../standard_navbar/standard_navbar.html | 1 + .../standard_navbar/standard_navbar.json | 13 ++++ 15 files changed, 217 insertions(+), 79 deletions(-) create mode 100644 frappe/public/js/frappe/utils/web_template.js create mode 100644 frappe/website/web_template/standard_footer/__init__.py create mode 100644 frappe/website/web_template/standard_footer/standard_footer.html create mode 100644 frappe/website/web_template/standard_footer/standard_footer.json create mode 100644 frappe/website/web_template/standard_navbar/__init__.py create mode 100644 frappe/website/web_template/standard_navbar/standard_navbar.html create mode 100644 frappe/website/web_template/standard_navbar/standard_navbar.json diff --git a/frappe/hooks.py b/frappe/hooks.py index 7ecc199814..8e16261faa 100644 --- a/frappe/hooks.py +++ b/frappe/hooks.py @@ -43,6 +43,11 @@ app_include_css = [ "assets/css/report.min.css", ] +doctype_js = { + "Web Page": "public/js/frappe/utils/web_template.js", + "Website Settings": "public/js/frappe/utils/web_template.js" +} + web_include_js = [ "website_script.js" ] diff --git a/frappe/patches.txt b/frappe/patches.txt index 35389eee43..1042ba933a 100644 --- a/frappe/patches.txt +++ b/frappe/patches.txt @@ -307,3 +307,4 @@ frappe.patches.v13_0.rename_notification_fields frappe.patches.v13_0.remove_duplicate_navbar_items frappe.patches.v13_0.enable_custom_script frappe.patches.v13_0.update_newsletter_content_type +execute:frappe.db.set_value('Website Settings', 'Website Settings', {'navbar_template': 'Standard Navbar', 'footer_template': 'Standard Footer'}) diff --git a/frappe/public/js/frappe/utils/web_template.js b/frappe/public/js/frappe/utils/web_template.js new file mode 100644 index 0000000000..64cf17b2d7 --- /dev/null +++ b/frappe/public/js/frappe/utils/web_template.js @@ -0,0 +1,71 @@ +function open_web_template_values_editor(template, current_values = {}) { + return new Promise(resolve => { + frappe.model.with_doc("Web Template", template).then((doc) => { + let d = new frappe.ui.Dialog({ + title: __("Edit Values"), + fields: get_fields(doc), + primary_action(values) { + d.hide(); + resolve(values); + }, + }); + d.set_values(current_values); + d.show(); + + d.sections.forEach((sect) => { + let fields_with_value = sect.fields_list.filter( + (field) => current_values[field.df.fieldname] + ); + + if (fields_with_value.length) { + sect.collapse(false); + } + }); + }); + }); + + function get_fields(doc) { + let normal_fields = []; + let table_fields = []; + + let current_table = null; + for (let df of doc.fields) { + if (current_table) { + current_table.fields = current_table.fields || []; + + if (df.fieldtype != 'Table Break') { + current_table.fields.push(df); + } else { + table_fields.push(df); + current_table = df; + } + } else if (df.fieldtype != 'Table Break') { + normal_fields.push(df); + } else { + table_fields.push(df); + current_table = df; + } + } + + let fields = [ + ...normal_fields, + ...table_fields.map(tf => { + let data = current_values[tf.fieldname] || []; + return { + label: tf.label, + fieldname: tf.fieldname, + fieldtype: 'Table', + fields: tf.fields.map((df, i) => ({ + ...df, + in_list_view: i <= 1, + columns: tf.fields.length == 1 ? 10 : 5 + })), + data, + get_data: () => data + }; + }) + ]; + + return fields; + } +} diff --git a/frappe/templates/base.html b/frappe/templates/base.html index cf55a29b54..6893842b95 100644 --- a/frappe/templates/base.html +++ b/frappe/templates/base.html @@ -68,7 +68,13 @@ {%- endblock -%} {%- block navbar -%} - {% include "templates/includes/navbar/navbar.html" %} + {{ web_blocks([{ + 'template': navbar_template or 'Standard Navbar', + 'values': _context_dict, + 'add_container': 0, + 'add_top_padding': 0, + 'add_bottom_padding': 0, + }]) }} {%- endblock -%} {% block content %} @@ -76,7 +82,13 @@ {% endblock %} {%- block footer -%} - {% include "templates/includes/footer/footer.html" %} + {{ web_blocks([{ + 'template': footer_template or 'Standard Footer', + 'values': _context_dict, + 'add_container': 0, + 'add_top_padding': 0, + 'add_bottom_padding': 0, + }]) }} {%- endblock -%} {% block base_scripts %} diff --git a/frappe/website/doctype/web_page/web_page.js b/frappe/website/doctype/web_page/web_page.js index 2964a29989..ab4f534c61 100644 --- a/frappe/website/doctype/web_page/web_page.js +++ b/frappe/website/doctype/web_page/web_page.js @@ -50,79 +50,10 @@ frappe.ui.form.on("Web Page Block", { edit_values(frm, cdt, cdn) { let row = frm.selected_doc; let values = JSON.parse(row.web_template_values || "{}"); - - function get_fields(doc) { - let normal_fields = []; - let table_fields = []; - - let current_table = null; - for (let df of doc.fields) { - if (current_table) { - current_table.fields = current_table.fields || []; - - if (df.fieldtype != 'Table Break') { - current_table.fields.push(df); - } else { - table_fields.push(df); - current_table = df; - } - } else if (df.fieldtype != 'Table Break') { - normal_fields.push(df); - } else { - table_fields.push(df); - current_table = df; - } - } - - let fields = [ - ...normal_fields, - ...table_fields.map(tf => { - let data = values[tf.fieldname] || []; - return { - label: tf.label, - fieldname: tf.fieldname, - fieldtype: 'Table', - fields: tf.fields.map((df, i) => ({ - ...df, - in_list_view: i <= 1, - columns: tf.fields.length == 1 ? 10 : 5 - })), - data, - get_data: () => data - }; - }) - ]; - - return fields; - } - - frappe.model.with_doc("Web Template", row.web_template).then((doc) => { - let d = new frappe.ui.Dialog({ - title: __("Edit Values"), - fields: get_fields(doc), - primary_action(values) { - frappe.model.set_value( - cdt, - cdn, - "web_template_values", - JSON.stringify(values) - ); - d.hide(); - }, + open_web_template_values_editor(row.web_template, values) + .then(new_values => { + frappe.model.set_value(cdt, cdn, "web_template_values", JSON.stringify(new_values)); }); - d.set_values(values); - d.show(); - - d.sections.forEach((sect) => { - let fields_with_value = sect.fields_list.filter( - (field) => values[field.df.fieldname] - ); - - if (fields_with_value.length) { - sect.collapse(false); - } - }); - }); }, }); diff --git a/frappe/website/doctype/web_template/web_template.json b/frappe/website/doctype/web_template/web_template.json index 4ec2a38575..656d02b134 100644 --- a/frappe/website/doctype/web_template/web_template.json +++ b/frappe/website/doctype/web_template/web_template.json @@ -7,6 +7,7 @@ "editable_grid": 1, "engine": "InnoDB", "field_order": [ + "type", "standard", "template", "fields" @@ -30,10 +31,16 @@ "fieldname": "standard", "fieldtype": "Check", "label": "Standard" + }, + { + "fieldname": "type", + "fieldtype": "Select", + "label": "Type", + "options": "Section\nNavbar\nFooter" } ], "links": [], - "modified": "2020-08-17 13:00:11.863241", + "modified": "2020-08-19 17:50:30.948659", "modified_by": "Administrator", "module": "Website", "name": "Web Template", diff --git a/frappe/website/doctype/website_settings/website_settings.js b/frappe/website/doctype/website_settings/website_settings.js index 0acaa9577f..be48761b7a 100644 --- a/frappe/website/doctype/website_settings/website_settings.js +++ b/frappe/website/doctype/website_settings/website_settings.js @@ -1,4 +1,17 @@ frappe.ui.form.on('Website Settings', { + setup(frm) { + frm.set_query('navbar_template', () => ({ + filters: { + type: 'Navbar' + } + })); + frm.set_query('footer_template', () => ({ + filters: { + type: 'Footer' + } + })); + }, + refresh: function(frm) { frm.add_custom_button(__('View Website'), () => { window.open('/', '_blank'); @@ -80,6 +93,30 @@ frappe.ui.form.on('Website Settings', { } return main_items.join('\n'); }, + + edit_navbar_template_values(frm) { + frm.events.edit_template_values(frm, 'navbar_template'); + }, + + edit_footer_template_values(frm) { + frm.events.edit_template_values(frm, 'footer_template'); + }, + + edit_template_values(frm, template_field) { + let values_field = template_field + '_values'; + let template = frm.doc[template_field]; + if (!template) { + frappe.show_alert(__('Please select {0}', [frm.get_docfield(template_field).label])); + return; + } + let values = JSON.parse(frm.doc[values_field] || "{}"); + open_web_template_values_editor(template, values) + .then(new_values => { + frm.set_value(values_field, JSON.stringify(new_values)); + }); + } + + }); frappe.ui.form.on('Top Bar Item', { diff --git a/frappe/website/doctype/website_settings/website_settings.json b/frappe/website/doctype/website_settings/website_settings.json index 65a45c1796..6ad0f2cd01 100644 --- a/frappe/website/doctype/website_settings/website_settings.json +++ b/frappe/website/doctype/website_settings/website_settings.json @@ -28,6 +28,9 @@ "navbar_search", "hide_login", "top_bar_items", + "navbar_template", + "navbar_template_values", + "edit_navbar_template_values", "call_to_action", "call_to_action_url", "banner", @@ -37,6 +40,9 @@ "copyright", "address", "footer_items", + "footer_template", + "footer_template_values", + "edit_footer_template_values", "hide_footer_signup", "integrations", "enable_view_tracking", @@ -137,7 +143,7 @@ "collapsible": 1, "fieldname": "top_bar", "fieldtype": "Section Break", - "label": "Top Bar" + "label": "Navbar" }, { "default": "0", @@ -357,14 +363,53 @@ "fieldname": "hide_login", "fieldtype": "Check", "label": "Hide Login" + }, + { + "fieldname": "navbar_template", + "fieldtype": "Link", + "label": "Navbar Template", + "options": "Web Template" + }, + { + "fieldname": "navbar_template_values", + "fieldtype": "Code", + "hidden": 1, + "label": "Navbar Template Values", + "options": "JSON" + }, + { + "depends_on": "eval:doc.navbar_template && doc.navbar_template !== 'Standard Navbar'", + "fieldname": "edit_navbar_template_values", + "fieldtype": "Button", + "label": "Edit Values" + }, + { + "fieldname": "footer_template", + "fieldtype": "Link", + "label": "Footer Template", + "options": "Web Template" + }, + { + "fieldname": "footer_template_values", + "fieldtype": "Code", + "hidden": 1, + "label": "Footer Template Values", + "options": "JSON" + }, + { + "depends_on": "eval:doc.footer_template && doc.footer_template !== 'Standard Footer'", + "fieldname": "edit_footer_template_values", + "fieldtype": "Button", + "label": "Edit Values" } ], "icon": "fa fa-cog", "idx": 1, + "index_web_pages_for_search": 1, "issingle": 1, "links": [], "max_attachments": 10, - "modified": "2020-08-07 13:34:19.181561", + "modified": "2020-08-21 14:02:55.168829", "modified_by": "Administrator", "module": "Website", "name": "Website Settings", @@ -388,4 +433,4 @@ "sort_field": "modified", "sort_order": "ASC", "track_changes": 1 -} \ No newline at end of file +} diff --git a/frappe/website/doctype/website_settings/website_settings.py b/frappe/website/doctype/website_settings/website_settings.py index 71857c9cbe..4a5c8bcf90 100644 --- a/frappe/website/doctype/website_settings/website_settings.py +++ b/frappe/website/doctype/website_settings/website_settings.py @@ -119,7 +119,8 @@ def get_website_settings(context=None): for k in ["banner_html", "banner_image", "brand_html", "copyright", "twitter_share_via", "facebook_share", "google_plus_one", "twitter_share", "linked_in_share", "disable_signup", "hide_footer_signup", "head_html", "title_prefix", - "navbar_search", "enable_view_tracking", "footer_logo", "call_to_action", "call_to_action_url"]: + "navbar_template", "footer_template", "navbar_search", "enable_view_tracking", + "footer_logo", "call_to_action", "call_to_action_url"]: if hasattr(settings, k): context[k] = settings.get(k) diff --git a/frappe/website/web_template/standard_footer/__init__.py b/frappe/website/web_template/standard_footer/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/frappe/website/web_template/standard_footer/standard_footer.html b/frappe/website/web_template/standard_footer/standard_footer.html new file mode 100644 index 0000000000..bfea3d948a --- /dev/null +++ b/frappe/website/web_template/standard_footer/standard_footer.html @@ -0,0 +1 @@ +{% include "templates/includes/footer/footer.html" %} diff --git a/frappe/website/web_template/standard_footer/standard_footer.json b/frappe/website/web_template/standard_footer/standard_footer.json new file mode 100644 index 0000000000..c0567f8b44 --- /dev/null +++ b/frappe/website/web_template/standard_footer/standard_footer.json @@ -0,0 +1,13 @@ +{ + "creation": "2020-08-21 13:55:08.557497", + "docstatus": 0, + "doctype": "Web Template", + "fields": [], + "idx": 0, + "modified": "2020-08-21 13:55:11.672764", + "modified_by": "Administrator", + "name": "Standard Footer", + "owner": "Administrator", + "standard": 1, + "type": "Footer" +} \ No newline at end of file diff --git a/frappe/website/web_template/standard_navbar/__init__.py b/frappe/website/web_template/standard_navbar/__init__.py new file mode 100644 index 0000000000..e69de29bb2 diff --git a/frappe/website/web_template/standard_navbar/standard_navbar.html b/frappe/website/web_template/standard_navbar/standard_navbar.html new file mode 100644 index 0000000000..46a0099e0b --- /dev/null +++ b/frappe/website/web_template/standard_navbar/standard_navbar.html @@ -0,0 +1 @@ +{% include "templates/includes/navbar/navbar.html" %} diff --git a/frappe/website/web_template/standard_navbar/standard_navbar.json b/frappe/website/web_template/standard_navbar/standard_navbar.json new file mode 100644 index 0000000000..c1eb6324a2 --- /dev/null +++ b/frappe/website/web_template/standard_navbar/standard_navbar.json @@ -0,0 +1,13 @@ +{ + "creation": "2020-08-19 17:49:37.288380", + "docstatus": 0, + "doctype": "Web Template", + "fields": [], + "idx": 0, + "modified": "2020-08-19 17:50:44.342765", + "modified_by": "Administrator", + "name": "Standard Navbar", + "owner": "Administrator", + "standard": 1, + "type": "Navbar" +} \ No newline at end of file From 67b9b0dd1410b0b3caff7ccc88df9fb819f23b36 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Fri, 21 Aug 2020 14:17:34 +0530 Subject: [PATCH 033/121] fix: Show Edit DocType button in Singles Form in dev mode --- frappe/public/js/frappe/form/toolbar.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/frappe/public/js/frappe/form/toolbar.js b/frappe/public/js/frappe/form/toolbar.js index ab62723161..f61b1bbc8c 100644 --- a/frappe/public/js/frappe/form/toolbar.js +++ b/frappe/public/js/frappe/form/toolbar.js @@ -272,12 +272,12 @@ frappe.ui.form.Toolbar = Class.extend({ }); } - if (frappe.user_roles.includes("System Manager") && me.frm.meta.issingle === 0) { + if (frappe.user_roles.includes("System Manager")) { let is_doctype_form = me.frm.doctype === 'DocType'; let doctype = is_doctype_form ? me.frm.docname : me.frm.doctype; let is_doctype_custom = is_doctype_form ? me.frm.doc.custom : false; - if (doctype != 'DocType' && !is_doctype_custom) { + if (doctype != 'DocType' && !is_doctype_custom && me.frm.meta.issingle === 0) { this.page.add_menu_item(__("Customize"), function() { if (me.frm.meta && me.frm.meta.custom) { frappe.set_route('Form', 'DocType', doctype); From 58a819c305235eda0a8327abe5cdc45c137f6aaf Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Fri, 21 Aug 2020 14:17:52 +0530 Subject: [PATCH 034/121] fix: import --- frappe/utils/jinja.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/frappe/utils/jinja.py b/frappe/utils/jinja.py index d73c4e0fa4..277a36c7fe 100644 --- a/frappe/utils/jinja.py +++ b/frappe/utils/jinja.py @@ -192,13 +192,13 @@ def inspect(var, render=True): return get_jenv().from_string(html).render(context) def web_blocks(blocks): - from frappe import get_doc + from frappe import get_doc, throw from frappe.website.doctype.web_page.web_page import get_web_blocks_html web_blocks = [] for block in blocks: if not block.get('template'): - frappe.throw('Web Template is not specified') + throw('Web Template is not specified') doc = { 'doctype': 'Web Page Block', From c555e819d5a85d5f8cb10ed16387161cc5db5742 Mon Sep 17 00:00:00 2001 From: Faris Ansari Date: Fri, 21 Aug 2020 15:36:14 +0530 Subject: [PATCH 035/121] feat: Extended spacing scale --- frappe/public/scss/page-builder.scss | 11 ++++++++ frappe/public/scss/variables.scss | 28 +++++++++++++++++++ frappe/templates/includes/blog/blogger.html | 4 +-- .../templates/includes/comments/comment.html | 2 +- .../templates/includes/comments/comments.html | 4 +-- frappe/templates/includes/search_result.html | 2 +- .../templates/includes/search_template.html | 4 +-- frappe/templates/includes/web_sidebar.html | 2 +- .../help_article/templates/help_article.html | 2 +- .../doctype/web_form/templates/web_form.html | 2 +- .../hero_with_right_image.html | 4 +-- .../section_with_cards.html | 6 ++-- .../section_with_cta/section_with_cta.html | 2 +- .../section_with_tabs/section_with_tabs.html | 4 +-- 14 files changed, 58 insertions(+), 19 deletions(-) diff --git a/frappe/public/scss/page-builder.scss b/frappe/public/scss/page-builder.scss index 505806f770..e0b7a4c87b 100644 --- a/frappe/public/scss/page-builder.scss +++ b/frappe/public/scss/page-builder.scss @@ -97,12 +97,23 @@ } } +.section[data-section-template="Hero with Right Image"] { + overflow-x: hidden; +} + .hero-with-right-image { position: relative; + display: flex; + flex-wrap: nowrap; .hero-content { display: flex; align-items: center; + flex: 0 0 100%; + + @include media-breakpoint-up(md) { + flex: 0 0 60%; + } } .hero-image { diff --git a/frappe/public/scss/variables.scss b/frappe/public/scss/variables.scss index 1339af29a9..774ab46528 100644 --- a/frappe/public/scss/variables.scss +++ b/frappe/public/scss/variables.scss @@ -60,6 +60,34 @@ $grid-breakpoints: ( 2xl: 1440px ) !default; +$spacers: ( + 0: 0, + 1: 0.25rem, + 2: 0.5rem, + 3: 0.75rem, + 4: 1rem, + 5: 1.25rem, + 6: 1.5rem, + 8: 2rem, + 10: 2.5rem, + 12: 3rem, + 14: 3.5rem, + 16: 4rem, + 18: 4.5rem, + 20: 5rem, + 22: 5.5rem, + 24: 6rem, + 28: 7rem, + 32: 8rem, + 36: 9rem, + 40: 10rem, + 44: 11rem, + 48: 12rem, + 52: 13rem, + 56: 14rem, + 64: 16rem, +); + @import '~bootstrap/scss/functions'; @import '~bootstrap/scss/variables'; @import "~bootstrap/scss/mixins"; diff --git a/frappe/templates/includes/blog/blogger.html b/frappe/templates/includes/blog/blogger.html index ef8f8257e8..6963cc7361 100644 --- a/frappe/templates/includes/blog/blogger.html +++ b/frappe/templates/includes/blog/blogger.html @@ -1,7 +1,7 @@ {% from "frappe/templates/includes/macros.html" import square_image_with_fallback %}
- {{ square_image_with_fallback(src=blogger_info.avatar, size='small', alt=blogger_info.full_name, class='align-self-start mr-3 rounded') }} + {{ square_image_with_fallback(src=blogger_info.avatar, size='small', alt=blogger_info.full_name, class='align-self-start mr-4 rounded') }}
{{ blogger_info.full_name }} @@ -10,4 +10,4 @@

{{ blogger_info.bio }}

{% endif %}
-
\ No newline at end of file +
diff --git a/frappe/templates/includes/comments/comment.html b/frappe/templates/includes/comments/comment.html index 1deb49bb3e..08a2b79ee6 100644 --- a/frappe/templates/includes/comments/comment.html +++ b/frappe/templates/includes/comments/comment.html @@ -1,7 +1,7 @@ {% from "frappe/templates/includes/macros.html" import square_image_with_fallback %}
- {{ square_image_with_fallback(src=frappe.get_gravatar(comment.comment_email or comment.sender), size='extra-small', alt=comment.sender_full_name, class='align-self-start mr-3') }} + {{ square_image_with_fallback(src=frappe.get_gravatar(comment.comment_email or comment.sender), size='extra-small', alt=comment.sender_full_name, class='align-self-start mr-4') }}
diff --git a/frappe/templates/includes/comments/comments.html b/frappe/templates/includes/comments/comments.html index ffd09523af..c490bedd72 100644 --- a/frappe/templates/includes/comments/comments.html +++ b/frappe/templates/includes/comments/comments.html @@ -1,6 +1,6 @@ -
+
{% if comment_text %} -
{{ comment_text }}
+
{{ comment_text }}
{% endif %} {% if not comment_list %}
diff --git a/frappe/templates/includes/search_result.html b/frappe/templates/includes/search_result.html index a34dd84607..21a0c33374 100644 --- a/frappe/templates/includes/search_result.html +++ b/frappe/templates/includes/search_result.html @@ -1,5 +1,5 @@ {% for d in results %} -
+
{{ d.title }}

{{ d.preview }}

diff --git a/frappe/templates/includes/search_template.html b/frappe/templates/includes/search_template.html index 91a9a1dc47..cf2cb33ac1 100644 --- a/frappe/templates/includes/search_template.html +++ b/frappe/templates/includes/search_template.html @@ -4,7 +4,7 @@
{% if title %} -

{{ title }}

+

{{ title }}

{% endif %} {% include "templates/includes/search_result.html" %} @@ -15,7 +15,7 @@
{%- endmacro %} -
+
diff --git a/frappe/templates/includes/web_sidebar.html b/frappe/templates/includes/web_sidebar.html index 86893b1310..6a261c8113 100644 --- a/frappe/templates/includes/web_sidebar.html +++ b/frappe/templates/includes/web_sidebar.html @@ -14,7 +14,7 @@ {{ _(item.title or item.label) }} {% else %} - + diff --git a/frappe/website/doctype/help_article/templates/help_article.html b/frappe/website/doctype/help_article/templates/help_article.html index 1b4bf123e6..92bb399797 100644 --- a/frappe/website/doctype/help_article/templates/help_article.html +++ b/frappe/website/doctype/help_article/templates/help_article.html @@ -18,7 +18,7 @@


{{ _("More articles on {0}").format(category.name) }}

-
+