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

Combine Standard Params and Advanced params panel #1231

Open
wucke13 opened this issue Feb 4, 2021 · 3 comments
Open

Combine Standard Params and Advanced params panel #1231

wucke13 opened this issue Feb 4, 2021 · 3 comments

Comments

@wucke13
Copy link

wucke13 commented Feb 4, 2021

I find my self jumping between both all the time. I think the UX would be way better, if both were combined. Maybe some visual marker next to the parameter name (parameter id) that marks advanced parameters could be used instead to still communicate to the user which parameters require attention when changing.

@Arne-W
Copy link
Contributor

Arne-W commented Feb 9, 2021

@wucke13 I thought about removing those 2 pages from the configuration because it is a lot of work to maintain them.
Surely you have seen that some parameters no longer work, which is due to the fact that some parameters have been renamed or removed.
Therefore, we have to adapt and maintain these pages for all ardupilot versions, which is relatively time-consuming.
Why don't you use the full parameter list? And which parameters are you using there?

@wucke13
Copy link
Author

wucke13 commented Feb 9, 2021

Therefore, we have to adapt and maintain these pages for all ardupilot versions, which is relatively time-consuming.

Not if you do it clever. It is possible to synthesize a reasonable good UI from the parameter definition files.

Why don't you use the full parameter list?

For it to work, the search needs to Filter, e.g. reduce the number of rows in the table. Also the information is very impractical, because most of it is cut away. Having the info about the currently selected row in the param table in a separate table would be a good start.

@Arne-W
Copy link
Contributor

Arne-W commented Feb 21, 2021

@wucke13 Doing things clever is always a good Idea 😄 but nevertheless it takes a lot of time to do such a change and I do not have the time to do such a restructuring at the moment - sorry.
Perhaps we can do something in the full parameter list to improve the user experience.

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

No branches or pull requests

2 participants