zvirja / traygarden Goto Github PK

Copy large value to clipboard (e.g. 100MB)

Result: TrayGarden starts to consume a lot of memory and CPU

Solution:
Value limit should be introduced.

Currently all the context menu entries in global menu have the same color.
But very often it might happen that some entries are irrelevant at this moment (e.g. it makes no sense to process url shortening if clipboard doesn't contain url). It would be nice to highlight them in another colors.

Add a way to specify subtypes. This may help to show "Select folder" dialog to choose value.

Currently in the TrayGarden.Services.PlantServices.GlobalMenu.Core.ContextMenuCollecting.MenuEntriesAppender there is such code:

public virtual void AppentMenuStripItem(string text, Icon icon, EventHandler clickHandler)
{
  if (text.IsNullOrEmpty() || icon == null || clickHandler == null)
    return;
  EventHandler asyncClickHandler = (sender, args) => Task.Factory.StartNew(() => clickHandler(sender, args));
  var menuItem = new ToolStripMenuItem(text, icon.ToBitmap(), asyncClickHandler);
  OutputItems.Add(menuItem);
}

So it's impossible to add a new context menu item without an icon. Although, it's possible to construct a ToolStripMenuItem without an icon by passing null as image.

Sometimes the configuration is corrupted.
Should introduce some extra logging to investigate this.

Add check to prevent second copy from running

Sometimes clipboard events are duplicated and handlers processes them a few time.
We should add some protection.
This might cause wrong results.