Remove keybinding full stops (napari#7304)

# References and relevant issues

Closes napari#7245 

# Description

Removes full stops from the end of action names used for shortcuts.
Improves name consistency and readability of error messages.

napari/components/_viewer_key_bindings.py

@@ -47,7 +47,7 @@ def extend_selection_to_layer_below(viewer: Viewer):
     viewer.layers.select_previous(shift=True)
 
 
-@register_viewer_action(trans._('Reset scroll.'))
+@register_viewer_action(trans._('Reset scroll'))
 def reset_scroll_progress(viewer: Viewer):
     # on key press
     viewer.dims._scroll_progress = 0
@@ -60,7 +60,7 @@ def reset_scroll_progress(viewer: Viewer):
 reset_scroll_progress.__doc__ = trans._('Reset dims scroll progress')
 
 
-@register_viewer_action(trans._('Toggle 2D/3D view.'))
+@register_viewer_action(trans._('Toggle 2D/3D view'))
 def toggle_ndisplay(viewer: Viewer):
     if viewer.dims.ndisplay == 2:
         viewer.dims.ndisplay = 3
@@ -73,7 +73,7 @@ def toggle_ndisplay(viewer: Viewer):
 # ```
 # RuntimeError: wrapped C/C++ object of type CanvasBackendDesktop has been deleted
 # ```
-@register_viewer_action(trans._('Toggle current viewer theme.'))
+@register_viewer_action(trans._('Toggle current viewer theme'))
 def toggle_theme(viewer: ViewerModel):
     """Toggle theme for current viewer"""
     themes = available_themes()
@@ -90,44 +90,44 @@ def toggle_theme(viewer: ViewerModel):
     viewer.theme = themes[idx]
 
 
-@register_viewer_action(trans._('Reset view to original state.'))
+@register_viewer_action(trans._('Reset view to original state'))
 def reset_view(viewer: Viewer):
     viewer.reset_view()
 
 
-@register_viewer_action(trans._('Delete selected layers.'))
+@register_viewer_action(trans._('Delete selected layers'))
 def delete_selected_layers(viewer: Viewer):
     viewer.layers.remove_selected()
 
 
 @register_viewer_action(
-    trans._('Increment dimensions slider to the left.'), repeatable=True
+    trans._('Increment dimensions slider to the left'), repeatable=True
 )
 def increment_dims_left(viewer: Viewer):
     viewer.dims._increment_dims_left()
 
 
 @register_viewer_action(
-    trans._('Increment dimensions slider to the right.'), repeatable=True
+    trans._('Increment dimensions slider to the right'), repeatable=True
 )
 def increment_dims_right(viewer: Viewer):
     viewer.dims._increment_dims_right()
 
 
-@register_viewer_action(trans._('Move focus of dimensions slider up.'))
+@register_viewer_action(trans._('Move focus of dimensions slider up'))
 def focus_axes_up(viewer: Viewer):
     viewer.dims._focus_up()
 
 
-@register_viewer_action(trans._('Move focus of dimensions slider down.'))
+@register_viewer_action(trans._('Move focus of dimensions slider down'))
 def focus_axes_down(viewer: Viewer):
     viewer.dims._focus_down()
 
 
 # Use non-breaking spaces and non-breaking hyphen for Preferences table
 @register_viewer_action(
     trans._(
-        'Change order of the visible axes, e.g.\u00a0[0,\u00a01,\u00a02]\u00a0\u2011>\u00a0[2,\u00a00,\u00a01].'
+        'Change order of the visible axes, e.g.\u00a0[0,\u00a01,\u00a02]\u00a0\u2011>\u00a0[2,\u00a00,\u00a01]'
     ),
 )
 def roll_axes(viewer: Viewer):
@@ -137,17 +137,17 @@ def roll_axes(viewer: Viewer):
 # Use non-breaking spaces and non-breaking hyphen for Preferences table
 @register_viewer_action(
     trans._(
-        'Transpose order of the last two visible axes, e.g.\u00a0[0,\u00a01]\u00a0\u2011>\u00a0[1,\u00a00].'
+        'Transpose order of the last two visible axes, e.g.\u00a0[0,\u00a01]\u00a0\u2011>\u00a0[1,\u00a00]'
     ),
 )
 def transpose_axes(viewer: Viewer):
     viewer.dims.transpose()
 
 
-@register_viewer_action(trans._('Rotate layers 90 degrees counter-clockwise.'))
+@register_viewer_action(trans._('Rotate layers 90 degrees counter-clockwise'))
 def rotate_layers(viewer: Viewer):
     if viewer.dims.ndisplay == 3:
-        show_info(trans._('Rotating layers only works in 2D.'))
+        show_info(trans._('Rotating layers only works in 2D'))
         return
     for layer in viewer.layers:
         if layer.ndim == 2:
@@ -180,7 +180,7 @@ def rotate_layers(viewer: Viewer):
         layer.affine = layer.affine.replace_slice(visible_dims, new_affine)
 
 
-@register_viewer_action(trans._('Toggle grid mode.'))
+@register_viewer_action(trans._('Toggle grid mode'))
 def toggle_grid(viewer: Viewer):
     viewer.grid.enabled = not viewer.grid.enabled
 
@@ -197,13 +197,13 @@ def toggle_unselected_visibility(viewer: Viewer):
             layer.visible = not layer.visible
 
 
-@register_viewer_action(trans._('Select and show only layer above.'))
+@register_viewer_action(trans._('Select and show only layer above'))
 def show_only_layer_above(viewer):
     viewer.layers.select_next()
     _show_only_selected_layer(viewer)
 
 
-@register_viewer_action(trans._('Select and show only layer below.'))
+@register_viewer_action(trans._('Select and show only layer below'))
 def show_only_layer_below(viewer):
     viewer.layers.select_previous()
     _show_only_selected_layer(viewer)

napari/layers/labels/_labels_key_bindings.py

@@ -72,7 +72,7 @@ def activate_labels_erase_mode(layer: Labels):
 
 @register_label_action(
     trans._(
-        'Set the currently selected label to the largest used label plus one.'
+        'Set the currently selected label to the largest used label plus one'
     ),
 )
 def new_label(layer: Labels):
@@ -97,29 +97,29 @@ def new_label(layer: Labels):
 
 
 @register_label_action(
-    trans._('Swap between the selected label and the background label.'),
+    trans._('Swap between the selected label and the background label'),
 )
 def swap_selected_and_background_labels(layer: Labels):
     """Swap between the selected label and the background label."""
     layer.swap_selected_and_background_labels()
 
 
 @register_label_action(
-    trans._('Decrease the currently selected label by one.'),
+    trans._('Decrease the currently selected label by one'),
 )
 def decrease_label_id(layer: Labels):
     layer.selected_label -= 1
 
 
 @register_label_action(
-    trans._('Increase the currently selected label by one.'),
+    trans._('Increase the currently selected label by one'),
 )
 def increase_label_id(layer: Labels):
     layer.selected_label += 1
 
 
 @register_label_action(
-    trans._('Decrease the paint brush size by one.'),
+    trans._('Decrease the paint brush size by one'),
     repeatable=True,
 )
 def decrease_brush_size(layer: Labels):
@@ -132,7 +132,7 @@ def decrease_brush_size(layer: Labels):
 
 
 @register_label_action(
-    trans._('Increase the paint brush size by one.'),
+    trans._('Increase the paint brush size by one'),
     repeatable=True,
 )
 def increase_brush_size(layer: Labels):

napari/layers/points/_points_key_bindings.py

@@ -67,7 +67,7 @@ def paste(layer: Points) -> None:
 
 
 @register_points_action(
-    trans._('Select/Deselect all points in the current view slice.'),
+    trans._('Select/Deselect all points in the current view slice'),
 )
 def select_all_in_slice(layer: Points) -> None:
     new_selected = set(layer._indices_view[: len(layer._view_data)])
@@ -97,7 +97,7 @@ def select_all_in_slice(layer: Points) -> None:
 
 
 @register_points_action(
-    trans._('Select/Deselect all points in the layer.'),
+    trans._('Select/Deselect all points in the layer'),
 )
 def select_all_data(layer: Points) -> None:
     # If all points are already selected, deselect all points

napari/layers/shapes/_shapes_key_bindings.py

@@ -180,7 +180,7 @@ def move_shapes_selection_to_back(layer: Shapes) -> None:
 
 @register_shapes_action(
     trans._(
-        'Finish any drawing, for example when using the path or polygon tool.'
+        'Finish any drawing, for example when using the path or polygon tool'
     ),
 )
 def finish_drawing_shape(layer: Shapes) -> None: