Address PR review feedback: improve theming, i18n, and accessibility

- Remove deprecated theme="legacy" prop from AG Grid
- Use browser locale for currency formatting instead of hardcoded en-US
- Add aria-label to jump-to-page input for accessibility

Co-authored-by: hotlong <[email protected]>

Author: Copilot

packages/ui/src/components/grids/AdvancedDataGrid.tsx

@@ -157,7 +157,9 @@ const NumberCellRenderer = (props: ICellRendererParams & { fieldType?: FieldType
   let formatted = num.toLocaleString()
 
   if (fieldType === 'currency') {
-    formatted = new Intl.NumberFormat('en-US', {
+    // TODO: Make locale and currency configurable via field config
+    // For now, use browser's default locale with USD
+    formatted = new Intl.NumberFormat(undefined, {
       style: 'currency',
       currency: 'USD',
     }).format(num)
@@ -494,7 +496,6 @@ export function AdvancedDataGrid({
       style={{ height: typeof height === 'number' ? `${height}px` : height, width: '100%' }}
     >
       <AgGridReact
-        theme="legacy"
         ref={gridRef}
         rowData={data}
         columnDefs={columnDefs}

packages/ui/src/components/grids/GridPagination.tsx

@@ -147,6 +147,7 @@ export function GridPagination({
             className="h-8 w-16"
             placeholder={(currentPage + 1).toString()}
             disabled={disabled || totalPages === 0}
+            aria-label="Jump to page number"
           />
         </form>