frontend(tkinter): add clickable link to tuning guide in step 11 dialog#995
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a clickable hyperlink to the tuning guide in a dialog that prompts users about copying flight controller values to a file after external configuration changes. The link provides quick access to relevant documentation about the experimental procedure.
Key Changes
- Added a clickable "Open Tuning Guide" link with appropriate styling (blue, underlined, hand cursor)
- The link opens a specific section of the ArduCopter tuning guide related to initial attitude PID gains
|
This very same dialog is used in other places in the configuration sequence. |
amilcarlucas
left a comment
There was a problem hiding this comment.
Also please sign-off the commits
7b84864 to
e4f658d
Compare
|
Thanks for the feedback! I’ve made the requested changes and pushed the updated code. |
edb9a5f to
bcc6289
Compare
When transitioning from Step 11 (initial ATC PID tuning) to Step 12, if "Automatically open documentation links..." is disabled, the popup message previously mentioned the tuning guide but did not provide any clickable links, which was confusing to users. This change adds a blue, underlined "Open Tuning Guide" label that opens the relevant ArduPilot tuning instructions page in the default web browser Closes ArduPilot#951 Signed-off-by: neo-0007 <hrishikeshgohain123@gmail.com>
bcc6289 to
59d4a23
Compare
Used the safe font Correct a documentation link Signed-off-by: Dr.-Ing. Amilcar do Carmo Lucas <amilcar.lucas@iav.de>
2b1d931 to
66be939
Compare
|
Thanks for the code, merged |
When transitioning from Step 11 (initial ATC PID tuning) to Step 12, if "Automatically open documentation links..." is disabled, the popup message previously mentioned the tuning guide but did not provide any clickable links, which was confusing to users.
This change adds a blue, underlined "Open Tuning Guide" label that opens the relevant ArduPilot tuning instructions page in the default web browser
Issue: #951
Before :

After :
