Feature: Add an entry point for manually downloading the corresponding firmware - #99
Open
laride wants to merge 1 commit into
Open
Feature: Add an entry point for manually downloading the corresponding firmware#99laride wants to merge 1 commit into
laride wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
Adds a UI entry point that lets users manually download the currently selected firmware binary (as an alternative to web flashing) to improve usability for unsupported browsers/environments.
Changes:
- Adds a “Download Firmware File” trigger and a Bootstrap modal with links to manual flashing docs and a download button.
- Adds
updateFirmwareDownloadLink()to keep the download URL in sync with chipset/devkit/framework/device selection. - Adds CSS styling for the new trigger container.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| js/index.js | Computes and updates the firmware download URL and toggles the download entry visibility based on selection. |
| index.html | Adds the download trigger UI and modal (including the download button). |
| css/custom.css | Styles the new download trigger container to match existing UI spacing. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
laride
force-pushed
the
feat/add-download-fw-btn
branch
from
August 2, 2026 03:57
649af97 to
aa5c75a
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
Suppressed comments (1)
css/custom.css:338
- The new
.firmware-download-trigger-containerrule duplicates the existing.app-info-trigger-containerstyling, which makes future layout tweaks easy to miss/inconsistently apply. Consider combining them into a single shared rule.
.firmware-download-trigger-container {
padding-top: 14px;
margin-left: 18px;
}
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Add an entry point for manually downloading the corresponding firmware.
Due to browser compatibility and other environmental factors, we cannot guarantee that Launchpad will work as expected for every user. Some users already have Flash Download Tool installed or are familiar with esptool, and may prefer to manually download and flash the firmware as an alternative.
Checklist
Before submitting a Pull Request, please ensure the following: