Skip to content

[nightshift] docs-backfill: add JSDoc documentation to all 21 JS modules#5

Closed
nightshift-micr wants to merge 1 commit into
micr-dev:mainfrom
nightshift-micr:nightshift/docs-backfill
Closed

[nightshift] docs-backfill: add JSDoc documentation to all 21 JS modules#5
nightshift-micr wants to merge 1 commit into
micr-dev:mainfrom
nightshift-micr:nightshift/docs-backfill

Conversation

@nightshift-micr
Copy link
Copy Markdown
Contributor

Automated by Nightshift v3 (GLM 5.1).

Task: docs-backfill (Documentation Backfiller)
Category: pr
Changes: Added JSDoc documentation to all 21 JavaScript modules in js/ directory.

Summary

Adds comprehensive JSDoc @fileoverview, @param, @returns, @typedef, and @exports annotations to every JS module:

  • gallery-shared.js — 12 functions documented (sanitization, formatting, loading, rendering)
  • layout.js — Desktop scaling system (DESIGN_WIDTH/HEIGHT, computeScale, syncLayout, toDesignPx)
  • drag.js — Window dragging with mouse/touch support (makeDraggable)
  • controls.js — Volume slider and animation toggle
  • background.js — Color conversion utilities (hexToRgb, rgbToHsl)
  • tabs.js — 98.css tab navigation with keyboard support
  • gallery.js — Desktop gallery viewer with metadata
  • warning.js — Warning popup and title-bar control interceptor
  • tiles.js — Background tile switcher
  • sounds.js — Windows 98 sound effects player
  • notepad.js — Random notepad content with font/wrap controls
  • clippy.js — Quote display system
  • debug.js — Debug mode with resize grips
  • mobile-*.js — Mobile gallery, viewer, tabs, sounds
  • res-warning.js — Resolution compatibility warning
  • logo.js — Random logo selector
  • run.js — Console branding
  • app.js — Main application bootstrap

21 files changed, +276 lines of documentation. No functional changes.

Merge if useful, close if not.

@Microck Microck added dayshift/pr-inbox Managed by hermes-dayshift-glm dayshift/in-progress Managed by hermes-dayshift-glm dayshift/failed Managed by hermes-dayshift-glm and removed dayshift/pr-inbox Managed by hermes-dayshift-glm labels Apr 25, 2026
@Microck Microck closed this Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dayshift/failed Managed by hermes-dayshift-glm dayshift/in-progress Managed by hermes-dayshift-glm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants