fix(tile-manager): remove rowCount prop

Author: onlyexeption

src/components/tile-manager/tile-manager.spec.ts

@@ -64,7 +64,6 @@ describe('Tile Manager component', () => {
     it('is correctly initialized with its default component state', () => {
       // TODO: Add checks for other settings when implemented
       expect(tileManager.columnCount).to.equal(0);
-      expect(tileManager.rowCount).to.equal(0);
       expect(tileManager.dragMode).to.equal('slide');
     });
 
@@ -186,19 +185,14 @@ describe('Tile Manager component', () => {
       expect(style.gridTemplateColumns).to.equal(
         'repeat(auto-fit, minmax(20px, 1fr))'
       );
-      expect(style.gridTemplateRows).to.equal(
-        'repeat(auto-fit, minmax(20px, 1fr))'
-      );
 
       tileManager.columnCount = 15;
-      tileManager.rowCount = 15;
 
       await elementUpdated(tileManager);
 
       style = getTileManagerBase().style;
 
       expect(style.gridTemplateColumns).to.equal('repeat(15, auto)');
-      expect(style.gridTemplateRows).to.equal('repeat(15, auto)');
     });
 
     it('should respect tile row and col start properties', async () => {

src/components/tile-manager/tile-manager.ts

@@ -83,11 +83,7 @@ export default class IgcTileManagerComponent extends EventEmitterMixin<
   public dragMode: 'slide' | 'swap' = 'slide';
 
   @property({ type: Number })
-  public columnCount = 0;
-
-  // remove this one
-  @property({ type: Number })
-  public rowCount = 0;
+  public columnCount = 10;
 
   /**
    * Gets the tiles sorted by their position in the layout.
@@ -100,21 +96,14 @@ export default class IgcTileManagerComponent extends EventEmitterMixin<
   }
 
   @watch('columnCount', { waitUntilFirstUpdate: true })
-  @watch('rowCount', { waitUntilFirstUpdate: true })
   protected updateRowsCols() {
     // REVIEW: Bind to internal CSS vars/parts or something
     // const gridTemplateColumns =
     //   this.columnCount > 0
     //     ? `repeat(${this.columnCount}, auto)`
     //     : 'repeat(auto-fit, minmax(20px, 1fr))';
-    // const gridTemplateRows =
-    //   this.rowCount > 0
-    //     ? `repeat(${this.rowCount}, auto)`
-    //     : 'repeat(auto-fit, minmax(20px, 1fr))';
-    // Object.assign(this._baseWrapper.style, {
-    //   gridTemplateColumns,
-    //   gridTemplateRows,
-    // });
+    //
+    // Object.assign(this._baseWrapper.style, gridTemplateColumns);
   }
 
   constructor() {

stories/tile-manager.stories.ts

@@ -39,15 +39,10 @@ const metadata: Meta<IgcTileManagerComponent> = {
     columnCount: {
       type: 'number',
       control: 'number',
-      table: { defaultValue: { summary: '0' } },
-    },
-    rowCount: {
-      type: 'number',
-      control: 'number',
-      table: { defaultValue: { summary: '0' } },
+      table: { defaultValue: { summary: '10' } },
     },
   },
-  args: { dragMode: 'slide', columnCount: 0, rowCount: 0 },
+  args: { dragMode: 'slide', columnCount: 10 },
 };
 
 export default metadata;
@@ -56,7 +51,6 @@ interface IgcTileManagerArgs {
   /** Determines whether the tiles slide or swap on drop. */
   dragMode: 'slide' | 'swap';
   columnCount: number;
-  rowCount: number;
 }
 type Story = StoryObj<IgcTileManagerArgs>;
 
@@ -130,7 +124,6 @@ export const Default: Story = {
     <igc-tile-manager
       .dragMode=${args.dragMode}
       .columnCount=${args.columnCount}
-      .rowCount=${args.rowCount}
     >
       <p>This text won't be displayed in Tile Manager</p>
       ${tiles}