Skip to content

Development#2841

Merged
Armanul46 merged 63 commits into
trunkfrom
development
Apr 30, 2026
Merged

Development#2841
Armanul46 merged 63 commits into
trunkfrom
development

Conversation

@Armanul46
Copy link
Copy Markdown
Collaborator

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Security fix
  • Improvement
  • New Feature
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Text changes
  • Other... Please describe:

Description

How to reproduce the issue or how to test the changes

Any linked issues

Fixes #

Checklist

RabbiIslamRony and others added 30 commits February 10, 2026 15:26
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)
- 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
nazmulhasan103 and others added 28 commits April 27, 2026 11:39
…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
Copy link
Copy Markdown
Member

@RabbiIslamRony RabbiIslamRony left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good

@Armanul46 Armanul46 merged commit 6f3c315 into trunk Apr 30, 2026
2 checks passed
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.

3 participants