From b7c9027c242ba23dcf2c69214ce52a41f7dbcfa3 Mon Sep 17 00:00:00 2001 From: Mathias Elle Date: Wed, 6 May 2026 23:42:09 +0200 Subject: [PATCH 1/8] refactor: centralize inspector configuration constants in Inspector class --- src/Block/Inspector.php | 12 +++++------- src/Console/Command/Dev/InspectorCommand.php | 20 ++++++-------------- src/Model/Config/Inspector.php | 13 +++++++++++++ 3 files changed, 24 insertions(+), 21 deletions(-) create mode 100644 src/Model/Config/Inspector.php diff --git a/src/Block/Inspector.php b/src/Block/Inspector.php index 6c3b3c05..cd906781 100644 --- a/src/Block/Inspector.php +++ b/src/Block/Inspector.php @@ -9,6 +9,7 @@ use Magento\Framework\App\State; use Magento\Framework\View\Element\Template; use Magento\Framework\View\Element\Template\Context; +use OpenForgeProject\MageForge\Model\Config\Inspector as InspectorConfig; /** * Block for MageForge Inspector @@ -17,9 +18,6 @@ */ class Inspector extends Template { - private const XML_PATH_INSPECTOR_ENABLED = 'dev/mageforge_inspector/enabled'; - private const XML_PATH_SHOW_BUTTON_LABELS = 'mageforge/inspector/show_button_labels'; - /** * @param Context $context * @param State $state @@ -51,7 +49,7 @@ public function shouldRender(): bool } // Check if inspector is enabled in configuration - if (!$this->scopeConfig->isSetFlag(self::XML_PATH_INSPECTOR_ENABLED)) { + if (!$this->scopeConfig->isSetFlag(InspectorConfig::XML_PATH_ENABLED)) { return false; } @@ -110,7 +108,7 @@ public function getToolbarJsUrl(): string */ public function getShowButtonLabels(): bool { - $value = $this->scopeConfig->getValue(self::XML_PATH_SHOW_BUTTON_LABELS); + $value = $this->scopeConfig->getValue(InspectorConfig::XML_PATH_SHOW_BUTTON_LABELS); // Default to true when not explicitly set to '0' return !is_string($value) || $value !== '0'; } @@ -122,8 +120,8 @@ public function getShowButtonLabels(): bool */ public function getTheme(): string { - $value = $this->scopeConfig->getValue('mageforge/inspector/theme'); - return is_string($value) && $value !== '' ? $value : 'dark'; + $value = $this->scopeConfig->getValue(InspectorConfig::XML_PATH_THEME); + return is_string($value) && $value !== '' ? $value : InspectorConfig::DEFAULT_THEME; } /** diff --git a/src/Console/Command/Dev/InspectorCommand.php b/src/Console/Command/Dev/InspectorCommand.php index 5b1e4f30..d531fae3 100644 --- a/src/Console/Command/Dev/InspectorCommand.php +++ b/src/Console/Command/Dev/InspectorCommand.php @@ -10,6 +10,7 @@ use Magento\Framework\App\State; use Magento\Framework\Console\Cli; use OpenForgeProject\MageForge\Console\Command\AbstractCommand; +use OpenForgeProject\MageForge\Model\Config\Inspector as InspectorConfig; use Symfony\Component\Console\Input\InputArgument; use Symfony\Component\Console\Input\InputInterface; use Symfony\Component\Console\Output\OutputInterface; @@ -19,7 +20,6 @@ */ class InspectorCommand extends AbstractCommand { - private const XML_PATH_INSPECTOR_ENABLED = 'dev/mageforge_inspector/enabled'; private const ARGUMENT_ACTION = 'action'; /** @@ -106,7 +106,7 @@ protected function executeCommand(InputInterface $input, OutputInterface $output return match ($action) { 'enable' => $this->enableInspector(), 'disable' => $this->disableInspector(), - 'status' => $this->showStatus(), + default => $this->showStatus(), }; } @@ -117,7 +117,7 @@ protected function executeCommand(InputInterface $input, OutputInterface $output */ private function enableInspector(): int { - $this->configWriter->save(self::XML_PATH_INSPECTOR_ENABLED, '1'); + $this->configWriter->save(InspectorConfig::XML_PATH_ENABLED, '1'); $this->cleanCache(); $this->io->success('MageForge Inspector has been enabled!'); @@ -142,7 +142,7 @@ private function enableInspector(): int */ private function disableInspector(): int { - $this->configWriter->save(self::XML_PATH_INSPECTOR_ENABLED, '0'); + $this->configWriter->save(InspectorConfig::XML_PATH_ENABLED, '0'); $this->cleanCache(); $this->io->success('MageForge Inspector has been disabled.'); @@ -193,11 +193,7 @@ private function showStatus(): int */ private function isDeveloperMode(): bool { - try { - return $this->state->getMode() === State::MODE_DEVELOPER; - } catch (\Exception $e) { - return false; - } + return $this->state->getMode() === State::MODE_DEVELOPER; } /** @@ -207,11 +203,7 @@ private function isDeveloperMode(): bool */ private function isInspectorEnabled(): bool { - try { - return $this->scopeConfig->isSetFlag(self::XML_PATH_INSPECTOR_ENABLED); - } catch (\Exception $e) { - return false; - } + return $this->scopeConfig->isSetFlag(InspectorConfig::XML_PATH_ENABLED); } /** diff --git a/src/Model/Config/Inspector.php b/src/Model/Config/Inspector.php new file mode 100644 index 00000000..53f6282b --- /dev/null +++ b/src/Model/Config/Inspector.php @@ -0,0 +1,13 @@ + Date: Thu, 7 May 2026 00:02:55 +0200 Subject: [PATCH 2/8] feat: add toolbar position configuration and implement in UI --- src/Block/Inspector.php | 21 ++++++- src/Model/Config/Inspector.php | 2 + src/Model/Config/Source/ToolbarPosition.php | 25 +++++++++ .../TemplateEngine/Plugin/InspectorHints.php | 10 +++- src/etc/adminhtml/system.xml | 6 ++ src/etc/config.xml | 1 + src/i18n/de_DE.csv | 18 ++++++ src/i18n/en_US.csv | 18 ++++++ src/view/frontend/templates/inspector.phtml | 1 + src/view/frontend/web/css/toolbar.css | 55 +++++++++++++++++++ src/view/frontend/web/js/toolbar/ui.js | 4 ++ 11 files changed, 155 insertions(+), 6 deletions(-) create mode 100644 src/Model/Config/Source/ToolbarPosition.php create mode 100644 src/i18n/de_DE.csv create mode 100644 src/i18n/en_US.csv diff --git a/src/Block/Inspector.php b/src/Block/Inspector.php index cd906781..d833aa5a 100644 --- a/src/Block/Inspector.php +++ b/src/Block/Inspector.php @@ -9,6 +9,7 @@ use Magento\Framework\App\State; use Magento\Framework\View\Element\Template; use Magento\Framework\View\Element\Template\Context; +use Magento\Store\Model\ScopeInterface; use OpenForgeProject\MageForge\Model\Config\Inspector as InspectorConfig; /** @@ -49,7 +50,7 @@ public function shouldRender(): bool } // Check if inspector is enabled in configuration - if (!$this->scopeConfig->isSetFlag(InspectorConfig::XML_PATH_ENABLED)) { + if (!$this->scopeConfig->isSetFlag(InspectorConfig::XML_PATH_ENABLED, ScopeInterface::SCOPE_STORE)) { return false; } @@ -108,7 +109,10 @@ public function getToolbarJsUrl(): string */ public function getShowButtonLabels(): bool { - $value = $this->scopeConfig->getValue(InspectorConfig::XML_PATH_SHOW_BUTTON_LABELS); + $value = $this->scopeConfig->getValue( + InspectorConfig::XML_PATH_SHOW_BUTTON_LABELS, + ScopeInterface::SCOPE_STORE + ); // Default to true when not explicitly set to '0' return !is_string($value) || $value !== '0'; } @@ -120,10 +124,21 @@ public function getShowButtonLabels(): bool */ public function getTheme(): string { - $value = $this->scopeConfig->getValue(InspectorConfig::XML_PATH_THEME); + $value = $this->scopeConfig->getValue(InspectorConfig::XML_PATH_THEME, ScopeInterface::SCOPE_STORE); return is_string($value) && $value !== '' ? $value : InspectorConfig::DEFAULT_THEME; } + /** + * Get configured toolbar position + * + * @return string + */ + public function getPosition(): string + { + $value = $this->scopeConfig->getValue(InspectorConfig::XML_PATH_POSITION, ScopeInterface::SCOPE_STORE); + return is_string($value) && $value !== '' ? $value : InspectorConfig::DEFAULT_POSITION; + } + /** * Render block HTML * diff --git a/src/Model/Config/Inspector.php b/src/Model/Config/Inspector.php index 53f6282b..d1aab7d1 100644 --- a/src/Model/Config/Inspector.php +++ b/src/Model/Config/Inspector.php @@ -9,5 +9,7 @@ class Inspector public const XML_PATH_ENABLED = 'dev/mageforge_inspector/enabled'; public const XML_PATH_SHOW_BUTTON_LABELS = 'mageforge/inspector/show_button_labels'; public const XML_PATH_THEME = 'mageforge/inspector/theme'; + public const XML_PATH_POSITION = 'mageforge/inspector/position'; public const DEFAULT_THEME = 'dark'; + public const DEFAULT_POSITION = 'bottom-left'; } diff --git a/src/Model/Config/Source/ToolbarPosition.php b/src/Model/Config/Source/ToolbarPosition.php new file mode 100644 index 00000000..4da044de --- /dev/null +++ b/src/Model/Config/Source/ToolbarPosition.php @@ -0,0 +1,25 @@ +> + */ + public function toOptionArray(): array + { + return [ + ['value' => 'bottom-left', 'label' => (string) __('Bottom Left')], + ['value' => 'bottom-right', 'label' => (string) __('Bottom Right')], + ['value' => 'top-left', 'label' => (string) __('Top Left')], + ['value' => 'top-right', 'label' => (string) __('Top Right')], + ]; + } +} diff --git a/src/Model/TemplateEngine/Plugin/InspectorHints.php b/src/Model/TemplateEngine/Plugin/InspectorHints.php index cbe07ee9..b2c6b6af 100644 --- a/src/Model/TemplateEngine/Plugin/InspectorHints.php +++ b/src/Model/TemplateEngine/Plugin/InspectorHints.php @@ -11,6 +11,7 @@ use Magento\Framework\View\TemplateEngineInterface; use Magento\Store\Model\ScopeInterface; use Magento\Store\Model\StoreManagerInterface; +use OpenForgeProject\MageForge\Model\Config\Inspector as InspectorConfig; use OpenForgeProject\MageForge\Model\TemplateEngine\Decorator\InspectorHintsFactory; /** @@ -20,8 +21,6 @@ */ class InspectorHints { - private const XML_PATH_INSPECTOR_ENABLED = 'dev/mageforge_inspector/enabled'; - /** * @param ScopeConfigInterface $scopeConfig * @param StoreManagerInterface $storeManager @@ -57,7 +56,12 @@ public function afterCreate( // Check if inspector is enabled in configuration $storeCode = $this->storeManager->getStore()->getCode(); - if (!$this->scopeConfig->isSetFlag(self::XML_PATH_INSPECTOR_ENABLED, ScopeInterface::SCOPE_STORE, $storeCode)) { + $isEnabled = $this->scopeConfig->isSetFlag( + InspectorConfig::XML_PATH_ENABLED, + ScopeInterface::SCOPE_STORE, + $storeCode + ); + if (!$isEnabled) { return $invocationResult; } diff --git a/src/etc/adminhtml/system.xml b/src/etc/adminhtml/system.xml index 4de1c0d1..898b67c4 100644 --- a/src/etc/adminhtml/system.xml +++ b/src/etc/adminhtml/system.xml @@ -28,6 +28,12 @@ mageforge/inspector/show_button_labels Show text labels on the Toolbar and Inspector buttons. Default: Yes. + + + OpenForgeProject\MageForge\Model\Config\Source\ToolbarPosition + mageforge/inspector/position + Position of the MageForge Toolbar on the page. Default: Bottom Left. + diff --git a/src/etc/config.xml b/src/etc/config.xml index 037f5b60..01c0a907 100644 --- a/src/etc/config.xml +++ b/src/etc/config.xml @@ -11,6 +11,7 @@ dark 1 + bottom-left diff --git a/src/i18n/de_DE.csv b/src/i18n/de_DE.csv new file mode 100644 index 00000000..ea9c2864 --- /dev/null +++ b/src/i18n/de_DE.csv @@ -0,0 +1,18 @@ +"MageForge","MageForge" +"Toolbar","Toolbar" +"General Toolbar Settings","Allgemeine Toolbar-Einstellungen" +"Show Toolbar","Toolbar anzeigen" +"Enable or disable the MageForge Toolbar. Default: Yes.","MageForge Toolbar aktivieren oder deaktivieren. Standard: Ja." +"Theme","Design" +"Choose between Dark, Light, or Auto (System Preference) theme.","Wähle zwischen Dunkel, Hell oder Auto (Systemeinstellung)." +"Show Button Labels","Schaltflächenbeschriftungen anzeigen" +"Show text labels on the Toolbar and Inspector buttons. Default: Yes.","Textbeschriftungen auf Toolbar- und Inspector-Schaltflächen anzeigen. Standard: Ja." +"Toolbar Position","Toolbar-Position" +"Position of the MageForge Toolbar on the page. Default: Bottom Left.","Position der MageForge Toolbar auf der Seite. Standard: Unten links." +"Dark","Dunkel" +"Light","Hell" +"Auto (System Preference)","Auto (Systemeinstellung)" +"Bottom Left","Unten links" +"Bottom Right","Unten rechts" +"Top Left","Oben links" +"Top Right","Oben rechts" diff --git a/src/i18n/en_US.csv b/src/i18n/en_US.csv new file mode 100644 index 00000000..ac329230 --- /dev/null +++ b/src/i18n/en_US.csv @@ -0,0 +1,18 @@ +"MageForge","MageForge" +"Toolbar","Toolbar" +"General Toolbar Settings","General Toolbar Settings" +"Show Toolbar","Show Toolbar" +"Enable or disable the MageForge Toolbar. Default: Yes.","Enable or disable the MageForge Toolbar. Default: Yes." +"Theme","Theme" +"Choose between Dark, Light, or Auto (System Preference) theme.","Choose between Dark, Light, or Auto (System Preference) theme." +"Show Button Labels","Show Button Labels" +"Show text labels on the Toolbar and Inspector buttons. Default: Yes.","Show text labels on the Toolbar and Inspector buttons. Default: Yes." +"Toolbar Position","Toolbar Position" +"Position of the MageForge Toolbar on the page. Default: Bottom Left.","Position of the MageForge Toolbar on the page. Default: Bottom Left." +"Dark","Dark" +"Light","Light" +"Auto (System Preference)","Auto (System Preference)" +"Bottom Left","Bottom Left" +"Bottom Right","Bottom Right" +"Top Left","Top Left" +"Top Right","Top Right" diff --git a/src/view/frontend/templates/inspector.phtml b/src/view/frontend/templates/inspector.phtml index 1301d9a4..07c93b8e 100644 --- a/src/view/frontend/templates/inspector.phtml +++ b/src/view/frontend/templates/inspector.phtml @@ -68,6 +68,7 @@ JS;
diff --git a/src/view/frontend/web/css/toolbar.css b/src/view/frontend/web/css/toolbar.css index 35959ff9..d9abd705 100644 --- a/src/view/frontend/web/css/toolbar.css +++ b/src/view/frontend/web/css/toolbar.css @@ -568,6 +568,46 @@ color: var(--mageforge-color-blue); } +/* ============================================================================ + Position Variants + ========================================================================== */ + +.mageforge-toolbar[data-position="bottom-right"] { + bottom: 16px; + left: auto; + right: 16px; +} + +.mageforge-toolbar[data-position="top-left"] { + bottom: auto; + top: 16px; + left: 16px; +} + +.mageforge-toolbar[data-position="top-right"] { + bottom: auto; + top: 16px; + left: auto; + right: 16px; +} + +/* Menu opens downward for top positions */ +.mageforge-toolbar[data-position^="top"] .mageforge-toolbar-menu { + bottom: auto; + top: calc(100% + 8px); + transform: translateY(-8px); +} + +.mageforge-toolbar[data-position^="top"] .mageforge-toolbar-menu.mageforge-menu-open { + transform: translateY(0); +} + +/* Menu anchors to right edge for right positions */ +.mageforge-toolbar[data-position$="right"] .mageforge-toolbar-menu { + left: auto; + right: 0; +} + /* ============================================================================ Theme Overrides ========================================================================== */ @@ -625,6 +665,21 @@ bottom: 10px; left: 10px; } + .mageforge-toolbar[data-position="bottom-right"] { + left: auto; + right: 10px; + } + .mageforge-toolbar[data-position="top-left"] { + bottom: auto; + left: 10px; + top: 10px; + } + .mageforge-toolbar[data-position="top-right"] { + bottom: auto; + left: auto; + right: 10px; + top: 10px; + } .mageforge-toolbar-menu { min-width: 300px; max-height: calc(100vh - 60px); diff --git a/src/view/frontend/web/js/toolbar/ui.js b/src/view/frontend/web/js/toolbar/ui.js index 9bb45bc7..d11887c4 100644 --- a/src/view/frontend/web/js/toolbar/ui.js +++ b/src/view/frontend/web/js/toolbar/ui.js @@ -11,6 +11,10 @@ export const uiMethods = { this.container.setAttribute('data-theme', this.$el.getAttribute('data-theme')); } + if (this.$el && this.$el.hasAttribute('data-position')) { + this.container.setAttribute('data-position', this.$el.getAttribute('data-position')); + } + if (this.$el && this.$el.getAttribute('data-show-labels') === '0') { this.container.classList.add('mageforge-toolbar--no-labels'); } From 90d35a325ccb0c5111e52ec794ae4b9f4b976239 Mon Sep 17 00:00:00 2001 From: Mathias Elle Date: Thu, 7 May 2026 00:16:16 +0200 Subject: [PATCH 3/8] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- src/view/frontend/web/css/toolbar.css | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/src/view/frontend/web/css/toolbar.css b/src/view/frontend/web/css/toolbar.css index d9abd705..c15246e0 100644 --- a/src/view/frontend/web/css/toolbar.css +++ b/src/view/frontend/web/css/toolbar.css @@ -600,6 +600,10 @@ .mageforge-toolbar[data-position^="top"] .mageforge-toolbar-menu.mageforge-menu-open { transform: translateY(0); + + @starting-style { + transform: translateY(-8px); + } } /* Menu anchors to right edge for right positions */ From b90c20dff9f5847af4543c173a0358e12607ddf1 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 6 May 2026 22:16:56 +0000 Subject: [PATCH 4/8] #181 - Fix translate attribute to include comment for Toolbar Position field in system.xml Agent-Logs-Url: https://github.com/OpenForgeProject/mageforge/sessions/5f83d607-c501-46ef-81b6-eb950da6237b Co-authored-by: dermatz <6103201+dermatz@users.noreply.github.com> --- src/etc/adminhtml/system.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/etc/adminhtml/system.xml b/src/etc/adminhtml/system.xml index 898b67c4..7cf2cc65 100644 --- a/src/etc/adminhtml/system.xml +++ b/src/etc/adminhtml/system.xml @@ -28,7 +28,7 @@ mageforge/inspector/show_button_labels Show text labels on the Toolbar and Inspector buttons. Default: Yes. - + OpenForgeProject\MageForge\Model\Config\Source\ToolbarPosition mageforge/inspector/position From 7b34d6308ca4cafbb20710120d7c87f6c7859351 Mon Sep 17 00:00:00 2001 From: Mathias Elle Date: Thu, 7 May 2026 00:17:06 +0200 Subject: [PATCH 5/8] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- src/i18n/en_US.csv | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/i18n/en_US.csv b/src/i18n/en_US.csv index ac329230..959513c4 100644 --- a/src/i18n/en_US.csv +++ b/src/i18n/en_US.csv @@ -2,7 +2,7 @@ "Toolbar","Toolbar" "General Toolbar Settings","General Toolbar Settings" "Show Toolbar","Show Toolbar" -"Enable or disable the MageForge Toolbar. Default: Yes.","Enable or disable the MageForge Toolbar. Default: Yes." +"Enable or disable the MageForge Toolbar. Default: No.","Enable or disable the MageForge Toolbar. Default: No." "Theme","Theme" "Choose between Dark, Light, or Auto (System Preference) theme.","Choose between Dark, Light, or Auto (System Preference) theme." "Show Button Labels","Show Button Labels" From 67a27d426fb147a9608775e1883086fae582cb8c Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 6 May 2026 22:20:21 +0000 Subject: [PATCH 6/8] #181 - Fix Show Toolbar default description to say No in de_DE CSV and system.xml Agent-Logs-Url: https://github.com/OpenForgeProject/mageforge/sessions/ad13f4cf-bed2-4f23-8ccd-d12bc2921c86 Co-authored-by: dermatz <6103201+dermatz@users.noreply.github.com> --- src/etc/adminhtml/system.xml | 4 ++-- src/i18n/de_DE.csv | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/etc/adminhtml/system.xml b/src/etc/adminhtml/system.xml index 7cf2cc65..a1b8a00b 100644 --- a/src/etc/adminhtml/system.xml +++ b/src/etc/adminhtml/system.xml @@ -10,11 +10,11 @@ OpenForgeProject_MageForge::config_inspector - + Magento\Config\Model\Config\Source\Yesno dev/mageforge_inspector/enabled - Enable or disable the MageForge Toolbar. Default: Yes. + Enable or disable the MageForge Toolbar. Default: No. diff --git a/src/i18n/de_DE.csv b/src/i18n/de_DE.csv index ea9c2864..3ab18754 100644 --- a/src/i18n/de_DE.csv +++ b/src/i18n/de_DE.csv @@ -2,7 +2,7 @@ "Toolbar","Toolbar" "General Toolbar Settings","Allgemeine Toolbar-Einstellungen" "Show Toolbar","Toolbar anzeigen" -"Enable or disable the MageForge Toolbar. Default: Yes.","MageForge Toolbar aktivieren oder deaktivieren. Standard: Ja." +"Enable or disable the MageForge Toolbar. Default: No.","MageForge Toolbar aktivieren oder deaktivieren. Standard: Nein." "Theme","Design" "Choose between Dark, Light, or Auto (System Preference) theme.","Wähle zwischen Dunkel, Hell oder Auto (Systemeinstellung)." "Show Button Labels","Schaltflächenbeschriftungen anzeigen" From ed3f4bcf9803f289cb114ddd50d7cadb9b0c0dc7 Mon Sep 17 00:00:00 2001 From: Mathias Elle Date: Thu, 7 May 2026 00:20:49 +0200 Subject: [PATCH 7/8] Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> --- src/i18n/en_US.csv | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/i18n/en_US.csv b/src/i18n/en_US.csv index 959513c4..fceb6c2b 100644 --- a/src/i18n/en_US.csv +++ b/src/i18n/en_US.csv @@ -6,7 +6,7 @@ "Theme","Theme" "Choose between Dark, Light, or Auto (System Preference) theme.","Choose between Dark, Light, or Auto (System Preference) theme." "Show Button Labels","Show Button Labels" -"Show text labels on the Toolbar and Inspector buttons. Default: Yes.","Show text labels on the Toolbar and Inspector buttons. Default: Yes." +"Show text labels on the Toolbar and Inspector buttons. Default: No.","Show text labels on the Toolbar and Inspector buttons. Default: No." "Toolbar Position","Toolbar Position" "Position of the MageForge Toolbar on the page. Default: Bottom Left.","Position of the MageForge Toolbar on the page. Default: Bottom Left." "Dark","Dark" From 972ae0313c85afcd1338f806552a173d5548a42a Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 6 May 2026 22:27:03 +0000 Subject: [PATCH 8/8] #181 - Add translate="label comment" to theme and show_button_labels fields; fix en_US Show Button Labels default Agent-Logs-Url: https://github.com/OpenForgeProject/mageforge/sessions/662a5940-af5f-45fe-ab1e-0ac89f1bb0f7 Co-authored-by: dermatz <6103201+dermatz@users.noreply.github.com> --- src/etc/adminhtml/system.xml | 4 ++-- src/i18n/en_US.csv | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/etc/adminhtml/system.xml b/src/etc/adminhtml/system.xml index a1b8a00b..697ba9dd 100644 --- a/src/etc/adminhtml/system.xml +++ b/src/etc/adminhtml/system.xml @@ -16,13 +16,13 @@ dev/mageforge_inspector/enabled Enable or disable the MageForge Toolbar. Default: No. - + OpenForgeProject\MageForge\Model\Config\Source\InspectorTheme mageforge/inspector/theme Choose between Dark, Light, or Auto (System Preference) theme. - + Magento\Config\Model\Config\Source\Yesno mageforge/inspector/show_button_labels diff --git a/src/i18n/en_US.csv b/src/i18n/en_US.csv index fceb6c2b..959513c4 100644 --- a/src/i18n/en_US.csv +++ b/src/i18n/en_US.csv @@ -6,7 +6,7 @@ "Theme","Theme" "Choose between Dark, Light, or Auto (System Preference) theme.","Choose between Dark, Light, or Auto (System Preference) theme." "Show Button Labels","Show Button Labels" -"Show text labels on the Toolbar and Inspector buttons. Default: No.","Show text labels on the Toolbar and Inspector buttons. Default: No." +"Show text labels on the Toolbar and Inspector buttons. Default: Yes.","Show text labels on the Toolbar and Inspector buttons. Default: Yes." "Toolbar Position","Toolbar Position" "Position of the MageForge Toolbar on the page. Default: Bottom Left.","Position of the MageForge Toolbar on the page. Default: Bottom Left." "Dark","Dark"