ArduPilot / ArduPilot/MissionPlanner

Plan: WP Rdius field selected by default when switching to screen

Open
#2,576 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C#
Stars
2.4k
Forks
2.9k
Avg merge
19h 16m
Merged PRs (30d)
4

Description

It seems like MP's puts the cursor into the "WP Radius" field when the user switches to the Plan screen. This is dangerous because it means if the user presses a key (especially a number) the field can be cleared which leads to issues (e.g. https://github.com/ArduPilot/ardupilot/issues/15070) where the WPNAV_RADIUS field is set incorrectly.
![image](https://user-images.githubusercontent.com/1498098/111392528-1f4cb980-86fa-11eb-8ce6-d19f33abe226.png)

I think it would be best if no fields are selected by default when switching to the Plan screen.

Contributor guide

No contributing guide indexed for this repository

Research direction

Reproduce the issue by switching to the Plan screen and checking whether the WP Radius field receives focus. Trace the screen-switch focus handling and verify that no field is selected by default, including after keyboard input is sent immediately after switching.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
desktop
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.