@sonny-ui/core 0.1.0-alpha.16 → 0.1.0-alpha.17

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sonny-ui/core",
-  "version": "0.1.0-alpha.16",
+  "version": "0.1.0-alpha.17",
   "description": "Angular UI component library inspired by shadcn/ui — signals, zoneless, Tailwind CSS v4",
   "peerDependencies": {
     "@angular/common": "^21.0.0",

package/src/lib/avatar-group/avatar-group.component.spec.ts

@@ -0,0 +1,74 @@
+import { Component, signal } from '@angular/core';
+import { TestBed, type ComponentFixture } from '@angular/core/testing';
+import { SnyAvatarGroupComponent, type AvatarGroupItem } from './avatar-group.component';
+
+@Component({
+  standalone: true,
+  imports: [SnyAvatarGroupComponent],
+  template: `<sny-avatar-group [items]="items()" [max]="max()" [size]="size()" />`,
+})
+class TestHost {
+  items = signal<AvatarGroupItem[]>([
+    { src: 'a.jpg', alt: 'Alice' },
+    { src: 'b.jpg', alt: 'Bob' },
+    { src: 'c.jpg', alt: 'Carol' },
+    { src: 'd.jpg', alt: 'David' },
+    { src: 'e.jpg', alt: 'Eve' },
+  ]);
+  max = signal(3);
+  size = signal<'sm' | 'md' | 'lg'>('md');
+}
+
+describe('SnyAvatarGroupComponent', () => {
+  let fixture: ComponentFixture<TestHost>;
+  let el: HTMLElement;
+
+  beforeEach(async () => {
+    await TestBed.configureTestingModule({ imports: [TestHost] }).compileComponents();
+    fixture = TestBed.createComponent(TestHost);
+    fixture.detectChanges();
+    el = fixture.nativeElement;
+  });
+
+  it('should render max avatars', () => {
+    const imgs = el.querySelectorAll('img');
+    expect(imgs.length).toBe(3);
+  });
+
+  it('should show overflow counter', () => {
+    const counter = el.querySelector('[title="2 more"]');
+    expect(counter).not.toBeNull();
+    expect(counter?.textContent).toContain('+2');
+  });
+
+  it('should not show counter when no overflow', () => {
+    fixture.componentInstance.max.set(5);
+    fixture.detectChanges();
+    const counter = el.querySelector('[title]');
+    expect(counter).toBeNull();
+  });
+
+  it('should render all when max >= items', () => {
+    fixture.componentInstance.max.set(10);
+    fixture.detectChanges();
+    const imgs = el.querySelectorAll('img');
+    expect(imgs.length).toBe(5);
+  });
+
+  it('should render fallback initials when no src', () => {
+    fixture.componentInstance.items.set([
+      { fallback: 'AB' },
+      { fallback: 'CD' },
+    ]);
+    fixture.componentInstance.max.set(2);
+    fixture.detectChanges();
+    const fallbacks = el.querySelectorAll('.bg-muted');
+    expect(fallbacks.length).toBe(2);
+    expect(fallbacks[0].textContent).toContain('AB');
+  });
+
+  it('should have aria-label on group', () => {
+    const group = el.querySelector('[role="group"]');
+    expect(group?.getAttribute('aria-label')).toBe('Group of 5 users');
+  });
+});

package/src/lib/avatar-group/avatar-group.component.ts

@@ -0,0 +1,89 @@
+import {
+  ChangeDetectionStrategy,
+  Component,
+  computed,
+  input,
+} from '@angular/core';
+import { cn } from '../core/utils/cn';
+
+export interface AvatarGroupItem {
+  src?: string;
+  alt?: string;
+  fallback?: string;
+}
+
+const sizeMap = {
+  sm: { avatar: 'h-7 w-7 text-xs', counter: 'h-7 w-7 text-[10px]' },
+  md: { avatar: 'h-9 w-9 text-sm', counter: 'h-9 w-9 text-xs' },
+  lg: { avatar: 'h-11 w-11 text-base', counter: 'h-11 w-11 text-sm' },
+};
+
+const spacingMap = {
+  tight: '-space-x-3',
+  normal: '-space-x-2',
+};
+
+export type AvatarGroupSize = 'sm' | 'md' | 'lg';
+export type AvatarGroupSpacing = 'tight' | 'normal';
+
+@Component({
+  selector: 'sny-avatar-group',
+  standalone: true,
+  changeDetection: ChangeDetectionStrategy.OnPush,
+  template: `
+    <div [class]="containerClass()" role="group" [attr.aria-label]="'Group of ' + items().length + ' users'">
+      @for (item of visibleItems(); track $index) {
+        @if (item.src) {
+          <img
+            [src]="item.src"
+            [alt]="item.alt ?? ''"
+            [class]="avatarClass()"
+          />
+        } @else {
+          <div [class]="fallbackClass()">
+            {{ item.fallback ?? '?' }}
+          </div>
+        }
+      }
+      @if (overflowCount() > 0) {
+        <div [class]="counterClass()" [title]="overflowCount() + ' more'">
+          +{{ overflowCount() }}
+        </div>
+      }
+    </div>
+  `,
+})
+export class SnyAvatarGroupComponent {
+  readonly items = input.required<AvatarGroupItem[]>();
+  readonly max = input(3);
+  readonly size = input<AvatarGroupSize>('md');
+  readonly spacing = input<AvatarGroupSpacing>('normal');
+
+  readonly visibleItems = computed(() => this.items().slice(0, this.max()));
+  readonly overflowCount = computed(() => Math.max(0, this.items().length - this.max()));
+
+  readonly containerClass = computed(() =>
+    cn('flex items-center', spacingMap[this.spacing()])
+  );
+
+  readonly avatarClass = computed(() =>
+    cn(
+      'inline-block rounded-full object-cover ring-2 ring-background',
+      sizeMap[this.size()].avatar
+    )
+  );
+
+  readonly fallbackClass = computed(() =>
+    cn(
+      'inline-flex items-center justify-center rounded-full bg-muted text-muted-foreground font-medium ring-2 ring-background',
+      sizeMap[this.size()].avatar
+    )
+  );
+
+  readonly counterClass = computed(() =>
+    cn(
+      'inline-flex items-center justify-center rounded-full bg-muted text-muted-foreground font-semibold ring-2 ring-background',
+      sizeMap[this.size()].counter
+    )
+  );
+}

package/src/lib/avatar-group/index.ts

@@ -0,0 +1 @@
+export { SnyAvatarGroupComponent, type AvatarGroupItem, type AvatarGroupSize, type AvatarGroupSpacing } from './avatar-group.component';

package/src/lib/number-input/index.ts

@@ -0,0 +1,2 @@
+export { SnyNumberInputComponent } from './number-input.component';
+export { numberInputVariants, type NumberInputSize } from './number-input.variants';

package/src/lib/number-input/number-input.component.spec.ts

@@ -0,0 +1,151 @@
+import { Component, signal } from '@angular/core';
+import { FormControl, ReactiveFormsModule } from '@angular/forms';
+import { TestBed, type ComponentFixture } from '@angular/core/testing';
+import { SnyNumberInputComponent } from './number-input.component';
+
+@Component({
+  standalone: true,
+  imports: [SnyNumberInputComponent],
+  template: `
+    <sny-number-input
+      [(value)]="num"
+      [min]="min()"
+      [max]="max()"
+      [step]="step()"
+      [disabled]="disabled()"
+    />
+  `,
+})
+class TestHost {
+  num = signal(5);
+  min = signal<number | null>(null);
+  max = signal<number | null>(null);
+  step = signal(1);
+  disabled = signal(false);
+}
+
+describe('SnyNumberInputComponent', () => {
+  let fixture: ComponentFixture<TestHost>;
+  let el: HTMLElement;
+
+  beforeEach(async () => {
+    await TestBed.configureTestingModule({ imports: [TestHost] }).compileComponents();
+    fixture = TestBed.createComponent(TestHost);
+    fixture.detectChanges();
+    el = fixture.nativeElement;
+  });
+
+  function getInput(): HTMLInputElement {
+    return el.querySelector('input') as HTMLInputElement;
+  }
+  function getButtons(): HTMLButtonElement[] {
+    return Array.from(el.querySelectorAll('button'));
+  }
+
+  it('should render with initial value', () => {
+    expect(getInput().value).toBe('5');
+  });
+
+  it('should increment on + click', () => {
+    getButtons()[1].click();
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(6);
+  });
+
+  it('should decrement on - click', () => {
+    getButtons()[0].click();
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(4);
+  });
+
+  it('should respect min', () => {
+    fixture.componentInstance.min.set(5);
+    fixture.detectChanges();
+    getButtons()[0].click();
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(5);
+  });
+
+  it('should respect max', () => {
+    fixture.componentInstance.max.set(5);
+    fixture.detectChanges();
+    getButtons()[1].click();
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(5);
+  });
+
+  it('should use step', () => {
+    fixture.componentInstance.step.set(10);
+    fixture.detectChanges();
+    getButtons()[1].click();
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(15); // 5 + 10
+  });
+
+  it('should handle manual input on blur', () => {
+    const input = getInput();
+    input.value = '42';
+    input.dispatchEvent(new Event('input'));
+    input.dispatchEvent(new Event('blur'));
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(42);
+  });
+
+  it('should revert invalid input on blur', () => {
+    const input = getInput();
+    input.value = 'abc';
+    input.dispatchEvent(new Event('input', { bubbles: true }));
+    input.dispatchEvent(new Event('blur', { bubbles: true }));
+    fixture.detectChanges();
+    // Value should remain unchanged at 5
+    expect(fixture.componentInstance.num()).toBe(5);
+  });
+
+  it('should handle ArrowUp/Down', () => {
+    const input = getInput();
+    input.dispatchEvent(new KeyboardEvent('keydown', { key: 'ArrowUp', bubbles: true }));
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(6);
+    input.dispatchEvent(new KeyboardEvent('keydown', { key: 'ArrowDown', bubbles: true }));
+    fixture.detectChanges();
+    expect(fixture.componentInstance.num()).toBe(5);
+  });
+
+  it('should disable when disabled', () => {
+    fixture.componentInstance.disabled.set(true);
+    fixture.detectChanges();
+    expect(getInput().disabled).toBe(true);
+    expect(getButtons().every(b => b.disabled)).toBe(true);
+  });
+});
+
+@Component({
+  standalone: true,
+  imports: [ReactiveFormsModule, SnyNumberInputComponent],
+  template: `<sny-number-input [formControl]="ctrl" />`,
+})
+class ReactiveHost {
+  ctrl = new FormControl(10);
+}
+
+describe('SnyNumberInputComponent — Reactive Forms', () => {
+  let fixture: ComponentFixture<ReactiveHost>;
+
+  beforeEach(async () => {
+    await TestBed.configureTestingModule({ imports: [ReactiveHost] }).compileComponents();
+    fixture = TestBed.createComponent(ReactiveHost);
+    fixture.detectChanges();
+  });
+
+  it('should display FormControl value', () => {
+    const input = fixture.nativeElement.querySelector('input') as HTMLInputElement;
+    expect(input.value).toBe('10');
+  });
+
+  it('should update FormControl on increment', () => {
+    const buttons = fixture.nativeElement.querySelectorAll('button');
+    buttons[1].click();
+    fixture.detectChanges();
+    expect(fixture.componentInstance.ctrl.value).toBe(11);
+  });
+});

package/src/lib/number-input/number-input.component.ts

@@ -0,0 +1,153 @@
+import {
+  ChangeDetectionStrategy,
+  Component,
+  computed,
+  effect,
+  forwardRef,
+  input,
+  model,
+  signal,
+  untracked,
+} from '@angular/core';
+import { ControlValueAccessor, NG_VALUE_ACCESSOR } from '@angular/forms';
+import { cn } from '../core/utils/cn';
+import { numberInputVariants, type NumberInputSize } from './number-input.variants';
+
+@Component({
+  selector: 'sny-number-input',
+  standalone: true,
+  changeDetection: ChangeDetectionStrategy.OnPush,
+  providers: [
+    { provide: NG_VALUE_ACCESSOR, useExisting: forwardRef(() => SnyNumberInputComponent), multi: true },
+  ],
+  template: `
+    <div [class]="containerClass()">
+      <button
+        type="button"
+        class="px-2.5 hover:bg-accent transition-colors border-r border-border disabled:opacity-40 disabled:cursor-not-allowed"
+        [disabled]="isDisabled() || atMin()"
+        (click)="decrement()"
+        aria-label="Decrease"
+      >
+        <svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M5 12h14"/></svg>
+      </button>
+      <input
+        #inputEl
+        type="text"
+        inputmode="decimal"
+        class="flex-1 w-14 text-center outline-none bg-transparent font-medium"
+        [value]="inputValue()"
+        [disabled]="isDisabled()"
+        [placeholder]="placeholder()"
+        [attr.aria-label]="'Number input'"
+        [attr.aria-valuemin]="min() ?? null"
+        [attr.aria-valuemax]="max() ?? null"
+        [attr.aria-valuenow]="value()"
+        role="spinbutton"
+        (input)="onInput($event)"
+        (blur)="commitValue()"
+        (keydown)="onKeydown($event)"
+      />
+      <button
+        type="button"
+        class="px-2.5 hover:bg-accent transition-colors border-l border-border disabled:opacity-40 disabled:cursor-not-allowed"
+        [disabled]="isDisabled() || atMax()"
+        (click)="increment()"
+        aria-label="Increase"
+      >
+        <svg xmlns="http://www.w3.org/2000/svg" width="14" height="14" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"><path d="M5 12h14"/><path d="M12 5v14"/></svg>
+      </button>
+    </div>
+  `,
+})
+export class SnyNumberInputComponent implements ControlValueAccessor {
+  readonly value = model(0);
+  readonly min = input<number | null>(null);
+  readonly max = input<number | null>(null);
+  readonly step = input(1);
+  readonly disabled = input(false);
+  readonly size = input<NumberInputSize>('md');
+  readonly placeholder = input('');
+  readonly class = input<string>('');
+
+  readonly inputValue = signal('0');
+  private readonly _disabledByCva = signal(false);
+  readonly isDisabled = computed(() => this.disabled() || this._disabledByCva());
+
+  readonly atMin = computed(() => this.min() !== null && this.value() <= this.min()!);
+  readonly atMax = computed(() => this.max() !== null && this.value() >= this.max()!);
+
+  readonly containerClass = computed(() =>
+    cn(numberInputVariants({ size: this.size() }), this.isDisabled() && 'opacity-50', this.class())
+  );
+
+  private _onChange: (value: number) => void = () => {};
+  private _onTouched: () => void = () => {};
+
+  constructor() {
+    effect(() => {
+      const val = this.value();
+      untracked(() => this.inputValue.set(String(val)));
+    });
+  }
+
+  writeValue(val: number): void {
+    this.value.set(val ?? 0);
+  }
+
+  registerOnChange(fn: (value: number) => void): void {
+    this._onChange = fn;
+  }
+
+  registerOnTouched(fn: () => void): void {
+    this._onTouched = fn;
+  }
+
+  setDisabledState(isDisabled: boolean): void {
+    this._disabledByCva.set(isDisabled);
+  }
+
+  increment(): void {
+    this.setValue(this.value() + this.step());
+  }
+
+  decrement(): void {
+    this.setValue(this.value() - this.step());
+  }
+
+  onInput(event: Event): void {
+    this.inputValue.set((event.target as HTMLInputElement).value);
+  }
+
+  commitValue(): void {
+    const parsed = parseFloat(this.inputValue());
+    if (isNaN(parsed)) {
+      this.inputValue.set(String(this.value()));
+    } else {
+      this.setValue(parsed);
+    }
+    this._onTouched();
+  }
+
+  onKeydown(event: KeyboardEvent): void {
+    switch (event.key) {
+      case 'ArrowUp':
+        event.preventDefault();
+        this.increment();
+        break;
+      case 'ArrowDown':
+        event.preventDefault();
+        this.decrement();
+        break;
+    }
+  }
+
+  private setValue(raw: number): void {
+    let clamped = raw;
+    if (this.min() !== null) clamped = Math.max(this.min()!, clamped);
+    if (this.max() !== null) clamped = Math.min(this.max()!, clamped);
+    const rounded = parseFloat(clamped.toFixed(10));
+    this.value.set(rounded);
+    this._onChange(rounded);
+  }
+}

package/src/lib/number-input/number-input.variants.ts

@@ -0,0 +1,17 @@
+import { cva } from 'class-variance-authority';
+
+export const numberInputVariants = cva(
+  'inline-flex items-center border border-border rounded-md bg-background transition-colors focus-within:ring-2 focus-within:ring-ring focus-within:ring-offset-2',
+  {
+    variants: {
+      size: {
+        sm: 'h-9 text-xs',
+        md: 'h-10 text-sm',
+        lg: 'h-11 text-base',
+      },
+    },
+    defaultVariants: { size: 'md' },
+  }
+);
+
+export type NumberInputSize = 'sm' | 'md' | 'lg';

package/src/lib/popover/index.ts

@@ -0,0 +1,6 @@
+export {
+  SnyPopoverDirective,
+  SnyPopoverTriggerDirective,
+  SnyPopoverContentDirective,
+  SNY_POPOVER,
+} from './popover.directives';

package/src/lib/popover/popover.directives.spec.ts

@@ -0,0 +1,147 @@
+import { Component, signal } from '@angular/core';
+import { TestBed, type ComponentFixture } from '@angular/core/testing';
+import {
+  SnyPopoverDirective,
+  SnyPopoverTriggerDirective,
+  SnyPopoverContentDirective,
+} from './popover.directives';
+
+@Component({
+  standalone: true,
+  imports: [SnyPopoverDirective, SnyPopoverTriggerDirective, SnyPopoverContentDirective],
+  template: `
+    <div snyPopover [matchWidth]="matchWidth()" [closeOnOutside]="closeOnOutside()" [closeOnEscape]="closeOnEscape()" #pop="snyPopover">
+      <button snyPopoverTrigger>Open</button>
+      <div snyPopoverContent class="p-4">
+        <p>Popover content</p>
+        <button class="close-btn" (click)="pop.close()">Close</button>
+      </div>
+    </div>
+  `,
+})
+class TestHost {
+  matchWidth = signal(false);
+  closeOnOutside = signal(true);
+  closeOnEscape = signal(true);
+}
+
+describe('SnyPopoverDirective', () => {
+  let fixture: ComponentFixture<TestHost>;
+  let el: HTMLElement;
+
+  beforeEach(async () => {
+    await TestBed.configureTestingModule({ imports: [TestHost] }).compileComponents();
+    fixture = TestBed.createComponent(TestHost);
+    fixture.detectChanges();
+    el = fixture.nativeElement;
+  });
+
+  function getTrigger(): HTMLButtonElement {
+    return el.querySelector('[snypopovertrigger]') as HTMLButtonElement;
+  }
+
+  function getContent(): HTMLElement | null {
+    return el.querySelector('[snyPopoverContent], [snypopovercontent]');
+  }
+
+  function isVisible(): boolean {
+    const content = getContent();
+    return content ? content.style.display !== 'none' : false;
+  }
+
+  it('should render trigger and hidden content', () => {
+    expect(getTrigger()).not.toBeNull();
+    expect(getContent()).not.toBeNull();
+    expect(isVisible()).toBe(false);
+  });
+
+  it('should open on trigger click', () => {
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+  });
+
+  it('should close on second trigger click', () => {
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(false);
+  });
+
+  it('should set aria-expanded on trigger', () => {
+    expect(getTrigger().getAttribute('aria-expanded')).toBe('false');
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(getTrigger().getAttribute('aria-expanded')).toBe('true');
+  });
+
+  it('should have aria-haspopup on trigger', () => {
+    expect(getTrigger().getAttribute('aria-haspopup')).toBe('dialog');
+  });
+
+  it('should have role=dialog on content', () => {
+    expect(getContent()?.getAttribute('role')).toBe('dialog');
+  });
+
+  it('should close on click outside', () => {
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+
+    document.body.click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(false);
+  });
+
+  it('should not close on click outside when closeOnOutside=false', () => {
+    fixture.componentInstance.closeOnOutside.set(false);
+    fixture.detectChanges();
+
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+
+    document.body.click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+  });
+
+  it('should close on escape', () => {
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+
+    const host = el.querySelector('[snyPopover], [snypopover]') as HTMLElement;
+    host.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape', bubbles: true }));
+    fixture.detectChanges();
+    expect(isVisible()).toBe(false);
+  });
+
+  it('should not close on escape when closeOnEscape=false', () => {
+    fixture.componentInstance.closeOnEscape.set(false);
+    fixture.detectChanges();
+
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+
+    const host = el.querySelector('[snyPopover], [snypopover]') as HTMLElement;
+    host.dispatchEvent(new KeyboardEvent('keydown', { key: 'Escape', bubbles: true }));
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+  });
+
+  it('should close programmatically via template ref', () => {
+    getTrigger().click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(true);
+
+    const closeBtn = el.querySelector('.close-btn') as HTMLButtonElement;
+    closeBtn.click();
+    fixture.detectChanges();
+    expect(isVisible()).toBe(false);
+  });
+});