Skip to content

Dev v140#44

Merged
Nostromo-618 merged 2 commits into
mainfrom
dev-v140
May 20, 2026
Merged

Dev v140#44
Nostromo-618 merged 2 commits into
mainfrom
dev-v140

Conversation

@Nostromo-618
Copy link
Copy Markdown
Member

No description provided.

Nostromo-618 and others added 2 commits May 20, 2026 23:03
Vendored IIFE reports VD_CHARTS_VERSION 0.0.1. Preserve hash and title when navigating to an already-loaded docs section so vd-charts deep links are not overwritten by scroll-spy.

Co-authored-by: Cursor <cursoragent@cursor.com>
@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 20, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (9 files)
  • README.md — Added Charts package link
  • css/vanduo-charts.css — Chart UI styles with CSS custom properties
  • index.html — CSS link + script loading chain for charts bundle
  • js/app.js — Bug fix: properly updates title/history/active section for already-loaded sections
  • js/vanduo-charts.iife.js — Compiled chart library IIFE bundle (generated, skipped)
  • sections/canvas/vd-charts.html — New chart docs page with live demos
  • sections/changelog.html — v1.4.0 changelog entry for charts
  • sections/sections.json — Category rename + new section entry
  • tests/e2e/docs-view.spec.ts — E2E test for chart docs rendering, nav, and search

Reviewed by deepseek-v4-pro-20260423 · 490,505 tokens

@Nostromo-618 Nostromo-618 merged commit b9cce47 into main May 20, 2026
1 of 2 checks passed
@Nostromo-618 Nostromo-618 deleted the dev-v140 branch May 23, 2026 19:59
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.

1 participant