Thanks!

It seems to fix the problem from the linked bug, but I don't understand the solution.
Could you please help me understand it?

I believe we should reinvent the whole think from the ground up. Instead of 2-level of focus in between focus and selected decorator, we should have only one. To treat the priority, we should encode in hbox and vbox a way to prioritize one of the child based on a value provided.

Could you please help me understand it?

Sure!
The key issue - every time a Container is rendered, the active child of the component is found out with

Component ActiveChild() override {
  if (children_.empty()) {
    return nullptr;
  }
   return children_[static_cast<size_t>(*selector_) % children_.size()];
}

a component is returned as active regardless of Container itself being active
Example: if button is the child of a container

const bool active = Active(); 
...
auto focus_management = focused ? focus : active ? select : nothing;

the button is always set to active and then sets itself to select. This results the hbox to be selected in ComputeRequirement method, even if the Container is not active

here's a representation of what happens:

My Solution

• We force the ContainerBase::ActiveChild to return nullptr when the Container itself is not active.
• We do this while Render to ensure none of inactive Container's child is selected by ContainerBase::ActiveChild
• Container::Tab is excluded as it needs to render tab's content regardless.

Instead of 2-level of focus in between focus and selected decorator, we should have only one.

If I;m not interpreting this incorrectly, you mean to get rid of either focus or select decorator entirely from FTXUI and re-implement the ComputeRequirement method?

In that case too we'll have to find a way so that the ContainerBase::ActiveChild returns a component only when it's necessary.

What do you think? I'll be happy to help!

Thanks!
I believe the proposed solution would only work when the children are direct child of the Container, but this might not always work when there are multiple nested Container.

There are 3 layers in FTXUI:

1. ftxui/screen - a grid of pixels -> terminal output
2. ftxui/dom - element to be drawn on the screen -> a grid of pixel
3. ftxui/component - Interactive component -> elements to be drawn for a frame.

I believe the problem currently is that the "focusing" happens in both ftxui/dom and ftxui/component.

I believe we should modify Requirement::Selection from:

  enum Selection {
    NORMAL = 0,
    SELECTED = 1,
    FOCUSED = 2,
  };

To

  bool is_selected;

Then we should unite the decorator focus and selected into a single one that would set is_selected.

Then we should add hbox/vbox variants with a second argument, an index representing "how to merge" the Requirement. We should reimplement Hbox::ComputeRequirement to set is_selected based whether one child is selected, and set selected_box based on the selected box of the selected children.
If the selected children doesn't have any selected box, we must selected another one.

Finally, we should Modify Container::Horizontal and Container::Vertical to use the second version of hbox and vbox, plubming the currently selected children as the selected element.

What do you think?