1 files changed, 77 insertions, 70 deletions

@@ -27,28 +27,26 @@ AreaPopup::AreaPopup(wxWindow* parent, int area_id)
   Bind(wxEVT_PAINT, &AreaPopup::OnPaint, this);
   Bind(wxEVT_DPI_CHANGED, &AreaPopup::OnDPIChanged, this);
 
-  UpdateIndicators();
+  ResetIndicators();
 }
 
-void AreaPopup::UpdateIndicators() {
-  const MapArea& map_area = GD_GetMapArea(area_id_);
+void AreaPopup::ResetIndicators() {
+  indicators_.clear();
 
+  const MapArea& map_area = GD_GetMapArea(area_id_);
   wxFont the_font = GetFont().Scale(GetDPIScaleFactor());
+  TrackerPanel* tracker_panel = dynamic_cast<TrackerPanel*>(GetParent());
 
   // Start calculating extents.
   wxMemoryDC mem_dc;
   mem_dc.SetFont(the_font.Bold());
-  wxSize header_extent = mem_dc.GetTextExtent(map_area.name);
+  header_extent_ = mem_dc.GetTextExtent(map_area.name);
 
-  int acc_height = header_extent.GetHeight() + FromDIP(20);
+  int acc_height = header_extent_.GetHeight() + FromDIP(20);
   int col_width = 0;
 
   mem_dc.SetFont(the_font);
 
-  TrackerPanel* tracker_panel = dynamic_cast<TrackerPanel*>(GetParent());
-
-  std::vector<int> real_locations;
-
   for (int section_id = 0; section_id < map_area.locations.size();
        section_id++) {
     const Location& location = map_area.locations.at(section_id);
@@ -67,7 +65,7 @@ void AreaPopup::UpdateIndicators() {
       }
     }
 
-    real_locations.push_back(section_id);
+    indicators_.emplace_back(section_id, kLOCATION, acc_height);
 
     wxSize item_extent = mem_dc.GetTextExtent(location.name);
     int item_height =
@@ -85,6 +83,8 @@ void AreaPopup::UpdateIndicators() {
         continue;
       }
 
+      indicators_.emplace_back(painting_id, kPAINTING, acc_height);
+
       const PaintingExit& painting = GD_GetPaintingExit(painting_id);
       wxSize item_extent = mem_dc.GetTextExtent(painting.display_name);
       int item_height =
@@ -98,84 +98,91 @@ void AreaPopup::UpdateIndicators() {
   }
 
   int item_width = col_width + FromDIP(10 + 32);
-  int full_width = std::max(header_extent.GetWidth(), item_width) + FromDIP(20);
+  full_width_ = std::max(header_extent_.GetWidth(), item_width) + FromDIP(20);
+  full_height_ = acc_height;
 
   Fit();
-  SetVirtualSize(full_width, acc_height);
+  SetVirtualSize(full_width_, full_height_);
+
+  UpdateIndicators();
+}
+
+void AreaPopup::UpdateIndicators() {
+  const MapArea& map_area = GD_GetMapArea(area_id_);
+  wxFont the_font = GetFont().Scale(GetDPIScaleFactor());
+  TrackerPanel* tracker_panel = dynamic_cast<TrackerPanel*>(GetParent());
+
+  rendered_ = wxBitmap(full_width_, full_height_);
 
-  rendered_ = wxBitmap(full_width, acc_height);
+  wxMemoryDC mem_dc;
   mem_dc.SelectObject(rendered_);
   mem_dc.SetPen(*wxTRANSPARENT_PEN);
   mem_dc.SetBrush(*wxBLACK_BRUSH);
-  mem_dc.DrawRectangle({0, 0}, {full_width, acc_height});
+  mem_dc.DrawRectangle({0, 0}, {full_width_, full_height_});
 
   mem_dc.SetFont(the_font.Bold());
   mem_dc.SetTextForeground(*wxWHITE);
   mem_dc.DrawText(map_area.name,
-                  {(full_width - header_extent.GetWidth()) / 2, FromDIP(10)});
-
-  int cur_height = header_extent.GetHeight() + FromDIP(20);
+                  {(full_width_ - header_extent_.GetWidth()) / 2, FromDIP(10)});
 
   mem_dc.SetFont(the_font);
 
-  for (int section_id : real_locations) {
-    const Location& location = map_area.locations.at(section_id);
-
-    bool checked = false;
-    if (IsLocationWinCondition(location)) {
-      checked = AP_HasReachedGoal();
-    } else if (tracker_panel->IsPanelsMode()) {
-      const Panel& panel = GD_GetPanel(*location.single_panel);
-      if (panel.non_counting) {
-        checked = AP_HasCheckedGameLocation(location.ap_location_id);
-      } else {
-        checked = tracker_panel->GetSolvedPanels().contains(panel.nodepath);
+  for (const IndicatorInfo& indicator : indicators_) {
+    switch (indicator.type) {
+      case kLOCATION: {
+        const Location& location = map_area.locations.at(indicator.id);
+
+        bool checked = false;
+        if (IsLocationWinCondition(location)) {
+          checked = AP_HasReachedGoal();
+        } else if (tracker_panel->IsPanelsMode()) {
+          const Panel& panel = GD_GetPanel(*location.single_panel);
+          if (panel.non_counting) {
+            checked = AP_HasCheckedGameLocation(location.ap_location_id);
+          } else {
+            checked = tracker_panel->GetSolvedPanels().contains(panel.nodepath);
+          }
+        } else {
+          checked = AP_HasCheckedGameLocation(location.ap_location_id) ||
+                    (location.hunt &&
+                     AP_HasCheckedHuntPanel(location.ap_location_id));
+        }
+
+        const wxBitmap* eye_ptr = checked ? checked_eye_ : unchecked_eye_;
+
+        mem_dc.DrawBitmap(*eye_ptr, {FromDIP(10), indicator.y});
+
+        bool reachable = IsLocationReachable(location.ap_location_id);
+        const wxColour* text_color = reachable ? wxWHITE : wxRED;
+        mem_dc.SetTextForeground(*text_color);
+
+        wxSize item_extent = mem_dc.GetTextExtent(location.name);
+        mem_dc.DrawText(
+            location.name,
+            {FromDIP(10 + 32 + 10),
+             indicator.y + (FromDIP(32) - mem_dc.GetFontMetrics().height) / 2});
+
+        break;
       }
-    } else {
-      checked =
-          AP_HasCheckedGameLocation(location.ap_location_id) ||
-          (location.hunt && AP_HasCheckedHuntPanel(location.ap_location_id));
-    }
-
-    const wxBitmap* eye_ptr = checked ? checked_eye_ : unchecked_eye_;
+      case kPAINTING: {
+        const PaintingExit& painting = GD_GetPaintingExit(indicator.id);
 
-    mem_dc.DrawBitmap(*eye_ptr, {FromDIP(10), cur_height});
+        bool reachable = IsPaintingReachable(indicator.id);
+        const wxColour* text_color = reachable ? wxWHITE : wxRED;
+        mem_dc.SetTextForeground(*text_color);
 
-    bool reachable = IsLocationReachable(location.ap_location_id);
-    const wxColour* text_color = reachable ? wxWHITE : wxRED;
-    mem_dc.SetTextForeground(*text_color);
+        bool checked = reachable && AP_IsPaintingChecked(painting.internal_id);
+        const wxBitmap* eye_ptr = checked ? checked_owl_ : unchecked_owl_;
+        mem_dc.DrawBitmap(*eye_ptr, {FromDIP(10), indicator.y});
 
-    wxSize item_extent = mem_dc.GetTextExtent(location.name);
-    mem_dc.DrawText(
-        location.name,
-                    {FromDIP(10 + 32 + 10),
-         cur_height + (FromDIP(32) - mem_dc.GetFontMetrics().height) / 2});
-
-    cur_height += FromDIP(10 + 32);
-  }
+        wxSize item_extent = mem_dc.GetTextExtent(painting.display_name);
+        mem_dc.DrawText(
+            painting.display_name,
+            {FromDIP(10 + 32 + 10),
+             indicator.y + (FromDIP(32) - mem_dc.GetFontMetrics().height) / 2});
 
-  if (AP_IsPaintingShuffle() && !tracker_panel->IsPanelsMode()) {
-    for (int painting_id : map_area.paintings) {
-      if (IsPaintingPostgame(painting_id)) {
-        continue;
+        break;
       }
-
-      const PaintingExit& painting = GD_GetPaintingExit(painting_id);
-
-      bool reachable = IsPaintingReachable(painting_id);
-      const wxColour* text_color = reachable ? wxWHITE : wxRED;
-      mem_dc.SetTextForeground(*text_color);
-
-      bool checked = reachable && AP_IsPaintingChecked(painting.internal_id);
-      const wxBitmap* eye_ptr = checked ? checked_owl_ : unchecked_owl_;
-      mem_dc.DrawBitmap(*eye_ptr, {FromDIP(10), cur_height});
-
-      wxSize item_extent = mem_dc.GetTextExtent(painting.display_name);
-      mem_dc.DrawText(painting.display_name,
-                      {FromDIP(10 + 32 + 10),
-           cur_height + (FromDIP(32) - mem_dc.GetFontMetrics().height) / 2});
-
-      cur_height += FromDIP(10 + 32);
     }
   }
 }
@@ -190,7 +197,7 @@ void AreaPopup::OnPaint(wxPaintEvent& event) {
 
 void AreaPopup::OnDPIChanged(wxDPIChangedEvent& event) {
   LoadIcons();
-  UpdateIndicators();
+  ResetIndicators();
 
   event.Skip();
 }