Automattic/jetpack

Feature request: Add filter for [portfolio]-Shortcode output

Open

#2,194 opened on Jun 1, 2015

View on GitHubContributor guide
 (10 comments) (4 reactions) (0 assignees)PHP (877 forks)user submission
EnhancementGood For CommunityHacktoberfest[Feature] Custom Content Types[Type] Good First Bug

Repository metrics

Stars
 (1,808 stars)
PR merge metrics
 (Avg merge 4d 5h) (645 merged PRs in 30d)

Description

Hello there!

It would be nice to have a filter for the shortcode function output of [portfolio]. I was searching the jetpack plugin files, but could not find any filter.. so after Jeremy's suggestion I figured I'd open a github issue for this matter. The related wordpress.org-support thread is the following: https://wordpress.org/support/topic/customizing-jetpack-portfolio-shortcode

I'd like to control the html output of the shortcode to be able to add a div if needed or implement a custom field I might have registered for my portfolio post types. For example I set up a custom field which allows me to insert a client logo. I would like to have this client logo (added via custom field in portfolio cpt) to be displayed with the portfolio item's title. I would like to be able to control where exactly this custom field shows up and which css classes are spread out.

A filter would allow such for the [portfolio] output.

Thanks

Contributor guide

Research direction
Locate the portfolio shortcode function in Jetpack's codebase and add a filter hook to allow customizing the HTML output. Identify the function that outputs the shortcode, then apply a WordPress filter to the output so that users can modify it via themes or plugins.
Tech stack
php
Domain
backend
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
PHPWordPress
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
75