fix(linewrap): try to prevent edge case by having one char margin (#338)

Sometimes, if we are unlucky, the line wrap (maybe due to whitespace somewhere) fails. So, we add one extra character to have a bit more elbow room.
engeir · Mar 22, 2023 · 2bdf2f0 · 2bdf2f0
1 parent 4b5294d
commit 2bdf2f0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/docs/demo/latest.png b/docs/demo/latest.png
diff --git a/pyproject.toml b/pyproject.toml
@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "ncdump-rich"
-version = "0.3.3"
+version = "0.3.4"
 description = "Rich NcDump"
 authors = ["Eirik Enger <eirroleng@gmail.com>"]
 license = "GPL-3.0"

diff --git a/src/ncdump_rich/ncdump.py b/src/ncdump_rich/ncdump.py
@@ -77,7 +77,7 @@ def print_ncattr(key: str) -> None:
     cprint("[bold white]NetCDF Global Attributes:[/bold white]")
     for nc_attr in nc_attrs:
         if any(
-            len(line) > width - 8
+            len(line) > width - 9
             for line in str(nc_file.getncattr(nc_attr)).splitlines()
         ):
             lineend = (
@@ -94,7 +94,7 @@ def print_ncattr(key: str) -> None:
                         [
                             textwrap.fill(
                                 " ".join(line.split()),
-                                width=width - 8,
+                                width=width - 9,
                                 tabsize=4,
                                 break_long_words=False,
                                 replace_whitespace=False,
@@ -111,7 +111,7 @@ def print_ncattr(key: str) -> None:
                         [
                             textwrap.fill(
                                 " ".join(line.split()),
-                                width=width - 8,
+                                width=width - 9,
                                 tabsize=4,
                                 break_long_words=False,
                                 replace_whitespace=False,
@@ -150,7 +150,7 @@ def print_ncattr(key: str) -> None:
         if len(nc_vars) > 20:
             cprint("\t[italic white]Number of variables: [/italic white]", len(nc_vars))
             cprint("\t[italic white]Variables list: [/italic white]")
-            pp = pprint.PrettyPrinter(width=width - 8, compact=True)
+            pp = pprint.PrettyPrinter(width=width - 9, compact=True)
             cprint(textwrap.indent(pp.pformat(nc_vars), "\t\t"))
         else:
             for var in nc_vars: