fix: change the position of the annotation

Author: jiji-hoon96

packages/table-core/src/features/RowSorting.ts

@@ -326,13 +326,10 @@ export const RowSorting: TableFeature = {
     column: Column<TData, TValue>,
     table: Table<TData>
   ): void => {
-    // Add deprecation warning for sortUndefined
     if (column.columnDef.sortUndefined && !column.columnDef.sortEmpty) {
       if (process.env.NODE_ENV !== 'production') {
         console.warn(
-          '[TanStack Table] sortUndefined is deprecated. ' +
-            'Please use sortEmpty instead for more comprehensive empty value handling. ' +
-            `Column: ${column.id}`
+          `[TanStack Table] sortUndefined is deprecated. Please use sortEmpty instead for more comprehensive empty value handling. Column: ${column.id}`
         )
       }
     }

packages/table-core/src/utils/getSortedRowModel.ts

@@ -62,7 +62,6 @@ export function getSortedRowModel<TData extends RowData>(): (
 
               let sortInt = 0
 
-              // Handle sortEmpty option first (takes precedence over sortUndefined)
               if (sortEmpty) {
                 const aValue = rowA.getValue(sortEntry.id)
                 const bValue = rowB.getValue(sortEntry.id)
@@ -75,11 +74,8 @@ export function getSortedRowModel<TData extends RowData>(): (
                 }
 
                 if (aIsEmpty || bIsEmpty) {
-                  // Determine position based on sortEmpty setting
                   const emptyPosition = sortEmpty === 'last' ? 1 : -1
                   sortInt = aIsEmpty ? emptyPosition : -emptyPosition
-                  
-                  // Return immediately - empty values maintain their position regardless of desc
                   return sortInt
                 }
               }

packages/table-core/tests/sortEmpty.test.ts

@@ -223,7 +223,6 @@ describe('sortEmpty option', () => {
       columnHelper.accessor('price', {
         header: 'Price',
         sortEmpty: 'last',
-        // No custom isEmptyValue - should use default
       }),
     ]