Strip empty list values (eg empty rows) while modfying objects.

Author: Zerline

sage_widget_adapters/combinat/tableau_grid_view_adapter.py

@@ -188,6 +188,7 @@ def remove_cell(self, obj, pos, dirty={}):
         tl[pos[0]].pop()
         if not tl[pos[0]]:
             tl.pop()
+        tl = [r for r in tl if r] # do not keep any empty row before the test
         return self._validate(tl)
 
 class SemistandardTableauGridViewAdapter(TableauGridViewAdapter):

sage_widget_adapters/generic_grid_view_adapter.py

@@ -194,7 +194,7 @@ def get_cell(obj, pos):
     def make_dirty(self, l, dirty={}):
         r"""
         Append 'dirty' values to list 'l'.
-        Return a list.
+        Return a list with no empty values.
 
         TESTS ::
 
@@ -205,6 +205,8 @@ def make_dirty(self, l, dirty={}):
             sage: ga.cellzero = 0
             sage: ga.make_dirty(t.to_list(), {(1,2):42})
             [[1, 2, 5, 6], [3, 7, 42], [4]]
+            sage: ga.make_dirty(t.to_list(), {(2,0):0})
+            [[1, 2, 5, 6], [3, 7]]
         """
         for p in dirty:
             if p[0] < len(l):
@@ -219,7 +221,7 @@ def make_dirty(self, l, dirty={}):
                 for i in range(p[0] - len(l)):
                     l.append([])
                 l.append([dirty[p]])
-        return l
+        return [val for val in l if val]
 
     def set_cell(self, obj, pos, val, dirty={}, constructorname=''):
         r"""