Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(admin-form): copy form shortcode to clipboard via button #2602 #3425

Closed
wants to merge 113 commits into from

Conversation

Sidsector9
Copy link
Contributor

Closes #2602

Description

This PR replaces the old shortcode display with a shortcode button which on clicking will copy the shortcode.

How Has This Been Tested?

By clicking the shortcode button on both the donations listing page and on the form meta page.

Screenshots (jpeg or gifs if applicable):

Shortcode Button

shortcodebutton

Shortcode button on hover

shortcodebuttonhover

Shortcode button on click

shortcodebuttonclick

Checklist:

  • My code is tested.
  • My code follows the WordPress code style.
  • My code follows has proper inline documentation.

Sidsector9 and others added 30 commits May 31, 2018 16:46
# Conflicts:
#	includes/admin/upgrades/upgrade-functions.php
#	includes/install.php
emgk and others added 27 commits June 26, 2018 15:10
# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
fix(donation-form): reduced code redundancy by creating helper function.
refactor(donor): create singular method to delete donor #2347
fix(admin-tools): export the donation of current year only.
fix(libraries): resolve tcpdf incompatibility with PHP 7.2 #3411
@Sidsector9 Sidsector9 changed the base branch from release/2.1.7 to release/2.3 June 29, 2018 15:12
@Sidsector9 Sidsector9 closed this Jun 29, 2018
@DevinWalker DevinWalker deleted the issue/2602 branch July 24, 2018 22:06
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.

None yet

7 participants