Skip to content

feat(kanban): add optional task description preview#63

Open
totondruppusu wants to merge 3 commits into
StepanKropachev:mainfrom
totondruppusu:feat/kanban-description-preview
Open

feat(kanban): add optional task description preview#63
totondruppusu wants to merge 3 commits into
StepanKropachev:mainfrom
totondruppusu:feat/kanban-description-preview

Conversation

@totondruppusu

Copy link
Copy Markdown

Before you submit

This plugin is maintained by one person in their spare time with a specific design vision and strict code standards. Please read this before opening a PR.

PRs without a linked issue are closed without review. New features and non-trivial changes must be discussed in an issue first — no exceptions.

AI-generated bulk PRs are closed immediately. If your PR rewrites large chunks of existing code, adds generic boilerplate, or ignores the project style, it will be closed with no further explanation.


What this PR does

Adds an option to show a brief preview of the description of tasks in board view.

Linked issue

Feature as discussed in issue #59

Checklist

  • I discussed this change in an issue and got a thumbs-up before writing code.
  • npm run build passes with zero errors.
  • npx eslint src/ passes with zero warnings — includes no inline styles, sentence-case UI text, no unnecessary non-null assertions.
  • New UI follows the Quiet Architect design system (no emojis, no hard corners, no 1px dividers, no pure black text, depth via tonal stacking).
  • New modals are created through ModalFactory, never instantiated directly.
  • No element.style.* assignments — CSS classes only.
  • No dynamic await import() — static imports only.
  • Dates use YYYY-MM-DD format; UTC operations only (T00:00:00Z, setUTCDate()).
  • This PR is focused on one logical change. Unrelated cleanup is in a separate PR.

@StepanKropachev

Copy link
Copy Markdown
Owner

Thanks for the contribution — the implementation follows the kanbanShowSubtasks precedent cleanly, scope matches what we agreed in #59 (default off, 3-line clamp, plain text), and the CHANGELOG entry is in the right spot.

A couple of things to address before I can merge:

  1. Merge conflict with main — please rebase. GitHub is showing the branch as conflicting.
  2. Trailing newline on styles.css — the diff strips the final newline at EOF. oxfmt requires a trailing newline, so pnpm check will fail on this. Running pnpm fix should restore it.

After rebasing, please confirm pnpm check && pnpm check:submission && pnpm test all pass locally — same gate CI runs.

Minor (not blocking): var(--pm-text-muted) has no fallback, but that's consistent with the rest of styles.css and fine since the card never renders inside a modal.

Looks great otherwise — thanks again!

@totondruppusu totondruppusu force-pushed the feat/kanban-description-preview branch from 1eac2ef to 6b4bdd6 Compare May 22, 2026 19:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants