Skip to content

Wrap manifest-import in tpen-page component#412

Merged
cubap merged 13 commits intomainfrom
copilot/sub-pr-406
Feb 2, 2026
Merged

Wrap manifest-import in tpen-page component#412
cubap merged 13 commits intomainfrom
copilot/sub-pr-406

Conversation

Copy link
Contributor

Copilot AI commented Jan 29, 2026

Addresses review feedback from PR #406 to use the standard tpen-page wrapper instead of a bespoke full-page interface.

Changes

  • interfaces/manifest-import/index.html: Wrapped tpen-manifest-import in tpen-page, removed 41 lines of custom CSS
  • components/manifest-import/index.js: Removed full-viewport styles (min-height: 100vh, background colors) from all render methods to integrate with tpen-page's content area

Before/After

<!-- Before: Bespoke interface -->
<body>
  <style>/* 41 lines of custom layout CSS */</style>
  <main>
    <tpen-manifest-import></tpen-manifest-import>
  </main>
</body>

<!-- After: Standard TPEN interface -->
<body>
  <tpen-page>
    <tpen-manifest-import></tpen-manifest-import>
  </tpen-page>
</body>

The component now inherits TPEN's header, footer, and navigation consistent with other interfaces (import-tpen28, profile, manage-columns).


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

cubap and others added 11 commits January 28, 2026 13:27
Introduce a new Manifest Import feature: adds a web component (components/manifest-import/index.js) that imports multiple IIIF manifests into TPEN, handles authentication, creates projects sequentially, and renders progress/results. Includes docs and support pages (IMPLEMENTATION.md, README.md, QUICKSTART.md), an examples page and interface (interfaces/manifest-import/index.html, examples.html, INDEX.md), and interface metadata (manifest.yml). Uses TPEN Services POST /project/import?createFrom=URL and integrates with TPEN auth/token handling.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
* Initial plan

* Fix URL format in INDEX.md to match permalink

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
* Initial plan

* Fix URL examples to use /import-manifest path

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
* Initial plan

* Replace custom #escapeHtml with shared utility from /js/utils.js

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
* Initial plan

* Fix /?manifest= to /import-manifest?manifest= across all documentation

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
* Initial plan

* Add main element wrapper for improved accessibility

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
Copilot AI changed the title [WIP] WIP address feedback to incorporate tpen-page component Wrap manifest-import in tpen-page component Jan 29, 2026
Copilot AI requested a review from cubap January 29, 2026 16:29
Base automatically changed from iiif-import-direct to main February 2, 2026 16:19
@cubap cubap marked this pull request as ready for review February 2, 2026 16:22
@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2026

@cubap cubap merged commit f91dc70 into main Feb 2, 2026
3 checks passed
@cubap cubap deleted the copilot/sub-pr-406 branch February 2, 2026 16:26
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