Table with Radio Button/CheckBox

[m21-cerutti]

Version/Branch of Dear ImGui:

Version: 1.83
Branch: master

Back-end/Renderer/Compiler/OS

Back-ends: imgui_impl_opengl3.cpp + custom OpenInventor backend
Compiler: MSVC 14.16
Operating System: Windows 10

My Issue/Question:
Linked to #125.

Hello, I am trying to do a table with radio button for selectionning one item in a table.
But the problem is it don't actualise the state, and never trigger that a radio button have been clicked.
I have also tried with CheckBox and Selectionable and slight difference in code, but same result.
There is also no demo from what i have seen in this version using this particular case.

Screenshots/Video

Standalone, minimal, complete and verifiable example: (see #2261)

struct ImGuiParameterState
{
  int selected_radio;
};

static ImGuiParameterState state;

if (ImGui::Begin("Control Panel"))
{
  if (ImGui::BeginTable("9x9_radio_table", 3))
  {
    int i;
    for (i = 0; i < 9; i++)
    {
      ImGui::TableNextColumn();
      if (ImGui::RadioButton("", &state.selected_radio, i))
      {
        std::cout << i << std::endl; // Never called
      }
    }
    ImGui::EndTable();
  }
  
  ImGui::End();
}

[ocornut]

Hello,
This is answered by the FAQ:

Q: Why is my widget not reacting when I click on it?
Q: How can I have widgets with an empty label?
Q: How can I have multiple widgets with the same label?

https://github.com/ocornut/imgui/blob/master/docs/FAQ.md#q-why-is-my-widget-not-reacting-when-i-click-on-it

[m21-cerutti]

Oh i'm verry sorry, i have read too quickly the usage.
I think although it can be a good example, here the modified code :

if (ImGui::BeginTable("9x9_radio_table", 3))
{
  for (int i = 0; i < 9; i++)
  {
    ImGui::TableNextColumn();
    std::string id = "##radio_table_" + std::to_string(i);
    if (ImGui::RadioButton(id.c_str(), &state.selected_radio, i))
    {
      int row = ImGui::TableGetRowIndex();
      int col = ImGui::TableGetColumnIndex();
      std::cout << row << ";" << col << std::endl;
    }
  }
  ImGui::EndTable();
}

[ocornut]

You might want to reread this FAQ entry carefully, because in this situation using PushID(i) would be both simpler and faster.

  for (int i = 0; i < 9; i++)
  {
    ImGui::TableNextColumn();
    ImGui::PushID(i);
    if (ImGui::RadioButton("", &state.selected_radio, i))
    {
      int row = ImGui::TableGetRowIndex();
      int col = ImGui::TableGetColumnIndex();
      std::cout << row << ";" << col << std::endl;
    }
    ImGui::PopID();
  }