Skip to content

Wrap Manage Collaborators interface in tpen-page component#378

Merged
cubap merged 5 commits intomainfrom
copilot/manage-collaborators-interface
Dec 19, 2025
Merged

Wrap Manage Collaborators interface in tpen-page component#378
cubap merged 5 commits intomainfrom
copilot/manage-collaborators-interface

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 19, 2025

The Manage Collaborators interface used a custom wrapper instead of the standard <tpen-page> component, making it inconsistent with other TPEN interfaces.

Changes

HTML Structure

  • Wrapped interface in <tpen-page> and <tpen-card> components following established patterns
  • Removed custom header (tpen-page provides standardized navigation)
  • Added required component imports (gui/site/index.js, gui/card/Card.js)

CSS Variables

  • Replaced hardcoded colors with CSS variables from main.css:
    • #007BFFvar(--primary-color)
    • #b14628var(--accent)
    • #28a745var(--success-color)
    • #dc3545var(--error-color)
    • 30+ similar replacements across all component styles

Code Cleanup

  • Removed 86 lines of unused CSS for obsolete wrapper structure
  • Deleted custom return button (redundant with tpen-page navigation)

Example

Before:

<div class="manage-interface collaborators-container">
  <header class="title">
    <h1>TRANSCRIPTION PROJECT MANAGEMENT</h1>
  </header>
  <main>
    <project-collaborators></project-collaborators>
  </main>
</div>

After:

<tpen-page class="grid">
  <tpen-card>
    <h2 slot="header">Manage Collaborators</h2>
    <div slot="body">
      <project-collaborators></project-collaborators>
    </div>
  </tpen-card>
</tpen-page>
Original prompt

This section details on the original issue you should resolve

<issue_title>Manage Collaborators Interface as a <tpen-page></issue_title>
<issue_description>It would be nice if the component for this was wrapped in the <tpen-page> UI component for consistency. It will feel more integrated that way. This could mean other minor adjustments to that component/interface for look-and-feel or even UX. </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)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: cubap <1119165+cubap@users.noreply.github.com>
Copilot AI changed the title [WIP] Wrap manage collaborators interface in tpen-page Wrap Manage Collaborators interface in tpen-page component Dec 19, 2025
Copilot AI requested a review from cubap December 19, 2025 00:38
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 19, 2025

Added grid-column styling for tpen-card in collaborators CSS. Renamed collaborators.html to manage-project/collaborators.html for better organization.
@cubap cubap marked this pull request as ready for review December 19, 2025 02:21
@cubap cubap merged commit 9b69b24 into main Dec 19, 2025
2 checks passed
@cubap cubap deleted the copilot/manage-collaborators-interface branch December 19, 2025 02:25
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 Collaborators Interface as a <tpen-page>

2 participants