Feature/1.0 check web dev 50265
dashboard and dashboard setting
{{ resolvedDiscussionCount }}/{{ discussionCount }} {{ resolvedCountText }} resolved
-
Toggle commit list
-
1 var DashboardSetting = {}; -
DeveloperEdited by Takumi Imai
一番上にファイルの肥大化とファイル理解のためにファイルの説明をコメントで書いてください。 /**
- ここに説明 *
- @since cms:1.4.3.2&1.4.3.3 web:1.0 */
-
-
1 var DashboardSetting = {}; 2 DashboardSetting.elementItems = { 3 newReport: { id: 'settingNewReport', name: 'chk-new', enabled: true }, 4 continousWork: { id: 'settingContinousWork', name: 'chk-proccess', enabled: true }, 5 reportWarning: { id: 'settingReportWarning', name: 'chk-alert', enabled: true }, 6 dashboardHome: { id: 'settingDashboardHome', name: 'chk-home', enabled: false }, 7 }; 8 9 DashboardSetting.baseApiUrl = CONSTANT.URL.CMS.BASE + ClientData.userInfo_accountPath() + CONSTANT.URL.CMS.API.BASE + 'dashboardSetting/'; 10 DashboardSetting.saveSettingApiUrl = DashboardSetting.baseApiUrl + 'saveSetting'; -
Developer
一か所しか使ってないので共通化しなくても大丈夫です。 DashboardSetting.saveSettingの中に書いた方が良いと思います
-
-
-
abvw/common/js/newdash/common.js 0 → 100755
98 } 99 // 開閉 100 $(this).toggleClass("hide"); 101 }); 102 103 // トグル切り替え 104 $(".toggle").on("click", function() { 105 $(this).toggleClass("checked"); 106 if(!$('input[name="check"]').prop("checked")) { 107 $(".toggle input").prop("checked", true); 108 } else { 109 $(".toggle input").prop("checked", false); 110 } 111 }); 112 113 -
Developer
この上の処置は元々あったものを書いてるそうです(藤川に聴きました) そのため使っていない場合 or 使っているが元の処理がある場合この上のロジックを消してください
-
-
1 var DashboardSetting = {}; 2 DashboardSetting.elementItems = { 3 newReport: { id: 'settingNewReport', name: 'chk-new', enabled: true }, 4 continousWork: { id: 'settingContinousWork', name: 'chk-proccess', enabled: true }, 5 reportWarning: { id: 'settingReportWarning', name: 'chk-alert', enabled: true }, 6 dashboardHome: { id: 'settingDashboardHome', name: 'chk-home', enabled: false }, 7 }; 8 9 DashboardSetting.baseApiUrl = CONSTANT.URL.CMS.BASE + ClientData.userInfo_accountPath() + CONSTANT.URL.CMS.API.BASE + 'dashboardSetting/'; 10 DashboardSetting.saveSettingApiUrl = DashboardSetting.baseApiUrl + 'saveSetting'; 11 DashboardSetting.getSettingApiUrl = DashboardSetting.baseApiUrl + 'getAllSettingData'; -
Developer
一か所しか使ってないので共通化しなくても大丈夫です。 DashboardSetting.getSettingDataの中に書いた方が良いと思います
-
-
-
3 newReport: { id: 'settingNewReport', name: 'chk-new', enabled: true }, 4 continousWork: { id: 'settingContinousWork', name: 'chk-proccess', enabled: true }, 5 reportWarning: { id: 'settingReportWarning', name: 'chk-alert', enabled: true }, 6 dashboardHome: { id: 'settingDashboardHome', name: 'chk-home', enabled: false }, 7 }; 8 9 DashboardSetting.baseApiUrl = CONSTANT.URL.CMS.BASE + ClientData.userInfo_accountPath() + CONSTANT.URL.CMS.API.BASE + 'dashboardSetting/'; 10 DashboardSetting.saveSettingApiUrl = DashboardSetting.baseApiUrl + 'saveSetting'; 11 DashboardSetting.getSettingApiUrl = DashboardSetting.baseApiUrl + 'getAllSettingData'; 12 13 DashboardSetting.defaultSettingJson = { 14 newReport: 1, 15 continousWork: 1, 16 reportWarning: 1, 17 dashboardHome: 0, 18 } -
Developer
DashboardSetting.elementItemsの中にデータを格納しないのはなぜですか？
-
-
69 $('.toggle').on('click', function () { 70 $(this).toggleClass('checked'); 71 const key = $(this).attr('data-key'); 72 var enabled = false; 73 if ($(this).hasClass('checked')) { 74 enabled = true; 75 } 76 DashboardSetting.saveSetting(key, enabled); 77 }); 78 }; 79 80 //設定画面の初期処理 81 DashboardSetting.init = function () { 82 DashboardSetting.getSettingData(function (settings) { 83 DashboardSetting.applySettings(settings); 84 }); -
Developer
DashboardSetting.getSettingDataの中にDashboardSetting.applySettingsを書かないのはなぜですか？
-
-
73 if ($(this).hasClass('checked')) { 74 enabled = true; 75 } 76 DashboardSetting.saveSetting(key, enabled); 77 }); 78 }; 79 80 //設定画面の初期処理 81 DashboardSetting.init = function () { 82 DashboardSetting.getSettingData(function (settings) { 83 DashboardSetting.applySettings(settings); 84 }); 85 DashboardSetting.bindToggleClick(); 86 }; 87 88 DashboardSetting.onClickSave = function () { -
Developer
英語でコメントを書いてください
-
-
66 67 //設定画面でクリックされてた時の処理 68 DashboardSetting.bindToggleClick = function () { 69 $('.toggle').on('click', function () { 70 $(this).toggleClass('checked'); 71 const key = $(this).attr('data-key'); 72 var enabled = false; 73 if ($(this).hasClass('checked')) { 74 enabled = true; 75 } 76 DashboardSetting.saveSetting(key, enabled); 77 }); 78 }; 79 80 //設定画面の初期処理 81 DashboardSetting.init = function () { -
Developer
英語でコメントを書いてください 元のコメントは仮で日本語で書いたものですので、消してください
-
-
53 54 DashboardSetting.saveSetting = function (key, enabled) { 55 if (DashboardSetting.elementItems.hasOwnProperty(key)) { 56 const elementItem = DashboardSetting.elementItems[key]; 57 elementItem.enabled = enabled; 58 } 59 let param = {}; 60 param.sid = COMMON.getSid(); 61 param[key] = enabled; 62 COMMON.cmsAjax(DashboardSetting.saveSettingApiUrl, param, false, function (json) { 63 64 }); 65 }; 66 67 //設定画面でクリックされてた時の処理 68 DashboardSetting.bindToggleClick = function () { -
Developer
英語でコメントを書いてください 元のコメントは仮で日本語で書いたものですので、消してください
-
-
57 elementItem.enabled = enabled; 58 } 59 let param = {}; 60 param.sid = COMMON.getSid(); 61 param[key] = enabled; 62 COMMON.cmsAjax(DashboardSetting.saveSettingApiUrl, param, false, function (json) { 63 64 }); 65 }; 66 67 //設定画面でクリックされてた時の処理 68 DashboardSetting.bindToggleClick = function () { 69 $('.toggle').on('click', function () { 70 $(this).toggleClass('checked'); 71 const key = $(this).attr('data-key'); 72 var enabled = false; -
Developer
-
-
-
39 if (DashboardSetting.elementItems.hasOwnProperty(key)) { 40 const elementItem = DashboardSetting.elementItems[key]; 41 elementItem.enabled = enabled; 42 let element = $('#' + elementItem.id); 43 if (element) { 44 if (enabled) { 45 element.addClass('checked'); 46 } else { 47 element.removeClass('checked'); 48 } 49 } 50 } 51 } 52 }; 53 54 DashboardSetting.saveSetting = function (key, enabled) { -
Developer
英語でコメントを書いてください
-
-
26 callback(json); 27 } 28 }, function() { 29 console.log('DashboardSetting.getSettingData error'); 30 if (callback) { 31 callback(DashboardSetting.defaultSettingJson); 32 } 33 }); 34 }; 35 36 DashboardSetting.applySettings = function (settings) { 37 for (const key in settings) { 38 const enabled = settings[key]; 39 if (DashboardSetting.elementItems.hasOwnProperty(key)) { 40 const elementItem = DashboardSetting.elementItems[key]; 41 elementItem.enabled = enabled; -
Developer
constに再代入するとエラーになると思いますが、この処理でエラーにはならなかったですか？
-
-
21 let param = { 22 sid: COMMON.getSid(), 23 }; 24 COMMON.cmsAjax(DashboardSetting.getSettingApiUrl, param, false, function (json) { 25 if (callback) { 26 callback(json); 27 } 28 }, function() { 29 console.log('DashboardSetting.getSettingData error'); 30 if (callback) { 31 callback(DashboardSetting.defaultSettingJson); 32 } 33 }); 34 }; 35 36 DashboardSetting.applySettings = function (settings) { -
Developer
英語でコメントを書いてください
-
-
7 }; 8 9 DashboardSetting.baseApiUrl = CONSTANT.URL.CMS.BASE + ClientData.userInfo_accountPath() + CONSTANT.URL.CMS.API.BASE + 'dashboardSetting/'; 10 DashboardSetting.saveSettingApiUrl = DashboardSetting.baseApiUrl + 'saveSetting'; 11 DashboardSetting.getSettingApiUrl = DashboardSetting.baseApiUrl + 'getAllSettingData'; 12 13 DashboardSetting.defaultSettingJson = { 14 newReport: 1, 15 continousWork: 1, 16 reportWarning: 1, 17 dashboardHome: 0, 18 } 19 20 DashboardSetting.getSettingData = function (callback) { 21 let param = { 22 sid: COMMON.getSid(), -
Developer
paramを再代入しないロジックになっているのでletではなくconstで良いと思います
-
-
1 var DashboardSetting = {}; 2 DashboardSetting.elementItems = { 3 newReport: { id: 'settingNewReport', name: 'chk-new', enabled: true }, 4 continousWork: { id: 'settingContinousWork', name: 'chk-proccess', enabled: true }, 5 reportWarning: { id: 'settingReportWarning', name: 'chk-alert', enabled: true }, 6 dashboardHome: { id: 'settingDashboardHome', name: 'chk-home', enabled: false }, 7 }; 8 9 DashboardSetting.baseApiUrl = CONSTANT.URL.CMS.BASE + ClientData.userInfo_accountPath() + CONSTANT.URL.CMS.API.BASE + 'dashboardSetting/'; 10 DashboardSetting.saveSettingApiUrl = DashboardSetting.baseApiUrl + 'saveSetting'; 11 DashboardSetting.getSettingApiUrl = DashboardSetting.baseApiUrl + 'getAllSettingData'; 12 13 DashboardSetting.defaultSettingJson = { -
Developer
英語でコメントを書いてください
-
-
added 1 commit
- 4126f0f5 - # jsで処理、論理を作成 ( Process with js, create logic Top Screen )
Toggle commit list -
abvw/js/dashboard/dashboard.js 0 → 100644
1 var DASHBOARD = {}; -
Developer
一番上にファイルの肥大化とファイル理解のためにファイルの説明をコメントで書いてください。 /**
- ここに説明 *
- @since cms:1.4.3.2&1.4.3.3 web:1.0 */
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
97 }); 98 } 99 } else if (item.id == 'chat') { 100 item.enabled = CHK.isChat; 101 } 102 }); 103 } 104 105 //ガジェット画面に移動 106 DASHBOARD.goPickup = function(pickupId) { 107 CHK.goUrlWithCurrentParams('pickup.html', {pickupActive: pickupId}); 108 } 109 110 //コミュニケーション画面に移動 111 DASHBOARD.goMessageList = function() { 112 CHK.goUrlWithCurrentParams('message-list.html', {}); -
Developer
goUrlWithCurrentParamsはcommonにあるものを使ってください
-
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
92 //読んでいないメッセージがあればカウントを +1 93 CHK.pushMessageList.forEach(function(message) { 94 if (!message.readingFlg) { 95 item.count += 1; 96 } 97 }); 98 } 99 } else if (item.id == 'chat') { 100 item.enabled = CHK.isChat; 101 } 102 }); 103 } 104 105 //ガジェット画面に移動 106 DASHBOARD.goPickup = function(pickupId) { 107 CHK.goUrlWithCurrentParams('pickup.html', {pickupActive: pickupId}); -
Developer
goUrlWithCurrentParamsはcommonにあるものを使ってください
-
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
96 } 97 }); 98 } 99 } else if (item.id == 'chat') { 100 item.enabled = CHK.isChat; 101 } 102 }); 103 } 104 105 //ガジェット画面に移動 106 DASHBOARD.goPickup = function(pickupId) { 107 CHK.goUrlWithCurrentParams('pickup.html', {pickupActive: pickupId}); 108 } 109 110 //コミュニケーション画面に移動 111 DASHBOARD.goMessageList = function() { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
91 if (typeof CHK.pushMessageList !== 'undefined') { 92 //読んでいないメッセージがあればカウントを +1 93 CHK.pushMessageList.forEach(function(message) { 94 if (!message.readingFlg) { 95 item.count += 1; 96 } 97 }); 98 } 99 } else if (item.id == 'chat') { 100 item.enabled = CHK.isChat; 101 } 102 }); 103 } 104 105 //ガジェット画面に移動 106 DASHBOARD.goPickup = function(pickupId) { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
72 //変数として設定しているガジェットに初期表示で取得したデータを設定 73 DASHBOARD.updateDataPickups = function() { 74 DASHBOARD.pickupItems.forEach(function(item) { 75 let enabled = CHK.dashboardSetting[item.id]; 76 if (enabled == true || enabled == false) { 77 item.enabled = enabled; 78 } 79 item.count = 0; 80 if (CHK.dashboardBatch[item.id]) { 81 item.count = CHK.dashboardBatch[item.id]; 82 } 83 }); 84 } 85 86 //変数として設定しているコミュニケーションに初期表示で取得したデータを設定 87 DASHBOARD.updateDataCommunications = function() { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
76 if (enabled == true || enabled == false) { 77 item.enabled = enabled; 78 } 79 item.count = 0; 80 if (CHK.dashboardBatch[item.id]) { 81 item.count = CHK.dashboardBatch[item.id]; 82 } 83 }); 84 } 85 86 //変数として設定しているコミュニケーションに初期表示で取得したデータを設定 87 DASHBOARD.updateDataCommunications = function() { 88 DASHBOARD.communicationItems.forEach(function(item) { 89 if (item.id == 'messageList') { 90 item.count = 0; 91 if (typeof CHK.pushMessageList !== 'undefined') { -
Developer
CHK.pushMessageListはどこにも定義されていないようですが、未実装なのでしょうか？
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
58 + countSpan 59 + '</a>' 60 + '</div>'); 61 return html; 62 } 63 64 //ダッシュボード画面の初期表示 65 DASHBOARD.init = function() { 66 DASHBOARD.updateDataPickups(); 67 DASHBOARD.initPickups(); 68 DASHBOARD.updateDataCommunications(); 69 DASHBOARD.initCommunications(); 70 } 71 72 //変数として設定しているガジェットに初期表示で取得したデータを設定 73 DASHBOARD.updateDataPickups = function() { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
60 + '</div>'); 61 return html; 62 } 63 64 //ダッシュボード画面の初期表示 65 DASHBOARD.init = function() { 66 DASHBOARD.updateDataPickups(); 67 DASHBOARD.initPickups(); 68 DASHBOARD.updateDataCommunications(); 69 DASHBOARD.initCommunications(); 70 } 71 72 //変数として設定しているガジェットに初期表示で取得したデータを設定 73 DASHBOARD.updateDataPickups = function() { 74 DASHBOARD.pickupItems.forEach(function(item) { 75 let enabled = CHK.dashboardSetting[item.id]; -
Developer
letではなく定数のconstで良いと思います
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
50 if (typeof item.count == 'undefined' || item.count < 1) { 51 countDClass = ' d-none'; 52 } 53 var countSpan = '<span class="count fs-7' + countDClass + '">' + countText + '</span>'; 54 var html = $('<div class="item" id="' + item.id + '">' 55 + '<a href="' + item.href + '" class="d-block text-dark text-decoration-none mb-1 p-3">' 56 + '<img src="' + item.img.src + '">' 57 + '<div class="fs-8 multi-lang" data-msg="' + item.msg + '">' + getMsg(item.msg) + '</div>' 58 + countSpan 59 + '</a>' 60 + '</div>'); 61 return html; 62 } 63 64 //ダッシュボード画面の初期表示 65 DASHBOARD.init = function() { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
29 } 30 } 31 32 //CHK_Dashboard.communicationItemsに設定した情報をhtmlに表示する 33 DASHBOARD.initCommunications = function() { 34 $("#communicationItems").empty(); 35 DASHBOARD.communicationItems.forEach(function(item) { 36 if (item.enabled == true) { 37 var html = DASHBOARD.initHtmlItem(item); 38 $("#communicationItems").append(html); 39 } 40 }); 41 } 42 43 //設定されている情報からhtmlを表示する 44 DASHBOARD.initHtmlItem = function(item) { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
47 countText = '99+'; 48 } 49 var countDClass = ''; 50 if (typeof item.count == 'undefined' || item.count < 1) { 51 countDClass = ' d-none'; 52 } 53 var countSpan = '<span class="count fs-7' + countDClass + '">' + countText + '</span>'; 54 var html = $('<div class="item" id="' + item.id + '">' 55 + '<a href="' + item.href + '" class="d-block text-dark text-decoration-none mb-1 p-3">' 56 + '<img src="' + item.img.src + '">' 57 + '<div class="fs-8 multi-lang" data-msg="' + item.msg + '">' + getMsg(item.msg) + '</div>' 58 + countSpan 59 + '</a>' 60 + '</div>'); 61 return html; 62 } -
Developer
countText、countDClass、countSpan、htmlはvarではなくletで良いと思います
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
18 DASHBOARD.pickupItems.forEach(function(item) { 19 if (item.enabled == true) { 20 var html = DASHBOARD.initHtmlItem(item); 21 $("#pickupItems").append(html); 22 } 23 }); 24 //pickup empty 25 if ($("#pickupItems").children().length > 0) { 26 $("#pickupHeader").removeClass('d-none'); 27 } else { 28 $("#pickupHeader").addClass('d-none'); 29 } 30 } 31 32 //CHK_Dashboard.communicationItemsに設定した情報をhtmlに表示する 33 DASHBOARD.initCommunications = function() { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
22 } 23 }); 24 //pickup empty 25 if ($("#pickupItems").children().length > 0) { 26 $("#pickupHeader").removeClass('d-none'); 27 } else { 28 $("#pickupHeader").addClass('d-none'); 29 } 30 } 31 32 //CHK_Dashboard.communicationItemsに設定した情報をhtmlに表示する 33 DASHBOARD.initCommunications = function() { 34 $("#communicationItems").empty(); 35 DASHBOARD.communicationItems.forEach(function(item) { 36 if (item.enabled == true) { 37 var html = DASHBOARD.initHtmlItem(item); -
Developer
varではなくconstで良いと思います
-
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
1 var DASHBOARD = {}; 2 3 DASHBOARD.pickupItems = [ 4 {id: 'newReport', href: "javascript:DASHBOARD.goPickup('newReport');", count: 0, enabled: true, img: {src: 'img/icon_new.svg'}, msg: 'newRegistrationTitle'}, 5 {id: 'continousWork', href: "javascript:DASHBOARD.goPickup('continousWork');", count: 0, enabled: true, img: {src: 'img/icon_proccess.svg'}, msg: 'continousWorkTitle'}, 6 {id: 'reportWarning', href: "javascript:DASHBOARD.goPickup('reportWarning');", count: 0, enabled: true, img: {src: 'img/icon_alert.svg'}, msg: 'reportWarningTitle'}, 7 ]; 8 DASHBOARD.communicationItems = [ 9 {id: 'messageList', href: "javascript:DASHBOARD.goMessageList();", count: 0, enabled: true, img: {src: 'img/icon_message_lsit.svg'}, msg: 'messageListTitle'}, 10 {id: 'sendMessage', href: "javascript:DASHBOARD.goSendMessage();", count: 0, enabled: true, img: {src: 'img/icon_send_message.svg'}, msg: 'sendMessageTitle'}, 11 {id: 'distanceSupport', href: "javascript:DASHBOARD.goDistanceSupport();", count: 0, enabled: true, img: {src: 'img/icon_remote.svg'}, msg: 'distanceSupportTitle'}, 12 {id: 'chat', href: "javascript:DASHBOARD.goChat();", count: 0, enabled: false, img: {src: 'img/icon_chat.svg'}, msg: 'chatTitle'}, 13 ]; 14 15 //CHK_Dashboard.pickupItemsに設定した情報をhtmlに表示する 16 DASHBOARD.initPickups = function() { -
Developer
英語でコメントを書いてください 元のコメントは仮のコメントです。消してください
-
-
abvw/js/dashboard/dashboard.js 0 → 100644
5 {id: 'continousWork', href: "javascript:DASHBOARD.goPickup('continousWork');", count: 0, enabled: true, img: {src: 'img/icon_proccess.svg'}, msg: 'continousWorkTitle'}, 6 {id: 'reportWarning', href: "javascript:DASHBOARD.goPickup('reportWarning');", count: 0, enabled: true, img: {src: 'img/icon_alert.svg'}, msg: 'reportWarningTitle'}, 7 ]; 8 DASHBOARD.communicationItems = [ 9 {id: 'messageList', href: "javascript:DASHBOARD.goMessageList();", count: 0, enabled: true, img: {src: 'img/icon_message_lsit.svg'}, msg: 'messageListTitle'}, 10 {id: 'sendMessage', href: "javascript:DASHBOARD.goSendMessage();", count: 0, enabled: true, img: {src: 'img/icon_send_message.svg'}, msg: 'sendMessageTitle'}, 11 {id: 'distanceSupport', href: "javascript:DASHBOARD.goDistanceSupport();", count: 0, enabled: true, img: {src: 'img/icon_remote.svg'}, msg: 'distanceSupportTitle'}, 12 {id: 'chat', href: "javascript:DASHBOARD.goChat();", count: 0, enabled: false, img: {src: 'img/icon_chat.svg'}, msg: 'chatTitle'}, 13 ]; 14 15 //CHK_Dashboard.pickupItemsに設定した情報をhtmlに表示する 16 DASHBOARD.initPickups = function() { 17 $("#pickupItems").empty(); 18 DASHBOARD.pickupItems.forEach(function(item) { 19 if (item.enabled == true) { 20 var html = DASHBOARD.initHtmlItem(item); -
Developer
varではなくconstで良いと思います
-
-
abvw/html/dashboard.html 0 → 100755
1 <!DOCTYPE html> -
Developer
multi-langでの多言語化ではなくi18n.jsでの多言語で表示してください 元はアプリと共通化するためにアプリで行っている多言語化を採用しようとしたのですが、アプリと共通化しないことになったのでロジックを変えました
-
-
abvw/html/dashboard-setting.html 0 → 100755
1 <!DOCTYPE html> -
Developer
multi-langでの多言語化ではなくi18n.jsでの多言語で表示してください 元はアプリと共通化するためにアプリで行っている多言語化を採用しようとしたのですが、アプリと共通化しないことになったのでロジックを変えました -
-
-
abvw/html/dashboard-setting.html 0 → 100755
30 戻る 31 </span> 32 </a> 33 <h1 class="fs-10 font-weight-bold mb-0 text-white multi-lang" data-msg="dashboardSettingHeaderTitle">ダッシュボード設定</h1> 34 <div></div> 35 </nav> 36 </header> 37 38 <!-- setting --> 39 <section class="container-fluid main-section"> 40 <main> 41 <h2 class="fs-8 font-weight-bold mt-4 pb-2 border-bottom text-dark multi-lang" data-msg="dashboardSettingDisplayHeader">表示切替</h2> 42 <div class="setting-table-wrap"> 43 <table id="displayItems"> 44 <tbody> 45 <!-- <tr> -
Developer
いらないものは消してください
-
-
-
abvw/common/js/newdash/common.js 0 → 100755
1 $(function(){ -
Developer
全体的にコメントを英語に置き換えてください。 jsファイルは外部から解析しにくいように難読化を行っています。 難読化を行うときに英語以外の文字が使用されているとうまく難読化できないのでコメントは英語にするようにしてください
-
-
abvw/common/header.html 0 → 100755
1 <header> 2 <nav class="navbar navbar-dark shadow-sm px-sm-auto px-0"> 3 <div class="container"> -
Developer
-
-
abvw/common/confirm-modal.html 0 → 100755
-
abvw/html/dashboard.html 0 → 100755
105 </a> 106 </li> 107 <li class="card mb-2"> 108 <a href="#" class="d-block text-dark text-decoration-none mb-1 p-3 text-center"> 109 <div class="bg-lightblue p-2 d-inline-block rounded-circle mb-3"> 110 <img src="../common/img/dahboard_icon_chat.svg" alt="チャット" class="img-fluid"> 111 </div> 112 <div class="fs-10">チャット</div> 113 </a> 114 </li> 115 </ul> 116 </div> 117 </main> 118 119 <!-- dashboard setting modal --> 120 <div class="modal fade" id="dashboard-setting-modal" tabindex="-1" role="dialog"> -
Developer
ダッシュボード設定のhtmlがありましたが、どちらを使用する予定ですか？
-
-
-
abvw/html/dashboard.html 0 → 100755
135 <tr> 136 <th class="fs-9 p-2 text-secondary multi-lang" data-msg="dashboardSettingNewReportTitle">新規報告</th> 137 <td class="p-2"><div id="settingNewReport" data-key="newReport" class="toggle ml-auto"><input type="checkbox" name="chk-new"></div></td> 138 </tr> 139 <tr> 140 <th class="fs-9 p-2 text-secondary multi-lang" data-msg="dashboardSettingContinousWorkTitle">工程作業</th> 141 <td class="p-2"><div id="settingContinousWork" data-key="continousWork" class="toggle ml-auto"><input type="checkbox" name="chk-proccess"></div></td> 142 </tr> 143 <tr> 144 <th class="fs-9 p-2 text-secondary multi-lang" data-msg="dashboardSettingReportWarningTitle">警告を含む作業</th> 145 <td class="p-2"><div id="settingReportWarning" data-key="reportWarning" class="toggle ml-auto"><input type="checkbox" name="chk-alert"></div></td> 146 </tr> 147 <tr> 148 <th class="fs-9 p-2 text-secondary multi-lang" data-msg="dashboardSettingDashboardHomeTitle">ダッシュボードをホーム画面にする</th> 149 <td class="p-2"><div id="settingDashboardHome" data-key="dashboardHome" class="toggle ml-auto"><input type="checkbox" name="chk-home"></div></td> 150 </tr> -
Developer
ダッシュボード画面はDASHBOARD.pickupItemsからhtmlを生成する処理をしています。 設定画面もDashboardSetting.elementItemsで値を持つ処理にしています。 上記のロジックに従うならこのhtml表示もDashboardSetting.elementItemsを使用してhtmlを動的に表示すると思うのですが、この処理にした理由は何ですか？
-
-
-
Toggle commit list
-
-
Toggle commit list
-
Toggle commit list
-
-
-
-
closed
Toggle commit list
