FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

MavLinkShell: Corrected Backspace keystroke read by Siri2K · Pull Request #25637 · PX4/PX4-Autopilot · GitHub

MavLinkShell: Corrected Backspace keystroke read - #25637

Merged
dakejahl merged 1 commit into
PX4:mainfrom
Siri2K:pr-mavlink-shell-backspace-correction
Sep 25, 2025
Merged

dakejahl merged 1 commit into
PX4:mainfrom
Siri2K:pr-mavlink-shell-backspace-correction

Conversation

Siri2K commented Sep 25, 2025
edited
Loading

Copy link
Copy Markdown
Contributor

Changed command ASCII used for backspace

Solved Problem

ASCII used for backspace was actually the "Delete" key ASCII Key 127
Fixes #25601

Solution

  • ASCII used for backspace was actually the "Delete" key ASCII Key 127 changed it to the "Backspace" Key ASCII Key 8` in Tools/mavlink_shell.py

Changelog Entry

For release notes:

Feature/Bugfix  #25601 
New parameter: 
Documentation: Need to clarify page ... / done, read docs.px4.io/...

Alternatives

We could also ...

Test coverage

Context

Related links, screenshot before/after, video

Siri2K marked this pull request as ready for review September 25, 2025 15:53
Changed ASCII command from ASCII #127 to ASCII #8

Good: Changed ASCII command from ASCII #127 to ASCII #8

Reported-by: jsm09a <https://github.com/jsm09a>

Signed-off-by: Siri <sriramj2@hotmail.com>
dakejahl merged commit 5023174 into PX4:main Sep 25, 2025
67 checks passed

Copy link
Copy Markdown
Contributor

Thanks @Siri2K . In the future please make sure to update the PR description to either remove the boiler plate or fill in all of the details.

Siri2K commented Sep 25, 2025 via email

Copy link
Copy Markdown
Contributor Author

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

mavlink_shell backspace not working

2 participants


Back | FazBrowse Home | New Git URL