From b5bf159df39b73b57d33de9a4f1ff01c86727108 Mon Sep 17 00:00:00 2001 From: daria <95225431+dariiing@users.noreply.github.com> Date: Thu, 10 Jul 2025 12:39:51 +0300 Subject: [PATCH] reformat code --- customize.dist/pages/features.js | 7 ++++--- customize.dist/pages/index.js | 2 +- www/common/common-icons.js | 25 +++++++++++++++++++++-- www/common/common-ui-elements.js | 5 +++-- www/common/drive-ui.js | 34 +++++++++++++++++--------------- 5 files changed, 49 insertions(+), 24 deletions(-) diff --git a/customize.dist/pages/features.js b/customize.dist/pages/features.js index 88985f6db..0df393f6d 100644 --- a/customize.dist/pages/features.js +++ b/customize.dist/pages/features.js @@ -13,8 +13,9 @@ define([ '/common/common-ui-elements.js', '/common/common-constants.js', '/common/pad-types.js', - '/common/extensions.js' -], function ($, h, Msg, AppConfig, LocalStore, Pages, Config, UIElements, Constants, PadTypes, Extensions) { + '/common/extensions.js', + '/common/common-icons.js' +], function ($, h, Msg, AppConfig, LocalStore, Pages, Config, UIElements, Constants, PadTypes, Extensions, Icons) { return function () { document.title = Msg.features; Msg.features_f_apps_note = PadTypes.availableTypes.map(function (app) { @@ -27,7 +28,7 @@ define([ var groupItemTemplate = function (title, content) { return h('li.list-group-item', [ - h('i', {'data-lucide':'check', 'aria-hidden':'true'}), + Icons.get('check'), h('div.cp-content', [ h('div.cp-feature', title), h('div.cp-note', content), diff --git a/customize.dist/pages/index.js b/customize.dist/pages/index.js index 76593dce6..100074f35 100644 --- a/customize.dist/pages/index.js +++ b/customize.dist/pages/index.js @@ -213,7 +213,7 @@ define([ ]), h('div.cp-app-drive', [ h('a.cp-drive-btn', {'href': '/drive/'}, [ - h('i', {'data-lucide':'hard-drive', 'aria-hidden':'true'}), + Icons.get('hdd'), Msg.team_cat_drive ]), extraButtons diff --git a/www/common/common-icons.js b/www/common/common-icons.js index ebbce934e..d363a22db 100644 --- a/www/common/common-icons.js +++ b/www/common/common-icons.js @@ -4,7 +4,7 @@ define([ const Icons = {}; const map = { - "drive-trash-empty": "trash-2", + "drive-trash": "trash-2", "drive-search": "search", "add": "plus", "features": "info", @@ -15,7 +15,28 @@ define([ "donate": "hand-coins", "login": "log-in", "register": "user", - "chevron-left": "chevron-left" + "chevron-left": "chevron-left", + "check": "check", + "hdd": "hard-drive", + "grid": "layout-grid", + "list": "list", + "filter": "list-filter-plus", + "folder-open": "folder-open", + "drive-recent": "clock", + "drive-expand": "square-plus", + "drive-owned-document": "id-card", + "share": "share-2", + "drive-password-document": "file-lock", + "ellipsis-vertical": "ellipsis-vertical", + "ellipsis-horizontal": 'ellipsis', + "toolbar-insert": "file-image", + "history-prev": 'arrow-left', + "history-next": "arrow-right", + "history-fast-next": "arrow-right-to-line", + "history-fast-prev": "arrow-left-to-line", + "history-timeline-position": "flag-triangle-right", + "history-restore": "archive-restore", + "close": "x" }; Icons.get = (name, attrs = {}) => { diff --git a/www/common/common-ui-elements.js b/www/common/common-ui-elements.js index 9809739c7..78f0389b6 100644 --- a/www/common/common-ui-elements.js +++ b/www/common/common-ui-elements.js @@ -21,10 +21,11 @@ define([ '/common/inner/invitation.js', '/common/visible.js', '/common/pad-types.js', + '/common/common-icons.js', 'css!/customize/fonts/cptools/style.css', ], function ($, Config, Broadcast, Util, Hash, Language, UI, Constants, Feedback, h, Clipboard, - Messages, AppConfig, Pages, NThen, InviteInner, Visible, PadTypes) { + Messages, AppConfig, Pages, NThen, InviteInner, Visible, PadTypes, Icons) { var UIElements = {}; var urlArgs = Config.requireConf.urlArgs; @@ -907,7 +908,7 @@ define([ button = $(h('button.cp-toolbar-mediatag', { //title: Messages.filePickerButton, // TODO display if the label text is collapsed }, [ - h('i', {'data-lucide': 'file-image', 'aria-hidden':'true'}), + Icons.get('toolbar-insert'), h('span.cp-toolbar-name', Messages.toolbar_insert) ])).click(common.prepareFeedback(type)); break; diff --git a/www/common/drive-ui.js b/www/common/drive-ui.js index 9061217ed..61fa01b0d 100644 --- a/www/common/drive-ui.js +++ b/www/common/drive-ui.js @@ -26,7 +26,8 @@ define([ '/customize/pages.js', '/common/pad-types.js', '/common/onlyoffice/broken-formats.js', - '/customize/fonts/lucide.js' + '/customize/fonts/lucide.js', + '/common/common-icons.js' ], function ( $, ApiConfig, @@ -49,7 +50,8 @@ define([ Pages, PadTypes, BrokenFormats, - Lucide) + Lucide, + Icons) { var APP = window.APP = { @@ -120,7 +122,7 @@ define([ var $fileMenuIcon = $('', {"class": "fa fa-ellipsis-h"}); //var $folderIcon = $('', {src: "/customize/images/icons/folder.svg", "class": "folder icon"}); var $folderEmptyIcon = $folderIcon.clone(); - var $folderOpenedIcon = $('', {'data-lucide': 'folder-open', 'aria-hidden': 'true', "class": "cp-app-drive-icon-folder"}); + var $folderOpenedIcon = $(Icons.get('folder-open')); //var $folderOpenedIcon = $('', {src: "/customize/images/icons/folderOpen.svg", "class": "folder icon"}); var $folderOpenedEmptyIcon = $folderOpenedIcon.clone(); var $sharedFolderIcon = $('', {"class": faSharedFolder + " cptools cp-app-drive-icon-folder"}); @@ -128,26 +130,26 @@ define([ //var $upIcon = $('', {"class": "fa fa-arrow-circle-up"}); var $unsortedIcon = $('', {"class": "fa fa-files-o"}); var $templateIcon = $('', {"class": "cptools cptools-template"}); - var $recentIcon = $('', {'data-lucide': 'clock', 'aria-hidden': 'true' }); + var $recentIcon = $(Icons.get('drive-recent')); var $trashIcon = $('', {"class": "fa " + faTrash}); - var $trashEmptyIcon = $('', {"data-lucide": "trash-2", 'aria-hidden': 'true' }); + var $trashEmptyIcon = $(Icons.get('drive-trash')); //var $collapseIcon = $('', {"class": "fa fa-minus-square-o cp-app-drive-icon-expcol"}); - var $expandIcon = $('', {'data-lucide': 'square-plus','class':'cp-app-drive-icon-expcol', 'aria-hidden': 'true' }); + var $expandIcon = $(Icons.get('drive-expand', {'class': 'cp-app-drive-icon-expcol'})); //var $listIcon = $('