Remove trailing context menu separator fixing #5390

spec/context-menu-manager-spec.coffee

@@ -149,6 +149,17 @@ describe "ContextMenuManager", ->
       shouldDisplay = false
       expect(contextMenu.templateForEvent(dispatchedEvent)).toEqual []
 
+    it "prunes a trailing separator", ->
+      contextMenu.add
+        '.grandchild': [
+          {label: 'A', command: 'a'},
+          {type: 'separator'},
+          {label: 'B', command: 'b'},
+          {type: 'separator'}
+        ]
+
+      expect(contextMenu.templateForEvent({target: grandchild}).length).toBe(3)
+
     it "throws an error when the selector is invalid", ->
       addError = null
       try

src/context-menu-manager.coffee

@@ -145,6 +145,10 @@ class ContextMenuManager
 
       currentTarget = currentTarget.parentElement
 
+    # Remove trailing separator
+    if template.length > 0 and template[template.length - 1].type is 'separator'
+      template.splice(template.length-1, 1)
+
     template
 
   # Returns an object compatible with `::add()` or `null`.