Jobs Theme: Fix multiple issues in redesign#576
Closed
dd32 wants to merge 11 commits intoWordPress:trunkfrom
Closed
Jobs Theme: Fix multiple issues in redesign#576dd32 wants to merge 11 commits intoWordPress:trunkfrom
dd32 wants to merge 11 commits intoWordPress:trunkfrom
Conversation
Replaces the dated 996-grid layout with a modern CSS custom properties design system featuring a hero section with dynamic stats, category filter pills, card-based job grid, sticky header, and dark footer. Fixes PHP warnings: array_pop by-reference, deprecated wp_no_robots. Updates local environment setup to deduplicate pages, add FAQ page, clear default widgets, and set post_author. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Removes old 996 grid CSS, dashicons CSS, layout CSS, rtl.css, wporg-footer.php, content.php, custom-header/customizer/jetpack includes, and old JS files (navigation, customizer, keyboard nav). Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Use proper escaping functions (esc_html_e, wp_kses_post, wp_kses), fix indentation to use tabs consistently, avoid inline control structures, and rename $cat to $job_cat to avoid overriding WP globals. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Correct indentation in content-list.php to match scope level expected by PHPCS, and use ternary instead of inline brace in header.php. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Show How to Apply links correctly (Via email/web form) by using wp_kses with explicit allowed tags, since the jobswp plugin strips <a> globally - Add empty state message when filtering by category with no matching jobs - Highlight active FAQ nav item in header - Account for wp-admin bar height in sticky header positioning - Replace malformed WordPress logo SVG with official version - Add Remove a Job and Feedback links to footer - Add remove-a-job note to post-a-job form - Use filemtime() for asset cache busting - Add howtoapply metadata to sample test data Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…ck page to test data. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Override hardcoded inline widths on the form container, inputs, and textarea so the feedback form fills available space properly. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
<a>fromwp_kses_allowed_htmlfilemtime()for asset cache busting instead of hardcoded date stringshowtoapplyandhowtoapply_methodmetadata to sample test dataTest plan
🤖 Generated with Claude Code