style(display): rename None back to All in max columns dropdown

Author: shuoweil

bigframes/display/table_widget.js

@@ -115,12 +115,12 @@ function render({ model, el }) {
 
   maxColumnsLabel.textContent = 'Max columns:';
 
-  // 0 represents "None" (all columns)
+  // 0 represents "All" (all columns)
   const maxColumnOptions = [5, 10, 15, 20, 0];
   for (const cols of maxColumnOptions) {
     const option = document.createElement('option');
     option.value = cols;
-    option.textContent = cols === 0 ? 'None' : cols;
+    option.textContent = cols === 0 ? 'All' : cols;
 
     const currentMax = model.get(ModelProperty.MAX_COLUMNS);
     // Handle None/null from python as 0/All

tests/js/table_widget.test.js

@@ -444,7 +444,7 @@ describe('TableWidget', () => {
       expect(Number(select.value)).toBe(initialMaxColumns);
     });
 
-    it('should handle None/null initial value as 0 (None)', () => {
+    it('should handle None/null initial value as 0 (All)', () => {
       model.get.mockImplementation((property) => {
         if (property === 'max_columns') {
           return null; // Python None is null in JS
@@ -456,7 +456,7 @@ describe('TableWidget', () => {
 
       const select = el.querySelector('.max-columns select');
       expect(Number(select.value)).toBe(0);
-      expect(select.options[select.selectedIndex].textContent).toBe('None');
+      expect(select.options[select.selectedIndex].textContent).toBe('All');
     });
 
     it('should update model when value changes', () => {