Skip to content

Conversation

@radurentea
Copy link
Collaborator

@radurentea radurentea commented Jul 23, 2025

Description

This pull request updates the status bar item creation for OpenOCD and QEMU managers to use left alignment and lower priority values for better visual organization in the VS Code status bar. It was also reported by @AndriiFilippov that on a smaller screens or with multiple extensions installed the OpenOCD Server button, would sometimes not be shown. This should be an improvement for those scenarios.

Status bar improvements for OpenOCD and QEMU managers:

  • src/espIdf/openOcd/openOcdManager.ts: Modified the registerOpenOCDStatusBarItem() method to use left alignment (vscode.StatusBarAlignment.Left) and priority 1 instead of right alignment and priority 1000. This ensures the OpenOCD status bar item appears on the left side with lower priority.

  • src/qemu/qemuManager.ts: Updated the registerQemuStatusBarItem() method to use left alignment (StatusBarAlignment.Left) and priority 0 instead of right alignment and priority 1005. This positions the QEMU status bar item on the left side with the lowest priority.

Status bar item naming improvements:

  • src/qemu/qemuManager.ts: Changed QEMU status bar item text from "[ESP-IDF: QEMU]" to "QEMU" to remove brackets and prefix.

  • src/espIdf/openOcd/openOcdManager.ts: Changed OpenOCD status bar item text from "[ESP-IDF: OpenOCD Server]" to "OpenOCD Server" to remove brackets and prefix.

  • src/cmdTreeView/cmdStore.ts: Updated command tooltips to remove "ESP-IDF:" prefix and brackets for better consistency:

    • "ESP-IDF: Select Flash Method""Select Flash Method"
    • "ESP-IDF: Build, Flash and Monitor""Build, Flash and Monitor"
    • "[OpenOCD Server]""OpenOCD Server"
    • "Debug""Launch Debug" (to avoid confusion with VS Code's built-in Debug status bar item)

Localization updates:

  • package.nls.json: Added new localization string "Launch Debug": "Launch Debug".

  • l10n/bundle.l10n.ru.json: Updated Russian translations for command tooltips and added "Launch Debug": "Запуск отладки".

  • l10n/bundle.l10n.zh-CN.json: Updated Chinese translations for command tooltips and added "Launch Debug": "启动调试".

  • l10n/bundle.l10n.es.json: Updated Spanish translations for command tooltips and added "Launch Debug": "Iniciar depuración".

  • l10n/bundle.l10n.pt.json: Updated Portuguese translations for command tooltips and added "Launch Debug": "Iniciar depuración".

Benefits:

  • Consistent positioning: Both OpenOCD and QEMU status bar items now appear on the left side of the status bar, providing a more organized layout.
  • Lower priority: Using priorities 0 and 1 ensures these items appear at the end of the status bar, reducing visual clutter for higher-priority items.
  • Cleaner naming: Removed brackets and "ESP-IDF:" prefix from status bar items and command tooltips for more concise display.
  • Better differentiation: Changed "Debug" to "Launch Debug" to avoid confusion with VS Code's built-in Debug status bar item.
  • Maintained functionality: All existing functionality is preserved, including tooltips, commands, and visibility states.
  • Improved localization: All changes are properly translated into supported languages (Russian, Chinese, Spanish, Portuguese).

Type of change

  • UI enhancement

Steps to test this pull request

Both ESP-IDF: QEMU and ESP-IDF: OpenOCD Server status bar buttons should appear as last items on the left side status bar items.
image

Test Configuration:

  • ESP-IDF Version: 6.0.0
  • OS (Windows,Linux and macOS): Win 11

Checklist

  • PR Self Reviewed
  • Applied Code formatting
  • Added Documentation
  • Added Unit Test
  • Verified on all platforms - Windows,Linux and macOS

@radurentea radurentea self-assigned this Jul 23, 2025
@github-actions
Copy link

github-actions bot commented Jul 23, 2025

Download the artifacts for this pull request:
You can test these changes by installing this VSIX by click menu View -> Command Palette..., type Install from VSIX and then select downloaded esp-idf-extension.vsix file to install the extension.

@radurentea radurentea added this to the v1.11.0 milestone Jul 23, 2025
@radurentea radurentea requested a review from brianignacio5 July 23, 2025 12:35
Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@radurentea radurentea requested a review from Fabricio-ESP July 24, 2025 09:16
@Fabricio-ESP Fabricio-ESP requested review from Fabricio-ESP and removed request for Fabricio-ESP August 1, 2025 11:32
@radurentea radurentea force-pushed the enhance/move-all-status-bar-items-to-left branch from 3521eae to d2ed52f Compare August 6, 2025 15:26
Copy link
Collaborator

@brianignacio5 brianignacio5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@Fabricio-ESP Fabricio-ESP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the change.

@radurentea radurentea force-pushed the enhance/move-all-status-bar-items-to-left branch from d2ed52f to 5efb5dc Compare August 18, 2025 08:04
@radurentea radurentea merged commit 792d6f8 into master Aug 18, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants