Skip to content

Add manuscript preview to the shared-account landing page - #16

Merged
DebelToni merged 2 commits into
mainfrom
feat/landing-manuscript
Oct 20, 2026
Merged

DebelToni merged 2 commits into
mainfrom
feat/landing-manuscript

Conversation

@Slavov88

@Slavov88 Slavov88 commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Add an interactive manuscript example to the shared-account landing page, with a responsive layout, clearer spacing, and the existing light/dark themes. Visitors can switch between a paper and its modular LaTeX files; the sign-in action stays ahead of the example on phones.

Based on main ac6c8d0. The descriptive Collaborative LaTeX heading and concise copy are unchanged. The current Continue to shared accounts anchor, validated deep-link return path, and static expired-attempt/provider recovery remain unchanged. No local credentials, invitations, or human bearer-token sessions are restored. Auth styling lives alongside the page; dashboard and workspace rules are preserved.

Validation:

  • Full frontend suite: 20 files, 126 tests passed. All existing shared-account tests retained; three new integration cases verify preview interactions preserve the safe return path and expired-attempt recovery.
  • Focused auth/App/session/preview checks: 25 tests passed.
  • Frontend TypeScript, ESLint, and production build passed; server build passed.
  • Local production-build browser checks cover desktop/phone layouts, both themes, source-file switching, and recovery copy/links. The disposable anonymous local provider fixture cannot issue grants; this is not evidence of live central-account activation.
  • Independent final diff review found no actionable issues.

No new dependencies, API calls, permissions, migrations, provider configuration, or storage. The example uses only local component state. Hosted CI and CodeQL must pass for this new revision; real Docker compiler checks run there because the local Docker daemon is unavailable. Existing owner-controlled review and deployment gates remain in effect.

@DebelToni DebelToni left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Integration review: needs attention before merge.

  1. This revision is still based on 010beb9; main is now ac6c8d0, and GitHub reports merge conflicts. Please rebase and resolve the AuthPage/styles integration before requesting review of the resulting revision.
  2. Preserve the current site-account flow: the Continue to shared accounts link, safe deep-link return path and expired-attempt handling. The proposed AuthPage/test snapshot still contains the retired local username/password/invitation forms and api.login/api.register assumptions. Update those tests for the central account redirect; do not restore local credentials, invite-only account creation or localStorage human sessions.
  3. Keep the descriptive Collaborative LaTeX heading and minimal useful copy. The proposed One manuscript / Many minds slogan is outside the agreed UI direction.

The manuscript example can be preserved in a compatible rebased revision. No PR code, tests, build or installer was executed on DGX. This is a blocked integration review, not approval of the complete UI revision; the new head/base will need review and hosted CI.

@Slavov88
Slavov88 force-pushed the feat/landing-manuscript branch from b85c9e2 to f03ab14 Compare October 20, 2026 11:53
@Slavov88 Slavov88 changed the title Give the landing page a manuscript-focused design Add manuscript preview to the shared-account landing page Oct 20, 2026

@DebelToni DebelToni left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed the complete seven-file diff at f03ab14 against current main ac6c8d0. No blocking findings.

The previous integration findings are addressed: rebased conflict-free; site-account redirect, safe return target and explicit expired-attempt recovery retained; no local password/invitation form or human token storage; descriptive Collaborative LaTeX heading retained. The manuscript/source example is fixed local JSX/text with no network calls, HTML injection, document access or additional permissions. Reviewed view/file controls, focus/pressed semantics, responsive CSS, dark-theme variables and the scoped removal of obsolete auth styles; dashboard styling is preserved.

Hosted exact-head test/analyze and CodeQL checks passed. No candidate code, tests, dependencies, hooks or builds were executed on DGX. No backend, dependency, workflow, security policy or host-automation changes. Visual rendering on the eventual live release remains a deployment check.

@DebelToni
DebelToni merged commit 66f6c6a into main Oct 20, 2026
3 checks passed
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