Checked all TODOs

Author: timothyschoen

Source/Components/ArrowPopupMenu.h

@@ -48,13 +48,6 @@ class ArrowPopupMenu final : public Component
         if (direction == ArrowDirection::TopBottom) {
             menuToAttachTo->setBounds(menuToAttachTo->getBounds().translated(-15, menuMargin - 3));
         } else {
-            // adjust the popupMenu to be in the correct y position
-            auto const menuBounds = menuToAttachTo->getBounds().reduced(menuMargin + 5);
-            auto const targetBounds = targetComponent->getBounds();
-
-            // TODO: this was totally broken as is for some reason so for now it's hard coded
-            int const yOffset = static_cast<int>(static_cast<float>(targetBounds.getBottom() - menuBounds.getBottom()) / getApproximateScaleFactorForComponent(menuToAttachTo));
-
             menuToAttachTo->setBounds(menuToAttachTo->getBounds().translated(30, -40));
         }
     }

Source/Sidebar/SearchPanel.h

@@ -31,9 +31,6 @@ class OpenInspector final : public Component {
         buttonOpenInspector.setColour(TextButton::buttonOnColourId, backgroundColour.contrasting(0.1f));
         buttonOpenInspector.setColour(ComboBox::outlineColourId, Colours::transparentBlack);
         buttonOpenInspector.setButtonText("Open inspector");
-        // TODO: something is probably wrong with the presentation of this popup overall, but the tooltip seems redundant
-        // buttonOpenInspector.setTooltip("Open inspector for object");
-
         addAndMakeVisible(buttonOpenInspector);
 
         setSize(108, 33);