Group Breakpoints Menu #4093
Group Breakpoints Menu #4093
Conversation
assets/panel/debugger.properties
Outdated
breakpointMenuItem.removeCondition.accesskey=c | ||
breakpointMenuItem.editCondition.label=Edit breakpoint condition | ||
breakpointMenuItem.editCondition.accesskey=n | ||
breakpointMenuItem.enableSelf2=Enable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
since we're doing this change. can we switch to use .label
assets/panel/debugger.properties
Outdated
breakpointMenuItem.enableAll2=Enable all | ||
breakpointMenuItem.enableAll2.accesskey=b | ||
breakpointMenuItem.disableAll2=Disable all | ||
breakpointMenuItem.disableAll2.accesskey=k |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no other letter available?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there are a lot of other items in the context menu so it is tough to come up w/ one
It may be better to disable menu items than hide them if it creates a situation where a separator is the first item in the context menu. |
Oh nice, that's where it's coming from! |
1a22d48
to
1adb720
Compare
that's okay, because we haven't finished "add condition". Once that's in there will always be a conditional item |
Summary of Changes
Old
New