Skip to content

ColorBox: Update no-color image with colordismiss icon#30825

Merged
r-farkhutdinov merged 8 commits intoDevExpress:25_2from
r-farkhutdinov:25_2_color_box_icon
Sep 2, 2025
Merged

ColorBox: Update no-color image with colordismiss icon#30825
r-farkhutdinov merged 8 commits intoDevExpress:25_2from
r-farkhutdinov:25_2_color_box_icon

Conversation

@r-farkhutdinov
Copy link
Copy Markdown
Contributor

No description provided.

@r-farkhutdinov r-farkhutdinov marked this pull request as ready for review September 1, 2025 19:19
@r-farkhutdinov r-farkhutdinov requested review from a team as code owners September 1, 2025 19:19
Comment on lines +301 to +302
this._$noColorIcon?.remove();
this._$noColorIcon = undefined;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems like this can be extracted to a separate function like cleanNoColorIcon or smth similar

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thank you. fixed in: 3c1fc11

@r-farkhutdinov r-farkhutdinov merged commit 12c3abc into DevExpress:25_2 Sep 2, 2025
445 of 447 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants