Change action name from "Skip All Breakpoints" to "Toggle Skip All Breakpoints"
Change title from "Skip All Breakpoints" to "Toggle Skip All Breakpoints" when toggled
Thanks for the proposal! It makes sense to have all toggle options uniformly prefixed with "toggle" to make the effect easier to understand.
Here are some remarks with respect to this change:
- There are some futher NLSs for the same action that may need to be adapted (see detailed comments).
- With this change, the names of the actions within the
Breakpointsview become inconsistent. There are other toggleable options, such as "Show Breakpoints Supported by Selected Target", which also miss a "Toggle" prefix.- To me "Toggle Skip All Breakpoints" sounds a bit odd, because in contrast to all other "Toggle X" actions, the "X" is not a noun here (like in "Toggle Breakpoint") but a verb-based action description. Unfortunately, I do not have a really good alternative either. "Toggle Skipping All Breakpoints" is probably not much better...
- There seems to be explicit documentation for this action, which probably has to be updated when changing the action name, e.g.: https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/4b5fccff1c7878e02430c22ede26c8788c60896e/eclipse.platform.common/bundles/org.eclipse.jdt.doc.user/reference/views/breakpoints/ref-skipall_viewaction.htm#L8
Understood. If you prefer to keep a consistent pattern with "Toggle X," you might consider using a noun that represents the action of skipping breakpoints. Here is a suggestion:
"Toggle Breakpoint Skipping"
- There seems to be explicit documentation for this action, which probably has to be updated when changing the action name, e.g.: https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/blob/4b5fccff1c7878e02430c22ede26c8788c60896e/eclipse.platform.common/bundles/org.eclipse.jdt.doc.user/reference/views/breakpoints/ref-skipall_viewaction.htm#L8
https://github.com/eclipse-platform/eclipse.platform.releng.aggregator/pull/1559
@HeikoKlare anything left here to do for @ShahzaibIbrahim?
@HeikoKlare anything left here to do for @ShahzaibIbrahim?
@vogella Yes, I have found the existing ticket for this particular issue (See: https://github.com/eclipse-platform/eclipse.platform.swt/issues/501) and the toggle "state/highlight" issue is for windows 11 only. I have one more suggestion in mind which I will propose soon, till then we can draft this PR.
