When Email Ext is used on a project, Email Template Testing appears in its context menu, even though this is a rarely used function that should certainly not be consuming that space. Unfortunately today there seems to be no way to suppress it, since ExtendedEmailPublisher.getProjectActions adds it to the project actions, and lib/hudson/actions.jelly (via Actionable/actions.jelly) calls <l:task> without specifying contextMenu. Perhaps there should be a marker interface for an Action which we would like to hide from the context menu, and actions.jelly would honor that (probably through a helper method in Functions).

          [JENKINS-19173] API to mark an Action to hide from context menu

          Jesse Glick created issue -

          Alex Earl added a comment -

          Would you want it globally disabled, or just for certain projects? I can easily add an option to the global config for email-ext to disable it showing up.

          Alex Earl added a comment - Would you want it globally disabled, or just for certain projects? I can easily add an option to the global config for email-ext to disable it showing up.

          Jesse Glick added a comment -

          I think it should be globally suppressed from the context menu, remaining in the project sidebar. Hence the need for a new API.

          Jesse Glick added a comment - I think it should be globally suppressed from the context menu, remaining in the project sidebar. Hence the need for a new API.

          Alex Earl added a comment -

          Yeah, that makes sense.

          Alex Earl added a comment - Yeah, that makes sense.

          Alex Earl added a comment -

          Not sure if I am the correct assignee for this since it has implications for core first and then email-ext.

          Alex Earl added a comment - Not sure if I am the correct assignee for this since it has implications for core first and then email-ext.
          Jesse Glick made changes -
          Assignee Original: Alex Earl [ slide_o_mix ]

          Alex Earl added a comment -

          Created https://github.com/jenkinsci/jenkins/pull/981, please review before I merge.

          Alex Earl added a comment - Created https://github.com/jenkinsci/jenkins/pull/981 , please review before I merge.
          Jesse Glick made changes -
          Assignee New: Jesse Glick [ jglick ]
          Jesse Glick made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]

          Code changed in jenkins
          User: Jesse Glick
          Path:
          changelog.html
          core/src/main/java/hudson/Functions.java
          core/src/main/java/jenkins/model/ModelObjectWithContextMenu.java
          core/src/main/resources/lib/hudson/actions.jelly
          test/src/test/java/jenkins/model/ContextMenuTest.java
          http://jenkins-ci.org/commit/jenkins/dceac8f0acbf4823a5c0de17f76f41ff130a2ede
          Log:
          [FIXED JENKINS-19173] Added ContextMenuVisibility marker interface for Action’s.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Jesse Glick Path: changelog.html core/src/main/java/hudson/Functions.java core/src/main/java/jenkins/model/ModelObjectWithContextMenu.java core/src/main/resources/lib/hudson/actions.jelly test/src/test/java/jenkins/model/ContextMenuTest.java http://jenkins-ci.org/commit/jenkins/dceac8f0acbf4823a5c0de17f76f41ff130a2ede Log: [FIXED JENKINS-19173] Added ContextMenuVisibility marker interface for Action’s.

            jglick Jesse Glick
            jglick Jesse Glick
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: