Skip to content

enh: copied-tooltip#1850

Open
Mazi02 wants to merge 2 commits into
devfrom
enh/tooltip
Open

enh: copied-tooltip#1850
Mazi02 wants to merge 2 commits into
devfrom
enh/tooltip

Conversation

@Mazi02

@Mazi02 Mazi02 commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Motivation and Context

Having a "copied" tooltip makes a better and cleaner UI.

Description

Adding a tooltip to confirm that keys or other things got copied.

Steps for Testing

  1. login as admin
  2. In admin panel, go to token management and create a new token.
  3. Click on the URL and or the key and the tooltip should appear.
  4. Also as you make a new stream and copy the rtmp URL and key it should appear.

NOTE: after 1.5 seconds it should disappear.

Screenshots

image image image

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR improves the web UI by adding a short-lived “Copied” tooltip when users copy stream URLs/keys and generated admin tokens, making copy actions clearer without adding persistent UI elements.

Changes:

  • Added Alpine helpers (copyAndShow, copied, copyTimeout) to display a “Copied” tooltip for various copy-to-clipboard actions.
  • Updated course lecture management and admin token templates to wrap copy targets with a tooltip container.
  • Introduced a shared .copy-tooltip CSS utility (including an arrow pseudo-element) for consistent tooltip styling.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
web/template/partial/course/manage/lecture-management-card.gohtml Adds tooltip behavior for copying stream server URL and stream key in lecture management.
web/template/admin/admin_tabs/token.gohtml Adds tooltip behavior for copying the generated token and RTMP details in the admin token flow.
web/assets/css/main.css Adds the reusable .copy-tooltip styling (positioning, colors, arrow).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +61 to 63
<option value="admin" class="text-4" x-show="role == 1" x-init="role = {{ .Role }}">
Scope: admin
</option>
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