[Github-comments] [geany/geany] Context menu update (#2550)

Matthew Brush notifications at xxxxx
Sat Jul 18 16:44:27 UTC 2020


@eht16 I was initially going to suggest making this a plugin - being unaware of the addons feature - but:

* It's a very small, simple and common feature that seems to be appropriate as a core feature.
* It would probably require more m4/bash/boilerplate code than [the entirety of the patch](https://patch-diff.githubusercontent.com/raw/geany/geany/pull/2550.diff).
* The size of this context menu is already effectively unbounded.
* Accessing this menu from the plugin API would be kludgy or require extending the API (as you pointed out).

That said, I'm indifferent. I won't merge unless there is consensus.



-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2550#issuecomment-660508980
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20200718/e0508733/attachment.htm>


More information about the Github-comments mailing list