Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

Merge pull request #2834 from adobe/glenn/help-divider

Remove extra divider from help menu
  • Loading branch information...
commit 722f7ec4a1308bb2a375ba81bfb0309230cbd1d3 2 parents 8910874 + a202845
Peter Flynn peterflynn authored

Showing 1 changed file with 8 additions and 2 deletions. Show diff stats Hide diff stats

  1. +8 2 src/command/Menus.js
10 src/command/Menus.js
@@ -1113,12 +1113,18 @@ define(function (require, exports, module) {
1113 1113 menu.addMenuItem(Commands.HELP_SHOW_EXT_FOLDER);
1114 1114
1115 1115
1116   - menu.addMenuDivider();
  1116 + var hasAboutItem = (brackets.platform !== "mac" || brackets.inBrowser);
  1117 +
  1118 + // Add final divider only if we have a twitter URL or about item
  1119 + if (hasAboutItem || brackets.config.twitter_url) {
  1120 + menu.addMenuDivider();
  1121 + }
  1122 +
1117 1123 if (brackets.config.twitter_url) {
1118 1124 menu.addMenuItem(Commands.HELP_TWITTER);
1119 1125 }
1120 1126 // supress redundant about menu item in mac shell
1121   - if (brackets.platform !== "mac" || brackets.inBrowser) {
  1127 + if (hasAboutItem) {
1122 1128 menu.addMenuItem(Commands.HELP_ABOUT);
1123 1129 }
1124 1130

0 comments on commit 722f7ec

Please sign in to comment.
Something went wrong with that request. Please try again.