Added option to remove all breakpoints#3608
Conversation
|
|
||
| # LOCALIZATION NOTE (breakpointMenuItem.deleteSelf): Access key to remove the | ||
| # currently selected breakpoint from the context menu | ||
| breakpointMenuItem.deleteAll.accesskey=r |
There was a problem hiding this comment.
should this be deleteSelf
| breakpointMenuItem.disableAll=Disable all breakpoints | ||
| breakpointMenuItem.deleteAll=Remove all breakpoints | ||
|
|
||
| # LOCALIZATION NOTE (breakpointMenuItem.deleteSelf): Access key to remove the |
There was a problem hiding this comment.
breakpointMenuItem.deleteSelf.accesskey
|
@sagorika1996 small l10n things, but otherwise looks great |
Codecov Report
@@ Coverage Diff @@
## master #3608 +/- ##
==========================================
- Coverage 54.47% 54.41% -0.06%
==========================================
Files 119 119
Lines 4663 4666 +3
Branches 963 963
==========================================
- Hits 2540 2539 -1
- Misses 2123 2127 +4
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #3608 +/- ##
==========================================
- Coverage 54.47% 54.39% -0.08%
==========================================
Files 119 119
Lines 4663 4666 +3
Branches 963 963
==========================================
- Hits 2540 2538 -2
- Misses 2123 2128 +5
Continue to review full report at Codecov.
|
| ); | ||
|
|
||
| const removeBreakpointKey = L10N.getStr( | ||
| "breakpointMenuItem.deleteAll.accesskey" |
There was a problem hiding this comment.
Should this be deleteSelf?
|
Thanks so much! |
Associated Issue: #3575
Summary of Changes