Skip to content

Refresh Support Parrot admin UI/UX#31

Merged
rodica-andronache merged 6 commits into
masterfrom
development
Jun 19, 2026
Merged

Refresh Support Parrot admin UI/UX#31
rodica-andronache merged 6 commits into
masterfrom
development

Conversation

@rodica-andronache

Copy link
Copy Markdown
Contributor
  • Rebuild the settings screen with a clean, WP-native card layout (status banner, intro card, details panel, action buttons), replacing the legacy printf/inline-styled markup
  • Use the official Themeisle logo in the header
  • Show share details as right-aligned label/value rows
  • Make "Copy all details" the primary action with modern navigator.clipboard copy + "Copied!" feedback
  • Load the screen CSS/JS for the customer (not just the parrot user)
  • Redirect to the settings page on plugin activation
  • Rename plugin to "Themeisle Support Parrot" and update description
  • Fix stray current-time echo in get_expiration_date()
  • Ensure admin notices render below the header (wp-header-end marker)

- Rebuild the settings screen with a clean, WP-native card layout
  (status banner, intro card, details panel, action buttons),
  replacing the legacy printf/inline-styled markup
- Use the official Themeisle logo in the header
- Show share details as right-aligned label/value rows
- Make "Copy all details" the primary action with modern
  navigator.clipboard copy + "Copied!" feedback
- Load the screen CSS/JS for the customer (not just the parrot user)
- Redirect to the settings page on plugin activation
- Rename plugin to "Themeisle Support Parrot" and update description
- Fix stray current-time echo in get_expiration_date()
- Ensure admin notices render below the header (wp-header-end marker)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

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.

⚠️ Not ready to approve

The new activation redirect logic can incorrectly redirect AJAX/REST/cron or unauthorized contexts, which risks breaking admin flows after activation.

Pull request overview

This PR refreshes the plugin’s admin settings screen to a more WordPress-native UI, modernizes the “copy details” interaction, and updates plugin branding/activation behavior to better guide users to the settings page.

Changes:

  • Rebuilds the settings screen markup into a header + notice area + card-based layout, and adds a “Copy all details” button with feedback.
  • Updates copy-to-clipboard behavior to use navigator.clipboard with a legacy fallback and a “Copied!” label.
  • Adds an activation redirect to the settings page and loads admin CSS/JS for the settings screen regardless of “parrot user” status.
File summaries
File Description
pirate-parrot.php New admin screen layout, activation redirect, updated status/notice rendering, and details rendering/copy payload preparation.
inc/js/parrot.js Adds copy-to-clipboard handling and “Copied!” UI feedback for the new details button.
inc/css/parrot.css Adds new WP-native card/header/status/button styling for the refreshed settings screen.

Copilot's findings

  • Files reviewed: 3/4 changed files
  • Comments generated: 5

Note

Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.


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

Comment thread pirate-parrot.php
Comment thread pirate-parrot.php
Comment thread pirate-parrot.php Outdated
Comment thread inc/js/parrot.js
Comment thread inc/js/parrot.js Outdated
rodica-andronache and others added 5 commits June 16, 2026 12:25
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@rodica-andronache

Copy link
Copy Markdown
Contributor Author

@copilot check why this PR can't be merged

Copilot AI commented Jun 18, 2026

Copy link
Copy Markdown

@copilot check why this PR can't be merged

PR #31 is currently blocked because a required check is still running, not failing. The only check on the head commit is copilot (workflow run 27760792851), and its status is in_progress, so GitHub keeps mergeable_state=blocked until it completes.

@rodica-andronache

Copy link
Copy Markdown
Contributor Author

@copilot, what is the required check that is still running? Where can i see it's status and change it?

Copilot AI commented Jun 18, 2026

Copy link
Copy Markdown

@copilot, what is the required check that is still running? Where can i see it's status and change it?

The required check still running is copilot (current run: 27761055123, job: 82135261473).

You can see it in:

What you can change:

  • On that Actions run page: Re-run jobs or Cancel workflow (if you have permission).
  • Whether this check is required is set in repo settings: Settings → Branches → Branch protection rule → Required status checks (admin/maintainer permission needed).

@rodica-andronache rodica-andronache merged commit b5614e0 into master Jun 19, 2026
2 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.

4 participants