pi-weave 0.3.0 → 0.3.1

package/src/web/client/note/note.model.ts

@@ -334,7 +334,7 @@ export function resolveMarkdownLink(index: WikiIndex, href: string): string | nu
  * Whether an unresolved wikilink should be drawn as a ghost.
  *
  * See {@link WikiIndex}: only when the payload independently confirms this
- * note has unresolved links. Obsidian's ghost styling is an *offer to create
+ * note has unresolved links. Ghost styling is an *offer to create
  * the note*, so showing it for a link that in fact resolves would be an offer
  * to create a note that already exists.
  */
@@ -847,32 +847,6 @@ export const SAVING_LABEL = "Saving…";
 export const EDITOR_ARIA_LABEL = "Note body";
 export const DISCARD_PROMPT = "Discard unsaved changes to this note?";
 
-/**
- * Whether the draft differs from the note as last loaded. Trailing whitespace
- * counts: the save trims, so calling such a draft clean would skip the prompt
- * on the one edit that gets silently reverted.
- */
-export function draftDirty(draft: string, body: string): boolean {
-  return draft !== body;
-}
-
-/**
- * Whether a save's reply still belongs to the editor that issued it. A
- * counter, not the slug: reopening the same note is a new session too.
- */
-export function saveLanded(issued: number, current: number): boolean {
-  return issued === current;
-}
-
-/**
- * Whether the draft moved since the bytes a save sent — keystrokes made
- * during the request were never in it, so a success must not close over them.
- * Unlike {@link saveLanded}, this is about the *text*, not the session.
- */
-export function draftMoved(sent: string, current: string | null): boolean {
-  return current !== null && current !== sent;
-}
-
 /** The note header: title, provenance badge, relative time, tags. */
 export interface NoteHeaderView {
   readonly title: string;
@@ -959,13 +933,7 @@ export const PREVIEW_GAP = 8;
 /** The card's kind line for a link nothing in the vault answers to. */
 export const GHOST_KIND = "no note";
 
-/**
- * The card's DOM id, shared by the element and the `aria-describedby` it is
- * announced through. A constant rather than an inline literal on both sides
- * because the link is a *string* of rendered HTML — see
- * `Note.tsx`'s layout effect — and the pair cannot be typed into agreement.
- */
-export const PREVIEW_ID = "weave-preview";
+
 
 /** A `PreviewElement` the preview's delegated handlers may land on. */
 export interface PreviewAnchor {

package/src/web/client/search/SearchPalette.tsx

@@ -9,12 +9,12 @@ import type { SearchState } from "./search.model";
 
 export interface SearchPaletteProps {
   graph: GraphPayload | null;
-  onSelect: (id: string) => void;
+  onSelect: (id: string, newTab?: boolean) => void;
   onClose: () => void;
   ports: { fetch: FetchLike };
 }
 
-function Row({ row, onPick, onHover }: { row: SearchRowView; onPick: () => void; onHover: () => void }) {
+function Row({ row, onPick, onHover }: { row: SearchRowView; onPick: (newTab: boolean) => void; onHover: () => void }) {
   return (
     <li
       id={row.domId}
@@ -22,7 +22,7 @@ function Row({ row, onPick, onHover }: { row: SearchRowView; onPick: () => void;
       aria-selected={row.active}
       class={`weave-hit weave-hit-${row.kind}${row.active ? " weave-hit-on" : ""}`}
       onMouseMove={onHover}
-      onClick={onPick}
+      onClick={(event) => onPick(event.metaKey || event.ctrlKey)}
     >
       <span class="weave-hit-badge">{row.badge}</span>
       <span class="weave-hit-label">{row.label}</span>
@@ -70,9 +70,9 @@ export function SearchPalette(props: SearchPaletteProps) {
   }, [state.query, props.ports.fetch]);
 
   const model = paletteModel(state, props.graph);
-  const pick = (index: number | null): void => {
+  const pick = (index: number | null, newTab = false): void => {
     const id = resultIdAt(model.rows, index);
-    if (id !== null) props.onSelect(id);
+    if (id !== null) props.onSelect(id, newTab);
     props.onClose();
   };
 
@@ -92,7 +92,7 @@ export function SearchPalette(props: SearchPaletteProps) {
           if (!next.handled) return;
           event.preventDefault();
           setState((current) => ({ ...current, cursor: next.cursor }));
-          if (next.dismiss) pick(next.activate);
+          if (next.dismiss) pick(next.activate, event.metaKey || event.ctrlKey);
         }}
       >
         <input
@@ -108,14 +108,15 @@ export function SearchPalette(props: SearchPaletteProps) {
         />
         {model.status === null ? (
           <ul id="weave-search-results" class="weave-hits" role="listbox" aria-label={PALETTE_TITLE}>
-            {model.rows.map((row, index) => <Row key={row.id} row={row} onPick={() => pick(index)} onHover={() => setState((current) => ({ ...current, cursor: index }))} />)}
+            {model.rows.map((row, index) => <Row key={row.id} row={row} onPick={(newTab) => pick(index, newTab)} onHover={() => setState((current) => ({ ...current, cursor: index }))} />)}
           </ul>
         ) : (
           <p class="weave-palette-status" role="status">{model.status}</p>
         )}
         <p class="weave-palette-foot">
           <span>{model.countLabel}</span>
-          <span class="weave-palette-hint">{PALETTE_HINT}</span>
+          <button type="button" class="weave-chip" disabled={model.count === 0} onClick={() => pick(state.cursor, true)}>Open in new tab</button>
+          <span class="weave-palette-hint">{PALETTE_HINT} · ⌘/Ctrl ↵ new tab</span>
         </p>
       </div>
     </div>

package/src/web/client/shell/ContextRail.tsx

@@ -22,13 +22,13 @@ import { Icon } from "../tree/Tree";
 export interface ContextRailProps {
   graph: GraphPayload | null;
   selectedId: string | null;
-  onSelect: (id: string) => void;
+  onSelect: (id: string, newTab?: boolean) => void;
 }
 
-function Row({ row, onSelect }: { row: ContextRow; onSelect: () => void }) {
+function Row({ row, onSelect }: { row: ContextRow; onSelect: (newTab: boolean) => void }) {
   return (
     <li class={`weave-ctx-row${row.selected ? " weave-row-on" : ""}`}>
-      <button type="button" class="weave-ctx-link" onClick={onSelect}>
+      <button type="button" class="weave-ctx-link" onClick={(event) => onSelect(event.metaKey || event.ctrlKey)}>
         <span class="weave-kind" aria-hidden="true">
           <Icon name={row.kindIcon} class="weave-icon" />
         </span>
@@ -64,26 +64,26 @@ function Heading({ view, onToggle }: { view: RailSectionView; onToggle: () => vo
   );
 }
 
-function Group({ view, onSelect, onToggle }: { view: RailSectionView; onSelect: (id: string) => void; onToggle: () => void }) {
+function Group({ view, onSelect, onToggle }: { view: RailSectionView; onSelect: (id: string, newTab?: boolean) => void; onToggle: () => void }) {
   return (
     <section class="weave-ctx-group">
       <Heading view={view} onToggle={onToggle} />
       <ul class="weave-ctx-rows" id={railPanelId(view.heading)} hidden={view.collapsed}>
         {view.rows.map((row) => (
-          <Row key={row.id} row={row} onSelect={() => onSelect(row.target)} />
+          <Row key={row.id} row={row} onSelect={(newTab) => onSelect(row.target, newTab)} />
         ))}
       </ul>
     </section>
   );
 }
 
-function Tag({ tag, onSelect }: { tag: TagGroupRow; onSelect: (id: string) => void }) {
+function Tag({ tag, onSelect }: { tag: TagGroupRow; onSelect: (id: string, newTab?: boolean) => void }) {
   return (
     <li class="weave-ctx-tag">
       <span class="weave-tag">#{tag.tag}</span>
       <ul class="weave-ctx-rows">
         {tag.siblings.map((row) => (
-          <Row key={row.id} row={row} onSelect={() => onSelect(row.target)} />
+          <Row key={row.id} row={row} onSelect={(newTab) => onSelect(row.target, newTab)} />
         ))}
       </ul>
     </li>
@@ -102,7 +102,6 @@ export function ContextRail(props: ContextRailProps) {
   const tags = railTagsView(model.tags, toggles);
   return (
     <div class="weave-rail" aria-label={CONTEXT_EMPTY.title}>
-      <h2 class="weave-col-title">{CONTEXT_EMPTY.title}</h2>
       {model.empty === null ? null : <p class="weave-ctx-empty">{model.empty}</p>}
       {model.groups.map((group) => {
         const view = railSectionView(group, toggles);
@@ -120,4 +119,4 @@ export function ContextRail(props: ContextRailProps) {
       )}
     </div>
   );
-}
+}