@nxgt/material 2.5.0 → 2.6.1

package/docs/troubleshooting.md

@@ -113,6 +113,8 @@ words in your error.
 - [A menu item selected by `[data-variant=destructive]` stops matching](#a-menu-item-selected-by-data-variantdestructive-stops-matching)
 - [A `Stepper` step selected by `#step-N` stops matching](#a-stepper-step-selected-by-step-n-stops-matching)
 - [Keys typed right after opening a dialog, sheet or drawer land inside it](#keys-typed-right-after-opening-a-dialog-sheet-or-drawer-land-inside-it)
+- [A `TimePicker` test expects AM / PM to add 12 hours, or a click on the pressed half to switch to PM](#a-timepicker-test-expects-am--pm-to-add-12-hours-or-a-click-on-the-pressed-half-to-switch-to-pm)
+- [Focus is no longer in a `Tree` after it renders](#focus-is-no-longer-in-a-tree-after-it-renders)
 
 **PDF reader**
 
@@ -138,6 +140,9 @@ words in your error.
 - [The zoom changes by itself while a long PDF is open](#the-zoom-changes-by-itself-while-a-long-pdf-is-open)
 - [A link in a PDF replaces the page the reader is on](#a-link-in-a-pdf-replaces-the-page-the-reader-is-on)
 - [`onPageChange` is not called when `initialPage` is past the end, or after a new file](#onpagechange-is-not-called-when-initialpage-is-past-the-end-or-after-a-new-file)
+- [`onPageChange` reports another page after opening on `initialPage`](#onpagechange-reports-another-page-after-opening-on-initialpage)
+- [`FilePreview` shows a file card with a Download link for a PDF](#filepreview-shows-a-file-card-with-a-download-link-for-a-pdf)
+- [`FilePreview` says "This file could not be previewed."](#filepreview-says-this-file-could-not-be-previewed)
 - [Reporting a bug](#reporting-a-bug)
 
 ---
@@ -2086,6 +2091,55 @@ const saveButtonRef = useRef<HTMLButtonElement>(null);
 A `Drawer` also takes Vaul's `autoFocus={false}`, which does the same as
 preventing the default.
 
+### A `TimePicker` test expects AM / PM to add 12 hours, or a click on the pressed half to switch to PM
+
+**When:** after upgrading to 2.6.0, a test (or code of yours) around a
+`TimePicker` — or the popover of a `DateField` with `showTimePicker`, a
+`TimeField` or a `DateRangeField` with `withTime`, which hold one — sees a
+different time, or no `onChange`, when it clicks AM or PM.
+**Why:** before 2.6.0 the toggle added 12 hours on every PM and took
+`% 12` on AM. A time already in the afternoon went 12 hours further — PM on
+14:00 gave 02:00 the next day — and a click on the half already pressed
+(which Radix reports as `''`, unpressing it) showed PM but took the time
+`% 12`, as AM does — a pressed PM on 14:00 gave 02:00, still shown as PM.
+The AM / PM shown was also read once, from the first `date`, so a `date`
+changed from outside kept its old half. Now AM / PM follows the `date` it is
+given, switching moves the time into that half and no further (09:30 + PM is
+21:30, 21:30 + PM stays 21:30, 21:30 + AM is 09:30, on the same day), and a
+click on the pressed half changes nothing and calls no `onChange`.
+`@nxgt/material-vue` 0.25.0 has the same fix. The hours and minutes inputs
+are also named "Hours" and "Minutes" (`material.time-picker.hours`,
+`material.time-picker.minutes`).
+**Fix:** expect the time in the half you picked, and no call for a click on
+the pressed half. Find the inputs by their names rather than by position:
+
+```ts
+await userEvent.click(canvas.getByRole('radio', { name: 'PM' }));
+expect(onChange).toHaveBeenLastCalledWith(new Date(2026, 0, 1, 21, 30)); // from 09:30
+await userEvent.type(canvas.getByRole('spinbutton', { name: 'Hours' }), '10');
+```
+
+### Focus is no longer in a `Tree` after it renders
+
+**When:** after upgrading to 2.6.0, a test (or code of yours) that renders a
+`Tree` — or a page that holds one — and then presses arrow keys or Enter
+without first focusing a row finds nothing happens in the tree; a check such
+as `expect(row).toHaveFocus()` right after rendering fails.
+**Why:** before 2.6.0, mounting a `Tree` moved focus to its focused row (the
+default selection, else the first row), so a page that rendered one lost the
+user's focus to it, and under Strict Mode a remount did it again. Focus now
+follows the keyboard only, as in `@nxgt/material-vue`: it moves when the row
+that has the keyboard changes, never on mount. Tab still reaches the tree:
+the row with `tabindex="0"` (`useTree`'s `tabbableId`) is the focused row
+when it is shown, else the first visible row.
+**Fix:** focus the tree as a user does, by Tab or a click, before pressing
+keys; to move focus into it on purpose, focus a row yourself:
+
+```ts
+await userEvent.click(canvas.getByRole('treeitem', { name: /Docs/ }));
+await userEvent.keyboard('{ArrowDown}');
+```
+
 ---
 
 ## PDF reader
@@ -2430,6 +2484,63 @@ before 2.2.0.
 to itself. Since 2.2.0 both are reported, once.
 **Fix:** update `@nxgt/material`. A page you kept in sync by hand can go.
 
+### `onPageChange` reports another page after opening on `initialPage`
+
+**When:** the reader opens on `initialPage` and, without the user scrolling,
+`onPageChange` reports another page, most often the last one, and the page field
+shows it. Seen on slow devices and busy machines, near the end of a
+document, with `@nxgt/material` 2.5.0 and earlier.
+**Why:** after placing the opening page, the reader ignored scrolling for
+only 150 ms. When the placement scroll landed later (a late frame), it was
+read as the user's, and at the end of the pages that is the last page.
+**Fix:** update `@nxgt/material`. The opening page now keeps the page number while
+the view stays on it; the user's own scroll, find in page or a screen
+reader moving the view still report the page they reach.
+
+### `FilePreview` shows a file card with a Download link for a PDF
+
+**When:** a PDF given to `FilePreview` shows its name, size and a Download
+link instead of the reader.
+**Why:** `FilePreview` imports `PdfReader` the first time it previews a PDF,
+and falls back to the file card when that import fails: the viewer's chunk
+could not be fetched (a deploy replaced the chunks the open page was built
+with, or a network error), or the bundle has no `react-pdf` and `pdfjs-dist` — they are
+dependencies of `@nxgt/material`, not optional peers, so this is a bundler
+configuration that left them out. Nothing is thrown; the next PDF previewed
+tries again.
+**Fix:** reload the page after a deploy; otherwise check that your bundler
+keeps `react-pdf` and `pdfjs-dist` and pdf.js's worker (see `PdfReader` in
+[Media & files](components/media-and-files.md#pdfreader)). The same file in a
+`PdfReader` of its own shows the underlying error. A PDF the reader loaded
+but cannot open (a corrupt or password-protected file, a URL that answers an
+error page) is different: the preview stays `ready` and `PdfReader` shows its
+own error inside it.
+
+### `FilePreview` says "This file could not be previewed."
+
+**When:** a `FilePreview` given a URL, or a file it cannot show, renders that
+message (and `onError` receives the error).
+**Why:** a text, JSON, CSV or Markdown URL is fetched with `fetch`: an HTTP
+error is a `FilePreviewHttpError` with its `status`, whose message is
+`Could not read …: HTTP 404` (the URL as you passed it, then the status —
+`Could not read /files/notes.txt: HTTP 404`), and a cross-origin URL
+without CORS headers is a `TypeError: Failed to fetch`. An image, video or
+audio file the browser cannot decode (a corrupt file, a codec the browser
+lacks, a URL that answers an error page) fails in its own element. A URL
+that is not `http(s):`, relative, `blob:`, or a `data:` URL of an image,
+video, audio file, plain text, CSV, Markdown, JSON or PDF (`javascript:`,
+`data:text/html`, and also a `data:` URL of XML, TSV or YAML, though
+`FilePreview` would show those kinds from a `Blob`) is refused before
+anything is linked, loaded or fetched: the error is a
+`FilePreviewUnsafeUrlError`, "Refused to preview a URL with an unsafe scheme", and there is no
+Download link.
+**Fix:** serve the file with `Access-Control-Allow-Origin` for your page's
+origin, or fetch it yourself (with your credentials) and pass the `Blob`;
+pass `type` when the server answers `application/octet-stream` for a file
+whose URL has no extension; pass a refused URL's file as a `Blob`, or from
+an `https:` URL. A text file larger than `maxTextBytes` is not an error: its
+beginning is shown, with "Showing the first … of this file." below.
+
 ---
 
 ## Reporting a bug

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@nxgt/material",
-	"version": "2.5.0",
+	"version": "2.6.1",
 	"license": "MIT",
 	"bugs": {
 		"email": "stevegires@gmail.com"
@@ -113,7 +113,7 @@
 		"docs:previews": "vitest run --project previews"
 	},
 	"dependencies": {
-		"@nxgt/material-core": "^0.2.0",
+		"@nxgt/material-core": "^0.3.1",
 		"@base-ui/react": "^1.6.0",
 		"@dnd-kit/core": "^6.3.1",
 		"@dnd-kit/sortable": "^10.0.0",

package/dist/lib/components/ui/fields/slider/slider-ticks.d.ts

@@ -1 +0,0 @@
-export declare function tickValues(min: number, max: number, ticks: boolean | number, step?: number): number[];