Skip to content

Conversation

@sjbur
Copy link
Contributor

@sjbur sjbur commented Oct 29, 2025

No description provided.

@sjbur sjbur changed the title Appointment Form Redesign: Update Appointment Form icons [2] Appointment Form Redesign: Update Appointment Form icons Oct 29, 2025
@sjbur sjbur self-assigned this Oct 29, 2025
@sjbur sjbur added the 25_2 label Oct 29, 2025
@sjbur sjbur marked this pull request as ready for review October 29, 2025 09:05
@sjbur sjbur requested a review from a team as a code owner October 29, 2025 09:05
Copilot AI review requested due to automatic review settings October 29, 2025 09:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates icon names in the DevExtreme scheduler appointment popup to align with a new icon set. The changes replace deprecated icon names ('back', 'preferences') with their modern equivalents ('arrowleft', 'optionsoutline').

Key changes:

  • Icon name updates in scheduler appointment popup components
  • Test selector updates to match new icon names
  • Screenshot baseline updates reflecting visual changes from new icons

Reviewed Changes

Copilot reviewed 3 out of 32 changed files in this pull request and generated no comments.

File Description
m_popup.ts Changed back button icon from 'back' to 'arrowleft'
m_form.ts Changed recurrence settings icon from 'preferences' to 'optionsoutline'
popup.ts (test mock) Updated test selector from aria-label="Back" to aria-label="arrowleft"
*.png (etalons) Updated screenshot baselines for visual regression tests

@sjbur sjbur changed the title [2] Appointment Form Redesign: Update Appointment Form icons Appointment Form Redesign: Update Appointment Form icons Oct 29, 2025
@sjbur sjbur merged commit 133bbea into DevExpress:25_2 Oct 29, 2025
147 of 148 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants