LedgerHQ / ledger-vscode-extension

A VS Code extension providing tools to accelerate development of apps for Ledger wallets.
Apache License 2.0
6 stars 3 forks source link

Few extension improvments and fix #41

Closed cedelavergne-ledger closed 3 months ago

cedelavergne-ledger commented 4 months ago

This PR contains:

agrojean-ledger commented 3 months ago

A few remarks regarding the Select build use case user interface, in my opinion :

  1. By moving the selection icon (small "gear") to the build section line, we can remove the line Select build use case which becomes redundant.
  2. The line build with debug mode is redundant with the Debug build use case selection from the new UI element.
  3. We need two default use cases for C apps :
    • Default (release mode compilation, we can actually call it release)
    • Debug (with DEBUG=1). If there is a Debug use case in the ledger_app.toml, it should be displayed instead of the default one.

image