Claudine/fix normalization and house cleaning (#3843)

* remove  normalization in  glyph filter

remove glyph normalization gui/lettering/main_panel, as it  breaks the glyph filtering

remove some spelling mistake in the comments, correct some  comment

if a glyph from the  svg font file is a non chasing mark (like combining accent, or cedilla) the inkscape label of the  corresponding glyph  group created by lettering_font_sample  was empty or unreadable, make it  visible . Same in the line label.

and although it is not related
correct one glyph in malika font

* correct warning in  lettering_font_sample

do  the warnings correctly when there is somethng wrong  with the glyph list in the font.json gile

* Update lettering_font_sample.py

fix a typo

1 files changed, 37 insertions, 26 deletions

diff --git a/lib/gui/lettering_font_sample.py b/lib/gui/lettering_font_sample.py
index e19544dc..9f42a755 100644
--- a/lib/gui/lettering_font_sample.py
+++ b/lib/gui/lettering_font_sample.py
@@ -8,6 +8,7 @@ from copy import deepcopy
 import wx
 import wx.adv
 from inkex import Group, errormsg
+import unicodedata
 
 from ..commands import ensure_command_symbols
 from ..i18n import _
@@ -178,29 +179,14 @@ class FontSampleFrame(wx.Frame):
         text = ''
         width = 0
         last_glyph = None
-        printed_warning = False
-        update_glyphlist_warning = _(
-            "The glyphlist for this font seems to be outdated.\n\n"
-            "Please update the glyph list for {font_name}:\n"
-            "* Open Extensions > Ink/Stitch > Font Management > Edit JSON\n"
-            "* Select this font and apply."
-        ).format(font_name=self.font.marked_custom_font_name)
-
-        self.duplicate_warning()
-
-        # font variant glyph list length falls short if a single quote sign is available
-        # let's add it in the length comparison
-        if len(set(self.font.available_glyphs)) != len(self.font_variant.glyphs):
-            errormsg(update_glyphlist_warning)
-            printed_warning = True
+        outdated = False
 
         for glyph in self.font.available_glyphs:
             glyph_obj = self.font_variant[glyph]
             if glyph_obj is None:
-                if not printed_warning:
-                    errormsg(update_glyphlist_warning)
-                printed_warning = True
+                outdated = True
                 continue
+
             if last_glyph is not None:
                 width_to_add = (glyph_obj.min_x - self.font.kerning_pairs.get(f'{last_glyph} {glyph}', 0)) * scale
                 width += width_to_add
@@ -219,8 +205,8 @@ class FontSampleFrame(wx.Frame):
             text += glyph
             width += width_to_add
 
+        self.out_dated_warning(outdated)
         self._render_text(text)
-
         self.GetTopLevelParent().Close()
 
     def sortable(self):
@@ -229,20 +215,40 @@ class FontSampleFrame(wx.Frame):
             color_sort = False
         return color_sort
 
-    def duplicate_warning(self):
-        # warn about duplicated glyphs
+    def out_dated_warning(self, outdated=False):
+        # called with outdated == True when some glyphs present in the font.json glyph list are not present in the svg font file
+
+        update_glyphlist_warning = _(
+            "The glyphlist for this font seems to be outdated.\n\n"
+            "Please update the glyph list for {font_name}:\n"
+            "* Open Extensions > Ink/Stitch > Font Management > Edit JSON\n"
+            "* Select this font and apply."
+        ).format(font_name=self.font.marked_custom_font_name)
+
+        # warning in case of duplicates in the glyph list of the font.json file
         if len(set(self.font.available_glyphs)) != len(self.font.available_glyphs):
-            duplicated_glyphs = " ".join(
-                [glyph for glyph in set(self.font.available_glyphs) if self.font.available_glyphs.count(glyph) > 1]
-            )
-            errormsg(_("Found duplicated glyphs in font file: {duplicated_glyphs}").format(duplicated_glyphs=duplicated_glyphs))
+            outdated = True
+
+        # this will cause a warning if some glyphs of the svg font are not present in the font.json glyph list
+        if len(set(self.font.available_glyphs)) != len(self.font_variant.glyphs):
+            outdated = True
+
+        if outdated:
+            errormsg(update_glyphlist_warning)
 
     def _render_text(self, text):
         lines = text.splitlines()
         position = {'x': 0, 'y': 0}
         for line in lines:
             group = Group()
-            group.label = line
+            label = ""
+            # make the label of the group line clearly show the non spacing marks
+            for character in line:
+                if unicodedata.category(character) != 'Mn':
+                    label += character
+                else:
+                    label += ' ' + character
+            group.label = label
             group.set("inkstitch:letter-group", "line")
             glyphs = []
             skip = []
@@ -297,7 +303,12 @@ class FontSampleFrame(wx.Frame):
         # because this is not unique it will be overwritten by inkscape when inserted into the document
         node.set("id", "glyph")
         node.set("inkstitch:letter-group", "glyph")
+        # force inkscape to show a label when the glyph is only a non spacing mark
+        if len(node.label) == 1 and unicodedata.category(node.label) == 'Mn':
+            node.label = ' ' + node.label
+
         group.add(node)
+
         return position
 
     def cancel(self, event):