Skip to content

Feature/ reject status#2785

Open
Armanul46 wants to merge 17 commits intosovware:developmentfrom
Armanul46:feature/reject-status
Open

Feature/ reject status#2785
Armanul46 wants to merge 17 commits intosovware:developmentfrom
Armanul46:feature/reject-status

Conversation

@Armanul46
Copy link
Copy Markdown
Collaborator

@Armanul46 Armanul46 commented Apr 5, 2026

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

  1. https://sejim.notion.site/PRD-Listing-Rejection-Flow-for-Directorist-31f0076360e880619d36d7eb8dec86d2

Any linked issues

Fixes #

Checklist

Armanul46 and others added 17 commits April 5, 2026 16:31
- 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)
Improve: Dashboard status badges and rejection popover UX
@Armanul46 Armanul46 added this to the v8.7.1 milestone Apr 27, 2026
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.

2 participants