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

Upload dialog: improve preview icon #6424

Merged
merged 10 commits into from
May 3, 2024

Conversation

distantnative
Copy link
Member

@distantnative distantnative commented Apr 30, 2024

This PR …

Enhancements

  • Upload dialog: better preview icons

For review team

  • Add changes & docs to release notes draft in Notion

@distantnative distantnative added the type: enhancement ✨ Suggests an enhancement; improves Kirby label Apr 30, 2024
@distantnative distantnative added this to the 4.3.0 milestone Apr 30, 2024
@distantnative distantnative requested a review from a team April 30, 2024 16:23
@distantnative distantnative self-assigned this Apr 30, 2024
@distantnative distantnative marked this pull request as draft April 30, 2024 17:36
@distantnative distantnative removed the request for review from a team April 30, 2024 17:36
@distantnative distantnative force-pushed the enhancement/upload-preview-icon branch from d8e9611 to 06633ca Compare April 30, 2024 17:40
@distantnative distantnative marked this pull request as ready for review April 30, 2024 17:41
@distantnative distantnative force-pushed the enhancement/upload-preview-icon branch from 06633ca to 34927ce Compare April 30, 2024 17:43
@bastianallgeier
Copy link
Member

I've used the chance to refactor the items into their own components. It adds the massive benefit that we can properly test them all in the lab.

@distantnative
Copy link
Member Author

You forgot the UploadReplaceDialog 😥😅

@bastianallgeier
Copy link
Member

Really happy how all of this turned out.

@bastianallgeier bastianallgeier merged commit c2bb45e into develop-minor May 3, 2024
4 checks passed
@bastianallgeier bastianallgeier deleted the enhancement/upload-preview-icon branch May 3, 2024 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement ✨ Suggests an enhancement; improves Kirby
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants