Skip to content

Add option to testimonials shortcode to show full testimonial#2815

Merged
dereksmart merged 8 commits intomasterfrom
fix-issue-2814
Oct 28, 2015
Merged

Add option to testimonials shortcode to show full testimonial#2815
dereksmart merged 8 commits intomasterfrom
fix-issue-2814

Conversation

@chaselivingston
Copy link
Copy Markdown
Contributor

Add the show_excerpt attribute to the testimonials shortcode to give users the option to show the full testimonial. Set to true by default.

Fixes #2814

@chaselivingston chaselivingston added [Status] In Progress Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] Shortcodes / Embeds [Feature] Custom Content Types Custom post or content types (usually for testimonials and portfolios) and their settings. and removed [Status] In Progress labels Oct 6, 2015
@chaselivingston chaselivingston self-assigned this Oct 6, 2015
@jeherve jeherve added this to the Community milestone Oct 7, 2015
@jeherve jeherve added [Status] Needs Review This PR is ready for review. tdiv labels Oct 7, 2015
@jeherve
Copy link
Copy Markdown
Member

jeherve commented Oct 8, 2015

If we make that change, it might make sense to do the same thing for the Portfolio shortcode as well, to keep things consistent.

@chaselivingston chaselivingston modified the milestones: 3.8, Community Oct 13, 2015
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.

You'll either need to add a closing php tag here, or omit the opening one in the line below

@dereksmart dereksmart added [Status] In Progress and removed [Status] Needs Review This PR is ready for review. labels Oct 21, 2015
@dereksmart dereksmart added [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Oct 28, 2015
@dereksmart
Copy link
Copy Markdown
Contributor

talked with @chaselivingston and we're going with a new argument for display_content rather than adding a whole new attribute

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.

Where does portfolio-entry get closed now?

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.

Looks like it doesn't :/

@zinigor
Copy link
Copy Markdown
Contributor

zinigor commented Oct 28, 2015

👍 Looks good now!

dereksmart added a commit that referenced this pull request Oct 28, 2015
Add option to testimonials shortcode to show full testimonial
@dereksmart dereksmart merged commit d480ccd into master Oct 28, 2015
@dereksmart dereksmart deleted the fix-issue-2814 branch October 28, 2015 16:32
@dereksmart dereksmart removed the [Status] Ready to Merge Go ahead, you can push that green button! label Oct 28, 2015
@danstramer
Copy link
Copy Markdown

danstramer commented Mar 29, 2018

Looked at the php file now and saw: display_content=full.
documentation here:
https://en.support.wordpress.com/testimonials-shortcode/
Dan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement Changes to an existing feature — removing, adding, or changing parts of it [Feature] Custom Content Types Custom post or content types (usually for testimonials and portfolios) and their settings. [Feature] Shortcodes / Embeds Touches WP.com Files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants