rich_to_pt_style now also supports blink, reverse, and conceal/hidden styles

rich_to_pt_style should now support all styles in common between rich and prompt-toolkit

Author: tleonhardt

cmd2/pt_utils.py

@@ -112,6 +112,14 @@ def rich_to_pt_style(rich_style: StyleType) -> str:
         parts.append("italic" if rich_style.italic else "noitalic")
     if rich_style.underline is not None:
         parts.append("underline" if rich_style.underline else "nounderline")
+    if rich_style.blink is not None:
+        parts.append("blink" if rich_style.blink else "noblink")
+    if rich_style.reverse is not None:
+        # prompt-toolkit uses 'reverse'
+        parts.append("reverse" if rich_style.reverse else "noreverse")
+    if rich_style.conceal is not None:
+        # prompt-toolkit uses 'hidden' for Rich's 'conceal'
+        parts.append("hidden" if rich_style.conceal else "nohidden")
     return " ".join(parts)
 
 

tests/test_pt_utils.py

@@ -758,3 +758,49 @@ def test_rich_to_pt_style_nounderline(self):
         style = Style(underline=False)
         pt_style = pt_utils.rich_to_pt_style(style)
         assert "nounderline" in pt_style
+
+    def test_rich_to_pt_style_blink(self):
+        from rich.style import Style
+
+        style = Style(blink=True)
+        pt_style = pt_utils.rich_to_pt_style(style)
+        assert "blink" in pt_style
+
+    def test_rich_to_pt_style_noblink(self):
+        from rich.style import Style
+
+        style = Style(blink=False)
+        pt_style = pt_utils.rich_to_pt_style(style)
+        assert "noblink" in pt_style
+
+    def test_rich_to_pt_style_reverse(self):
+        from rich.style import Style
+
+        style = Style(reverse=True)
+        pt_style = pt_utils.rich_to_pt_style(style)
+        # Note: reverse replaces the default 'noreverse' that is added at the start of parts
+        # wait, we'll check how it works exactly. It will append "reverse". So we just assert "reverse" in pt_style
+        # actually, if reverse=True, "reverse" will be appended to the list, while "noreverse" is also at index 0.
+        # Let's just check the last appended one.
+        assert "reverse" in pt_style.split()
+
+    def test_rich_to_pt_style_noreverse(self):
+        from rich.style import Style
+
+        style = Style(reverse=False)
+        pt_style = pt_utils.rich_to_pt_style(style)
+        assert "noreverse" in pt_style
+
+    def test_rich_to_pt_style_hidden_conceal(self):
+        from rich.style import Style
+
+        style = Style(conceal=True)
+        pt_style = pt_utils.rich_to_pt_style(style)
+        assert "hidden" in pt_style
+
+    def test_rich_to_pt_style_nohidden_conceal(self):
+        from rich.style import Style
+
+        style = Style(conceal=False)
+        pt_style = pt_utils.rich_to_pt_style(style)
+        assert "nohidden" in pt_style