"clear" not working about maccy

[thewisenerd]

this "works", not sure if correct though:

in Menu.swift

   func clear() {
+    for item in historyItems {
+        removeItem(item)
+    }
     historyItems.removeAll()
   }

[thewisenerd]

also, I noticed that we have one NSMenu and historyItems is inserted into it; if we want to implement #26 or #40, we should ideally have history as a separate "submenu" so we can call NSMenu.removeAll()?
disclaimer: I am not aware of Swift or AppKit conventions, so, 🧂

[p0deje]

Thanks for the bug report and the proposed fix!

also, I noticed that we have one NSMenu and historyItems is inserted into it; if we want to implement #26 or #40, we should ideally have history as a separate "submenu" so we can call NSMenu.removeAll()?

I am not sure if submenu works in this case as it might be shown as a single item that expands to submenu items when hovered (e.g. on screenshot)

Otherwise, If that's not true, I think it's a great idea that will simplify things so much.