ArduPilot / ArduPilot/MissionPlanner
New Compass ordering screen
- Dominant language
- C#
- Stars
- 2.4k
- Forks
- 2.9k
- Avg merge
- 19h 16m
- Merged PRs (30d)
- 4
Description

While reviewing's @bugobliterator's compass ordering PR https://github.com/ArduPilot/ardupilot/pull/12629 I also tested MP's new screen. Overall it's a good looking screen but I quickly found several issues:
- does it really support 8 many rows? We don't have this many parameters in AP so I can't see how it could
- row highlighting:
- checking on the top "Set" column colours them all but it's not clear what this is for
- if the user can't modify individual fields, maybe it would be better to have the whole row light up
- why highlight a row at all?
- if multiple rows are selected, I might then expect the Up/Down buttons to affect both rows but it doesn't. I'd remove the ability to select multiple rows.
- a more common method is to have just two up/down buttons that affect the highlighted rows
- disconnecting the board and then pushing the up/down buttons causes a scary error message to appear
- reboot required message issues:
- moving away from the screen caused a "Reboot required" message to appear but the flight controller was disconnected. This message box appeared every time I moved off the screen no matter whether I answered "yes" or "no"
- reboot required appeared again when I clicked on the flight data screen even when I wasn't looking at the compass screen anymore and I'd already said "yes" once and the flight controller wasn't even connected
- I think it would be best to get rid of this pop-up completely and instead add a "Reboot" button below the "Compass Priority" table. Perhaps change the colour of the button if the user changes the table in a way that means a reboot is required.
- similar to [this other issue re the HW ID page](https://github.com/ArduPilot/MissionPlanner/issues/2336), it would be nice to get rid of the "BUS_TYPE_" prefix for the values appearing in the "BusType" column
- "compass's" should be "compasses". Perhaps we should replace "moving around" with "reordering".
Contributor guide
No contributing guide indexed for this repository
Research direction
Locate the Compass ordering screen in Mission Planner and reproduce the reported selection, disconnected-board, reboot-message, BusType, and wording behaviors. Review the linked compass ordering PR and related HW ID issue for context; done means the screen's ordering controls, reboot handling, displayed values, and text behave consistently with the requested changes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- csharp
- Domain
- desktop, frontend
- Issue type
- Bug
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100