Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Common: Update aux switch table to be an RST simple table #5479

Merged
merged 1 commit into from
Dec 12, 2023

Conversation

TunaLobster
Copy link
Contributor

This is a format change that should still allow for easy editting and addition of new switch options. This should also keep the table to a reasonable width on the page.

I haven't had a chance to build yet. Haven't finished settting up on this new machine.

@TunaLobster TunaLobster force-pushed the pr/aux-table branch 2 times, most recently from a29ec7b to 2766b42 Compare October 13, 2023 16:09
@Hwurzburg
Copy link
Contributor

does not change appearance or scroll bar issue....it would be simpler to maintain

@TunaLobster
Copy link
Contributor Author

Figured out that rst tables don't word wrap in the cell. I added a couple of line blocks to force wrapping. So the scroll bar is shorter now.

@Hwurzburg
Copy link
Contributor

Hwurzburg commented Oct 13, 2023

might as well do that on every long line to prevent it running more than 2x the width and have a short bar
and same trick to make the first column more narrow, improving the second one at the same time

@Hwurzburg
Copy link
Contributor

Hwurzburg commented Dec 12, 2023

going to merge this after I fix conflicts...I am going to reformat so the slider is eliminated...will take some time...thanks

@Hwurzburg Hwurzburg merged commit 6acd7c5 into ArduPilot:master Dec 12, 2023
4 checks passed
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.

2 participants