Development#2841
Merged
Merged
Conversation
Introduce a public nullable callable property `$map_card_renderer` on `Directorist_Listings`. When set, it is invoked instead of the default template for each map marker card — in both `openstreet_map_card_data()` and `load_google_map()`. This gives integrations (e.g. page builders, themes) a clean, zero-overhead way to replace the map popup markup without relying on global template-path filters or shared static state. Signature: callable( int $listing_id, array $opt ): string The property defaults to null so all existing behaviour is fully preserved.
Consolidates thumb-card template and loop_get_the_thumbnail() into a single static call — no Directorist_Listings instantiation needed. Key improvements over the previous approach: - Zero class instantiation, zero setup_postdata, zero global state - Attachment IDs fetched once and reused for both the front image and the optional blur-background layer (previously fetched twice) - Accepts a listing ID directly instead of relying on get_the_ID() Helper::get_listing_card_image( int $listing_id ): string Helper::listing_thumbnail_img( ... ): string (private, handles single image, multiple images/swiper, and fallback default image)
This reverts commit 22ce4a9.
- Change "See why" trigger from <a> to <button> with ARIA attributes - Add Escape key handler and aria-expanded toggling - Add fallback message when no rejection reason exists - Replace hardcoded colors with CSS custom properties - Add RTL support using logical properties (margin-inline-start, inset-inline-start) - Add mobile responsive bottom-sheet popover for viewports <= 576px - Add scrollable body for long rejection reasons (max-height: 200px) - Add esc_url() on thumbnail src and esc_html() on rejection email placeholder - Remove unnecessary esc_sql() on dashboard search (WP_Query handles this)
After install/activate/update actions trigger location.reload(), the tab always re-initialized with Extensions active regardless of which tab the user had open. Write the active tab's data-target to sessionStorage on click and restore it on load, scoped per tab group via the data-parent attribute.
Move preview publish nonce generation to the preview page so guest submissions use the authenticated session after account creation. Also add an edit capability check before updating listing status after review.
Improve: Dashboard status badges and rejection popover UX
…persistence fix: restore active tab after page reload on extensions & themes page and Remove PR Review WorkFlow
…-screen-fix Fix: Remove legacy help support screen fix
feat(Listings): add map_card_renderer extension point
…elementor-activation-notice Remove deprecated Elementor activation notice
…ngs-link-navigation Fix: “Go to Settings” button redirecting to wrong page and stripping subdirectory path
Add Nearby sort option for location-based listing results
Improve: Reset Tag Default Style for Single Listing Content
removed help & support menu
Feature/ reject status
Fix guest preview publish nonce flow
Readme updated & build
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
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.
PR Type
What kind of change does this PR introduce?
Description
How to reproduce the issue or how to test the changes
Any linked issues
Fixes #
Checklist