Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(assets): add save icon #1612

Merged
merged 1 commit into from
Mar 28, 2024
Merged

feat(assets): add save icon #1612

merged 1 commit into from
Mar 28, 2024

Conversation

XL-YiBai
Copy link
Contributor

@XL-YiBai XL-YiBai commented Mar 27, 2024

PR Checklist

If you have any questions, you can refer to the Contributing Guide

What is the current behavior?

Please describe the current behavior and link to a relevant issue.

Issue Number

Example: #123

What is the new behavior?

Please describe the new behavior or provide screenshots.

Does this PR introduce a breaking change?

  • Yes
  • No

Specific Instructions

Are there any specific instructions or things that should be known prior to review?

Other information

image

@XL-YiBai XL-YiBai marked this pull request as ready for review March 27, 2024 16:40
@XL-YiBai XL-YiBai changed the title feat(assets): update iconfont feat(assets): add save icon Mar 28, 2024
Copy link
Member

@ysfscream ysfscream left a comment

Choose a reason for hiding this comment

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

LGTM

@ysfscream ysfscream added desktop MQTTX Desktop enhancement New feature or request UI/UX Improve some UI \ UX web MQTTX Web labels Mar 28, 2024
@ysfscream ysfscream added this to the v1.9.10 milestone Mar 28, 2024
@ysfscream ysfscream merged commit f51fc50 into emqx:main Mar 28, 2024
4 checks passed
@XL-YiBai XL-YiBai deleted the my/main branch March 28, 2024 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
desktop MQTTX Desktop enhancement New feature or request UI/UX Improve some UI \ UX web MQTTX Web
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants