@estiva-app/ui 0.60.1 → 0.60.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@estiva-app/ui",
-  "version": "0.60.1",
+  "version": "0.60.2",
   "description": "Estiva's design tokens (the contract) and a small set of primitives (a convenience) for every Estiva app.",
   "type": "module",
   "license": "MIT",

package/registry.json

@@ -4,7 +4,7 @@
     "kind": "package",
     "repo": "estiva-ui",
     "package": "@estiva-app/ui",
-    "packageVersion": "0.60.1",
+    "packageVersion": "0.60.2",
     "exports": 134,
     "typeExports": 141,
     "files": null

package/src/AppProvider.mdx

@@ -54,7 +54,9 @@ function Root({ children }) {
   did, which is also what a story or a test sees without one.
 - `pictureFor` gets the person's name and returns an address, or nothing for
   the initials.
-- `navigate` gets the link's `href` exactly as written. The browser keeps
+- `navigate` gets the link's path, query and `#` part (`/message/1?x=2`),
+  also when the `href` is the app's whole address (`https://app.test/message/1`):
+  a router reads a whole address as a path under the current page. The browser keeps
   every other click: one already handled (`preventDefault`), one with Ctrl,
   Cmd, Shift or Alt, the middle button, a link that opens a new tab or
   downloads, and an address on another site or in another scheme

package/src/AppProvider.test.tsx

@@ -86,6 +86,13 @@ describe('changing page', () => {
     expect(navigate).toHaveBeenCalledWith('/topics')
   })
 
+  it("hands over the path when the href is the app's whole address", async () => {
+    const navigate = vi.fn()
+    inApp(navigate, <Link href={`${window.location.origin}/message/1?thread=2#r3`}>Message</Link>)
+    await userEvent.click(screen.getByRole('link'))
+    expect(navigate).toHaveBeenCalledWith('/message/1?thread=2#r3')
+  })
+
   it("runs the caller's own onClick first", async () => {
     const order: string[] = []
     inApp(() => order.push('navigate'), <Link href="/topics" onClick={() => order.push('caller')}>Topics</Link>)

package/src/AppProvider.tsx

@@ -10,8 +10,9 @@ export interface AppProviderProps {
   pictureFor?: (name: string) => string | null | undefined
   /**
    * How this app changes page without reloading. A plain click on a link to
-   * this app's own address calls it with the link's `href` instead of loading
-   * the page again: `Link` (and so `Card` and `Breadcrumb`'s crumbs),
+   * this app's own address calls it with the link's path (`/message/1`, also
+   * when the `href` is the whole `https://…` address) instead of loading the
+   * page again: `Link` (and so `Card` and `Breadcrumb`'s crumbs),
    * `NavItem` (and so `NavTree`'s rows), `RailItem`, `Chip`, `InlineChip`, and
    * `IconButton` (and so `ToolbarLink`).
    */
@@ -71,6 +72,8 @@ export function useInAppClick<E extends HTMLElement>(href: string | undefined, o
     }
     if (url.origin !== window.location.origin) return
     event.preventDefault()
-    navigate(href)
+    // The path, never the whole address: a router reads `https://…` as a path
+    // under the current page and lands nowhere (Peek's message link, 8 October).
+    navigate(`${url.pathname}${url.search}${url.hash}`)
   }
 }