Skip to content

Commit e0b719d

Browse files
committed
feat: add strings for elements highlight feature
1 parent 9acfcb7 commit e0b719d

File tree

5 files changed

+26
-10
lines changed

5 files changed

+26
-10
lines changed

src/LiveDevelopment/BrowserScripts/RemoteFunctions.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -562,23 +562,23 @@ function RemoteFunctions(config) {
562562

563563
// Only include select parent option if element supports it
564564
if (showSelectParentOption) {
565-
content += `<span data-action="select-parent" title="Select Parent">
565+
content += `<span data-action="select-parent" title="${config.strings.selectParent}">
566566
${ICONS.arrowUp}
567567
</span>`;
568568
}
569569

570570
// Only include edit text option if element supports it
571571
if (showEditTextOption) {
572-
content += `<span data-action="edit-text" title="Edit Text">
572+
content += `<span data-action="edit-text" title="${config.strings.editText}">
573573
${ICONS.edit}
574574
</span>`;
575575
}
576576

577577
// Always include duplicate and delete options
578-
content += `<span data-action="duplicate" title="Duplicate">
578+
content += `<span data-action="duplicate" title="${config.strings.duplicate}">
579579
${ICONS.copy}
580580
</span>
581-
<span data-action="delete" title="Delete">
581+
<span data-action="delete" title="${config.strings.delete}">
582582
${ICONS.trash}
583583
</span>
584584
</div>`;

src/LiveDevelopment/main.js

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,16 @@ define(function main(require, exports, module) {
6666
showInfo: true
6767
},
6868
isLPEditFeaturesActive: isLPEditFeaturesActive,
69-
elemHighlights: "hover" // default value, this will get updated when the extension loads
69+
elemHighlights: "hover", // default value, this will get updated when the extension loads
70+
// this strings are used in RemoteFunctions.js
71+
// we need to pass this through config as remoteFunctions runs in browser context and cannot
72+
// directly reference Strings file
73+
strings: {
74+
selectParent: Strings.LIVE_DEV_MORE_OPTIONS_SELECT_PARENT,
75+
editText: Strings.LIVE_DEV_MORE_OPTIONS_EDIT_TEXT,
76+
duplicate: Strings.LIVE_DEV_MORE_OPTIONS_DUPLICATE,
77+
delete: Strings.LIVE_DEV_MORE_OPTIONS_DELETE
78+
}
7079
};
7180
// Status labels/styles are ordered: error, not connected, progress1, progress2, connected.
7281
var _status,

src/extensionsIntegrated/Phoenix-live-preview/LivePreviewSettings.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ define(function (require, exports, module) {
8989
values: Object.keys(SUPPORTED_FRAMEWORKS)
9090
});
9191
PreferencesManager.definePreference(LiveDevelopmentMain.PREFERENCE_PROJECT_ELEMENT_HIGHLIGHT, "string", "hover", {
92-
description: "show live preview element highlights on 'hover' or 'click'. Defaults to 'hover'"
92+
description: Strings.LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT_PREFERENCE
9393
});
9494

9595
async function detectFramework($frameworkSelect, $hotReloadChk) {

src/extensionsIntegrated/Phoenix-live-preview/livePreviewSettings.html

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,10 @@ <h1 class="dialog-title">{{Strings.LIVE_DEV_SETTINGS_TITLE}}</h1>
2727
</div>
2828

2929
<div class="form-group" id="elementHighlightWrapper">
30-
<hr>
31-
<label for="elementHighlight">Show Live Preview Element Highlights on: </label>
30+
<label for="elementHighlight">{{Strings.LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT}} </label>
3231
<select name="elementHighlight" id="elementHighlight">
33-
<option value="hover">hover</option>
34-
<option value="click">click</option>
32+
<option value="hover">{{Strings.LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT_HOVER}}</option>
33+
<option value="click">{{Strings.LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT_CLICK}}</option>
3534
</select>
3635
</div>
3736
</div>

src/nls/root/strings.js

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -179,6 +179,14 @@ define({
179179
"LIVE_DEV_SETTINGS_FRAMEWORK": "Server Framework",
180180
"LIVE_DEV_SETTINGS_FRAMEWORK_CUSTOM": "Custom",
181181
"LIVE_DEV_SETTINGS_FRAMEWORK_PREFERENCES": "Server Framework, currently supports only docusaurus",
182+
"LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT": "Show Live Preview Element Highlights on:",
183+
"LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT_HOVER": "hover",
184+
"LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT_CLICK": "click",
185+
"LIVE_DEV_SETTINGS_ELEMENT_HIGHLIGHT_PREFERENCE": "show live preview element highlights on 'hover' or 'click'. Defaults to 'hover'",
186+
"LIVE_DEV_MORE_OPTIONS_SELECT_PARENT": "Select Parent",
187+
"LIVE_DEV_MORE_OPTIONS_EDIT_TEXT": "Edit Text",
188+
"LIVE_DEV_MORE_OPTIONS_DUPLICATE": "Duplicate",
189+
"LIVE_DEV_MORE_OPTIONS_DELETE": "Delete",
182190
"LIVE_PREVIEW_CUSTOM_SERVER_BANNER": "Getting preview from your custom server {0}",
183191

184192
"LIVE_DEV_DETACHED_REPLACED_WITH_DEVTOOLS": "Live Preview was canceled because the browser's developer tools were opened",

0 commit comments

Comments
 (0)