Skip to content

Refactor: rename Avatar.hovered to isOpen (or bioOpen) #50

Description

@khurchla

Surfaced by the clarity-review pass on the v0.1.0 release branch (B3). Do NOT apply for v0.1.0 — captured for post-launch.

After #47, the hovered flag in src/components/Avatar.svelte is semantically "the bio panel is open." It is toggled by click, focus, hover, and escape alike — not just hover. The hovered name predates that and reads as a literal pointer-over flag, which is misleading when reading handleEscape, handleClose, and aria-expanded={hovered}.

setOpen is already the setter; the state name should match. Suggested rename: hoveredisOpen (or bioOpen if the disambiguation against a future full-card-expand state matters).

Files

  • src/components/Avatar.svelte (single-file rename)

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions