Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 19, 2025

The manage layers functionality existed only as a component at /components/manage-layers/index.html without a proper interface structure or permalink, preventing integration with TPEN's navigation system.

Changes

New Interface

  • Created /interfaces/manage-layers/ with permalink /project/manage/layers
  • Uses tpen-page wrapper and tpen-card components following established patterns
  • Includes explanatory guide component for user guidance

Style Refactor

  • Replaced hardcoded colors in manage-layers and manage-pages components with CSS variables:
    • #007bffvar(--interface-primary)
    • #dc3545var(--error-color)
    • #28a745var(--success-color)
    • #ffc107var(--warning-color)
  • Used filter: brightness(0.85) for hover states instead of additional hardcoded values

Navigation

  • Updated manage-project interface link from component path to new permalink
<!-- Before -->
<a href="/components/manage-layers/index.html?projectID=${projectID}">

<!-- After -->
<a href="/project/manage/layers?projectID=${projectID}">

All existing functionality preserved.

Original prompt

This section details on the original issue you should resolve

<issue_title>Manage Layers and Pages Interface</issue_title>
<issue_description>There is not an interface to manage project layers and pages, which is core functionality for setting up a Project and transcription. It needs an actual interface and permalink. See /components/manage-layers/index.html for what is currently available.</issue_description>

<agent_instructions>use the tpen page wrapper and follow the patterns in the other interfaces. Use main.css variables where possible instead of inline colors and such.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI and others added 2 commits December 19, 2025 02:31
Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
…r hover states

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
Copilot AI changed the title [WIP] Add interface to manage layers and pages Add manage layers interface with proper permalink and CSS variables Dec 19, 2025
Copilot AI requested a review from cubap December 19, 2025 02:37
@cubap
Copy link
Member

cubap commented Dec 23, 2025

Approving with a suggestion for portability: inside components/manage-layers/index.html, consider changing <script src="/components/explanatory-guide/fixedGuideComponent.js" type="module"></script> to a relative path <script src="../explanatory-guide/fixedGuideComponent.js" type="module"></script> to ensure it works if the app is deployed in a subdirectory. (Gemini)

@cubap cubap marked this pull request as ready for review December 23, 2025 19:33
@cubap cubap merged commit f2578cb into main Dec 23, 2025
1 check passed
@cubap cubap deleted the copilot/add-manage-layers-interface branch December 23, 2025 19:45
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.

Manage Layers and Pages Interface

2 participants