added desktop viewport

[airen1986]

Summary by CodeRabbit

• Bug Fixes
  • Updated viewport configuration to use a fixed 1280px width for optimized desktop display.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 930e4993-6a6e-4d62-bb99-3c9fac50e4b1

📥 Commits

Reviewing files that changed from the base of the PR and between ebbb95d and ae72db5.

📒 Files selected for processing (2)

• src/common/js/scl-app.js
• src/index.html

---

📝 Walkthrough

Walkthrough

Changes modify viewport settings to enforce a desktop-style layout at 1280px width. A new enforceDesktopViewport function is introduced in scl-app.js to manage the viewport meta tag, alongside duplicate function definitions and invocation points. The index.html viewport meta tag shifts from device-width to a fixed 1280px width.

Changes

Cohort / File(s)	Summary
Viewport Configuration src/index.html, src/common/js/scl-app.js	Enforces desktop viewport rendering at 1280px width. Introduces DESKTOP_VIEWPORT constant and enforceDesktopViewport function to update meta tags, but includes duplicate function definitions and redundant invocation sites that may create maintenance issues.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Poem

🐰 A viewport for desktops, 1280 pixels wide,
We hopped through the code with considerable pride,
But oh dear, oh my—what's this duplication I see?
Two functions, two calls—should there really be three?
A fix for the mobile, now locked in desktop mode! 🖥️

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feature/pr-setup

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}