edgetx icon indicating copy to clipboard operation
edgetx copied to clipboard

Incompleteness of Input Selection with Filtering

Open richardclli opened this issue 3 years ago • 10 comments

Describe the bug Channel/Switch selection is incomplete when filter is used

To Reproduce

  1. Choose a switch in Inputs, or Logical Switch screens, etc.
  2. By default a complete list is shown
  3. After tapping the filter tab, some choices in the default list is not shown, e.g. "---", "6P6", ... "6P1"

Expected behavior Need to review the filtering and control. Need to have a complete list e.g. "---" must be included, and or the default list maybe wrong.

Screenshots screenshot_tx16s_21-11-11_12-54-42 screenshot_tx16s_21-11-11_12-54-33 screenshot_tx16s_21-11-11_12-54-57

General information

  • EdgeTX version 2.5.0
  • Hardware TX16S

richardclli avatar Nov 11 '21 04:11 richardclli

This issue elaborates on this somewhat also: https://github.com/EdgeTX/edgetx/issues/228

pfeerick avatar Nov 11 '21 11:11 pfeerick

@pfeerick are you working now on popup list? #212

JimB40 avatar Nov 11 '21 14:11 JimB40

This is the filters, not the menu ;)

pfeerick avatar Nov 11 '21 23:11 pfeerick

Okay let's be on the same page with naming What I see above is called in UI list This particular list has additional filter options displayed as buttons Above shown list is part of pop-up window. This particular pop-up window has no window header displayed.

Can you explain now what is the difference from code perspective? And what you call menu? Someone did above as separate piece of code?

JimB40 avatar Nov 12 '21 08:11 JimB40

Changing the appearance of the menu (which is what you are thinking of as to re : what I'm working on for the menu) is not the same as changing the behaviour of, or changing the function that generate the data to be displayed.

pfeerick avatar Nov 12 '21 08:11 pfeerick

What you call "Menu"? :) in #228 above shown you called "pop-up list filters" in #212 there is "pop-up header" still there is a "selectable list" beneath as in #228

Different piece of code? and what is menu? :)

JimB40 avatar Nov 12 '21 08:11 JimB40

They are all menus... (a popup window showing a list). If you set a title, it has a title (window header), if you set a toolbar, it has a toolbar on the side (buttons), which is being used as a filter. Now do you see why I simply circled the screen element in that issue and said "this isn't working" rather than worry about "what is it called"? 😆

Changing the appearance generally means changes to libopenui, and to some extent the behaviour of the menu itself. Changing what is displayed, and determined what data to display is something that needs to be changed in the relevant place. i.e. inputs source list gets changed on inputs page code, logical switches switch gets changed on logical switches page code, etc, etc. As I said, changing appearance is not the same as changing functionality or data that goes into it, so this is not the same or related issue ;)

pfeerick avatar Nov 12 '21 09:11 pfeerick

I know that. just asking if we are talking about same piece of code. So as I understand any changes to pop-up UI (colors etc) affect all pop-ups no mater if they display commands for field, available source list, available switches list etc

JimB40 avatar Nov 12 '21 11:11 JimB40

This issue is still present in a0770beb5f8616e2f14f2828de12b3ff8b1f3397. It seems like it is also worse, in that it seems presses of PAGE > and PAGE < to switch between the filters either has a lot of lag, or is being ignored. Also, multiple filters are hi-lighted when pressed, instead of being mutually exclusive.

pfeerick avatar Aug 03 '22 04:08 pfeerick

And while we're at it... here is another filters related issue #2344

pfeerick avatar Sep 22 '22 11:09 pfeerick