1 files changed, 61 insertions, 44 deletions

diff --git a/charselect.cpp b/charselect.cpp
index 541f1e0f..c0758765 100644
--- a/charselect.cpp
+++ b/charselect.cpp
@@ -26,42 +26,8 @@ void Courtroom::construct_char_select()
   ui_spectator = new AOButton(ui_char_select_background, ao_app);
   ui_spectator->setText("Spectator");
 
-  QPoint f_spacing = ao_app->get_button_spacing("char_button_spacing", "courtroom_design.ini");
-
-  const int button_width = 60;
-  int x_spacing = f_spacing.x();
-  int x_mod_count = 0;
-
-  const int button_height = 60;
-  int y_spacing = f_spacing.y();
-  int y_mod_count = 0;
-
   set_size_and_pos(ui_char_buttons, "char_buttons");
 
-  char_columns = ((ui_char_buttons->width() - button_width) / (x_spacing + button_width)) + 1;
-  char_rows = ((ui_char_buttons->height() - button_height) / (y_spacing + button_height)) + 1;
-
-  max_chars_on_page = char_columns * char_rows;
-
-  for (int n = 0 ; n < max_chars_on_page ; ++n)
-  {
-    int x_pos = (button_width + x_spacing) * x_mod_count;
-    int y_pos = (button_height + y_spacing) * y_mod_count;
-
-    ui_char_button_list.append(new AOCharButton(ui_char_buttons, ao_app, x_pos, y_pos));
-
-    connect(ui_char_button_list.at(n), SIGNAL(clicked()), char_button_mapper, SLOT(map())) ;
-    char_button_mapper->setMapping (ui_char_button_list.at(n), n) ;
-
-    ++x_mod_count;
-
-    if (x_mod_count == char_columns)
-    {
-      ++y_mod_count;
-      x_mod_count = 0;
-    }
-  }
-
   connect (char_button_mapper, SIGNAL(mapped(int)), this, SLOT(char_clicked(int)));
   connect(ui_back_to_lobby, SIGNAL(clicked()), this, SLOT(on_back_to_lobby_clicked()));
 
@@ -97,7 +63,10 @@ void Courtroom::set_char_select_page()
   ui_char_select_right->hide();
 
   for (AOCharButton *i_button : ui_char_button_list)
+  {
     i_button->hide();
+    i_button->move(0,0);
+  }
 
   int total_pages = char_list.size() / max_chars_on_page;
   int chars_on_page = 0;
@@ -121,26 +90,24 @@ void Courtroom::set_char_select_page()
   if (current_char_page > 0)
     ui_char_select_left->show();
 
-  for (int n_button = 0 ; n_button < chars_on_page ; ++n_button)
+  put_button_in_place(current_char_page * max_chars_on_page, chars_on_page);
+
+  /*for (int n_button = 0 ; n_button < chars_on_page ; ++n_button)
   {
     int n_real_char = n_button + current_char_page * max_chars_on_page;
     AOCharButton *f_button = ui_char_button_list.at(n_button);
 
-    f_button->reset();
-    f_button->set_image(char_list.at(n_real_char).name);
     f_button->show();
 
     if (char_list.at(n_real_char).taken)
       f_button->set_taken();
-  }
+  }*/
 
 }
 
 void Courtroom::char_clicked(int n_char)
 {
-  int n_real_char = n_char + current_char_page * max_chars_on_page;
-
-  QString char_ini_path = ao_app->get_character_path(char_list.at(n_real_char).name) + "char.ini";
+  QString char_ini_path = ao_app->get_character_path(char_list.at(n_char).name) + "char.ini";
   qDebug() << "char_ini_path" << char_ini_path;
 
   if (!file_exists(char_ini_path))
@@ -150,15 +117,65 @@ void Courtroom::char_clicked(int n_char)
     return;
   }
 
-  if (n_real_char == m_cid)
+  if (n_char == m_cid)
   {
     enter_courtroom(m_cid);
   }
   else
   {
-    ao_app->send_server_packet(new AOPacket("CC#" + QString::number(ao_app->s_pv) + "#" + QString::number(n_real_char) + "#" + get_hdid() + "#%"));
+    ao_app->send_server_packet(new AOPacket("CC#" + QString::number(ao_app->s_pv) + "#" + QString::number(n_char) + "#" + get_hdid() + "#%"));
   }
 
-  ui_ic_chat_name->setPlaceholderText(char_list.at(n_real_char).name);
+  ui_ic_chat_name->setPlaceholderText(char_list.at(n_char).name);
 }
 
+void Courtroom::put_button_in_place(int starting, int chars_on_this_page)
+{
+    QPoint f_spacing = ao_app->get_button_spacing("char_button_spacing", "courtroom_design.ini");
+
+    int x_spacing = f_spacing.x();
+    int x_mod_count = 0;
+
+    int y_spacing = f_spacing.y();
+    int y_mod_count = 0;
+
+    char_columns = ((ui_char_buttons->width() - button_width) / (x_spacing + button_width)) + 1;
+    char_rows = ((ui_char_buttons->height() - button_height) / (y_spacing + button_height)) + 1;
+
+    int startout = starting;
+    for (int n = starting ; n < startout+chars_on_this_page ; ++n)
+    {
+      int x_pos = (button_width + x_spacing) * x_mod_count;
+      int y_pos = (button_height + y_spacing) * y_mod_count;
+
+      ui_char_button_list.at(n)->move(x_pos, y_pos);
+      ui_char_button_list.at(n)->show();
+
+      ++x_mod_count;
+
+      if (x_mod_count == char_columns)
+      {
+        ++y_mod_count;
+        x_mod_count = 0;
+      }
+    }
+}
+
+void Courtroom::character_loading_finished()
+{
+    // We move them out of the reachable area, so they can't be accidentally clicked.
+    // First, we'll make all the character buttons in the very beginning.
+    // We hide them too, just in case.
+    for (int n = 0; n < char_list.size(); n++)
+    {
+      AOCharButton* character = new AOCharButton(ui_char_buttons, ao_app, 0, 0);
+      character->hide();
+      character->set_image(char_list.at(n).name);
+      ui_char_button_list.append(character);
+
+      connect(character, SIGNAL(clicked()), char_button_mapper, SLOT(map()));
+      char_button_mapper->setMapping(character, ui_char_button_list.size() - 1);
+    }
+
+    put_button_in_place(0, max_chars_on_page);
+}