Removed HIG typography hints from comments

This was a misguided attempt to encourage following the GNOME HIG
typography guidelines in translations
(https://developer.gnome.org/hig/guidelines/typography.html#take-advantage-of-unicode). For example, with some strings I'd have a message about using a U+2026 ellipsis (…) over three dots (...), or using a U+2014 em dash (—) over the minus sign (-). But different languages follow different conventions anyway, so providing Unicode hints from guidelines written for US English is a bit ignorant on my part.

Author: Wartybix

src/attempt_fail_box.py

@@ -75,8 +75,6 @@ def __init__(
         # {vid_br_unit} represents a bitrate unit, like 'kbps'.
         # {extra_details} will either display resolution/FPS and audio quality
         # details, or just resolution/FPS if the video has no audio streams.
-        # Please use U+202F Narrow no-break space (' ') between video bitrate
-        # and unit.
         target_str = _("{vid_br} {vid_br_unit} ({extra_details})").format(
             vid_br = f'{str(vid_bitrate // 1000)}',
             # TRANSLATORS: This is the SI unit for 'kilobits per second'.
@@ -90,8 +88,7 @@ def __init__(
         if compressed_size_bytes >= target_size_bytes:
             self.failure_icon.set_from_icon_name('arrow2-up-symbolic')
             # TRANSLATORS: {size} represents an integer. {unit} represents a
-            # file size unit like 'MB'. Please use U+202F Narrow no-break space
-            # (' ') between size and unit.
+            # file size unit like 'MiB'.
             fail_msg = _('Compressed file size was too large ({size} {unit})').format(
                 size = f'{compressed_size_mb}',
                 # TRANSLATORS: this is the SI unit for 'mebibyte'.
@@ -101,8 +98,7 @@ def __init__(
         else:
             self.failure_icon.set_from_icon_name('arrow2-down-symbolic')
             # TRANSLATORS: {size} represents an integer. {unit} represents a
-            # file size unit like 'MB'. Please use U+202F Narrow no-break space
-            # (' ') between size and unit.
+            # file size unit like 'MiB'.
             fail_msg = _('Compressed file size was too small ({size} {unit})').format(
                 size = f'{compressed_size_mb}',
                 # TRANSLATORS: this is the SI unit for 'mebibyte'.

src/current_attempt_box.py

@@ -89,8 +89,6 @@ def set_attempt_details(
         # {vid_br_unit} represents a bitrate unit, like 'kbps'.
         # {extra_details} will either display resolution/FPS and audio quality
         # details, or just resolution/FPS if the video has no audio streams.
-        # Please use U+202F Narrow no-break space (' ') between video bitrate
-        # and unit.
         target_str = _('Compressing to {vid_br} {vid_br_unit} ({extra_details})').format(
             vid_br = f'{vid_bitrate // 1000}',
             # TRANSLATORS: This is the SI unit for 'kilobits per second'.
@@ -121,15 +119,12 @@ def set_progress(
         # https://gitlab.gnome.org/GNOME/nautilus/-/blob/af7e419eaa7e167ecbc059d51e06e72e11a2f1c8/src/nautilus-file-operations.c
 
         # TRANSLATORS: {} represents the progress percentage value.
-        # Please use U+202F Narrow no-break space (' ') between {} and %, if
-        # applicable to your language.
         percentage_text = _('{} %').format(progress_percent)
 
         if seconds_left is None:
             progress_text = percentage_text
         elif seconds_left == -1:
             # TRANSLATORS: {} represents the progress percentage.
-            # Please use U+2014 em dash ('—'), if applicable to your language.
             progress_text = _('{} — Almost done').format(percentage_text)
         else:
             time_shown = ''
@@ -139,7 +134,6 @@ def set_progress(
                 # Used as part of a larger string, like:
                 # '5% -- About 30 seconds left'
                 time_shown = ngettext('{} second', '{} seconds', seconds_left).format(seconds_left)
-                # time_shown.format(seconds_left)
             elif seconds_left < (60 * 60):
                 minutes = seconds_left // 60
 
@@ -174,7 +168,6 @@ def set_progress(
             # TRANSLATORS: {percentage} represents the progress percentage.
             # {time_shown} represents a string showing the estimated time to
             # completion (like '50 minutes').
-            # Please use U+2014 em dash ('—'), if applicable to your language.
             progress_text = _('{percentage} — About {time_shown} left').format(
                 percentage = percentage_text,
                 time_shown = time_shown

src/error_dialog.py

@@ -43,8 +43,7 @@ def __init__(
 
         self.preference_page.set_description(
             # TRANSLATORS: {} represents the filename of the video with the
-            # error. Please use “” instead of "", if applicable to your
-            # language.
+            # error.
             _('There was a problem compressing “{}”').format(safe_video_name)
         )
         buffer = self.text_view.get_buffer()

src/main.py

@@ -181,7 +181,6 @@ def on_about_action(self, *args: Any) -> None:
 
         about.add_acknowledgement_section(
             # TRANSLATORS: Braces represent the name of the repository (e.g. 8mb)
-            # Please use ‘’ characters instead of '', if applicable to your language.
             _('‘{}’ repository by').format('8mb'),
             [
                 'Matthew Baggett https://github.com/matthewbaggett/',

src/sources_row.py

@@ -491,8 +491,7 @@ def set_complete(
         update_ui(
             self.complete_label.set_label,
             # TRANSLATORS: {size} represents a file size value in MB.
-            # {unit} represents a file size unit, like 'MB'. Please use U+202F
-            # narrow no-break space (' ') between size and unit.
+            # {unit} represents a file size unit, like 'MiB'.
             _('Video compressed to {size} {unit}.').format(
                 size = compressed_size_mb,
                 # TRANSLATORS: this is the SI unit for 'mebibyte'.
@@ -520,8 +519,7 @@ def refresh_state(
             self.set_warning(
                 # TRANSLATORS: {original_size} and {target_size} represent
                 # integers. {unit_original} and {unit_target} represent file
-                # size units, like 'MB'. Please use U+202F Narrow no-break
-                # space (' ') between values and units.
+                # size units, like 'MB'.
                 _('Video file size ({original_size} {unit_original}) already meets the target size ({target_size} {unit_target}).')
                     .format(
                         original_size = size_mb,
@@ -542,8 +540,6 @@ def refresh_state(
             self.set_incompatible(
                 # TRANSLATORS: {size} represents an integer. {unit} represents
                 # a file size unit like 'MB'.
-                # Please use U+202F Narrow no-break space (' ') between value
-                # and unit.
                 _('Target size ({size} {unit}) is too low for this file.')
                     # TRANSLATORS: this is the SI unit for 'mebibyte'.
                     .format(size = target_size, unit = _('MiB')),

src/window.blp

@@ -75,7 +75,6 @@ template $ConstrictWindow: Adw.ApplicationWindow {
                 title: _("Compress Videos");
                 description: _("Drag and drop videos here");
                 child: Button {
-                  // TRANSLATORS: please use U+2026 Horizontal ellipsis (…) instead of '...', if applicable to your language
                   label: _("_Open…");
                   use-underline: true;
                   action-name: "win.open";
@@ -272,7 +271,6 @@ template $ConstrictWindow: Adw.ApplicationWindow {
               orientation: vertical;
 
               Adw.Banner warning_banner {
-                // TRANSLATORS: Please use U+2014 em dash ('—') instead of '-', if applicable to your language.
                 title: _("Issues with video sources — fix before compressing");
               }
 
@@ -324,7 +322,6 @@ template $ConstrictWindow: Adw.ApplicationWindow {
             ActionBar export_bar {
               [center]
               Button export_button {
-                // TRANSLATORS: please use U+2026 Horizontal ellipsis (…) instead of '...', if applicable to your language
                 label: _("_Export To…");
                 use-underline: true;
                 action-name: "win.export";
@@ -341,7 +338,6 @@ template $ConstrictWindow: Adw.ApplicationWindow {
 
               [center]
               Button cancel_button {
-                // TRANSLATORS: please use U+2026 Horizontal ellipsis (…) instead of '...', if applicable to your language
                 label: _("_Cancel…");
                 use-underline: true;
                 action-name: "win.cancel";

src/window.py

@@ -221,25 +221,22 @@ def __init__(self, **kwargs: Any) -> None:
 
         # TRANSLATORS: 'FPS' meaning 'frames per second'.
         # {} represents the FPS value, for example 30 or 60.
-        # Please use U+202F Narrow no-break space (' ') between value and unit.
         fps_label = _('{} FPS')
 
         self.clear_row.set_title(fps_label.format('_30'))
         self.smooth_row.set_title(fps_label.format('_60'))
 
         self.fps_help_label.set_label(
             # TRANSLATORS: FPS meaning 'frames per second'. {} represents an
-            # integer. Please use U+202F narrow no-break space (' ') between
-            # the {} and translated equivalent of 'FPS'.
+            # integer.
             _('Videos compressed to low bitrates may be capped to {} FPS, regardless of the option set.')
                 .format('24')
         )
 
         default_tolerance = self.settings.get_default_value('tolerance')
 
         self.adv_options_help_label.set_label(
-            # TRANSLATORS: {} represents an integer. Please use U+202F Narrow
-            # no-break space (' ') between {} and '%'.
+            # TRANSLATORS: {} represents an integer.
             # xgettext:no-python-format
             _('Decreasing the tolerance maximizes image quality by reducing how much compressed file sizes can be under target. However, this can increase the number of attempts needed to meet the target, increasing compression time. A tolerance of {} % or more is recommended.')
                 .format(default_tolerance)
@@ -364,8 +361,7 @@ def set_compressing_title(self, current_index: int, export_dir: str):
         if len(sources) == 1:
             file_name = sources[0].display_name
             # TRANSLATORS: {} represents the filename of the video currently
-            # being processed. Please use “” instead of "", if applicable to
-            # your language.
+            # being processed.
             self.set_title(_('Processing “{}”').format(file_name))
         else:
             self.set_title(
@@ -396,8 +392,7 @@ def set_queued_title(self, daemon: bool) -> None:
         elif len(sources) == 1:
             vid_name = sources[0].display_name
             # TRANSLATORS: {} represents the filename of the video currently
-            # queued. Please use “” instead of '', if applicable to your
-            # language.
+            # queued.
             self.set_title(_('“{}” Queued').format(vid_name))
         else:
             vid_count = len(sources)
@@ -537,8 +532,7 @@ def show_cancel_dialog(self, quit_on_stop: bool) -> None:
         dialog = Adw.AlertDialog.new(
             _('Stop Compression?'),
             # TRANSLATORS: {} represents the filename of the video currently
-            # being compressed. Please use “” instead of "", if applicable to
-            # your language.
+            # being compressed.
             _('Progress made compressing “{}” will be permanently lost')
                 .format(self.currently_processed)
         )
@@ -607,7 +601,6 @@ def send_complete_notification(
             video_name = sources_list[0].display_name
             # TRANSLATORS: {} represents the filename of the video that has
             # been processed.
-            # Please use “” instead of "", if applicable to your language.
             notification.set_body(_('“{}” processed').format(video_name))
         else:
             notification.set_body(
@@ -724,8 +717,6 @@ def bulk_compress(self, destination_dir: str) -> None:
 
             def update_progress(fraction, seconds_left):
                 if fraction == 0.0 and codec == VideoCodec.VP9:
-                    # TRANSLATORS: please use U+2026 Horizontal ellipsis (…)
-                    # instead of '...', if applicable to your language
                     progress_box.set_progress_text(_('Analyzing…'), daemon)
                     video.enable_spinner(True, daemon)
                     progress_box.pulse_progress(daemon)
@@ -820,8 +811,7 @@ def trash_video():
 
                 toast = Adw.Toast.new(
                     # TRANSLATORS: {} represents the filename of the video with
-                    # the error. Please use “” instead of "", if applicable to
-                    # your language.
+                    # the error.
                     _('Error compressing “{}”').format(video.display_name)
                 )
                 toast.set_use_markup(False)