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: Use the blurhash in Files #47605

Merged
merged 3 commits into from
Aug 29, 2024
Merged

feat: Use the blurhash in Files #47605

merged 3 commits into from
Aug 29, 2024

Conversation

artonge
Copy link
Contributor

@artonge artonge commented Aug 29, 2024

List view Grid view
Screenshot from 2024-08-29 14-30-30 Screenshot from 2024-08-29 14-30-26

@artonge artonge requested a review from skjnldsv as a code owner August 29, 2024 12:28
@artonge artonge self-assigned this Aug 29, 2024
@artonge artonge added this to the Nextcloud 31 milestone Aug 29, 2024
@artonge artonge force-pushed the artonge/feat/files_blurhash branch from c025414 to 7019db8 Compare August 29, 2024 12:31
@artonge
Copy link
Contributor Author

artonge commented Aug 29, 2024

/compile amend /

Copy link
Contributor

@susnux susnux left a comment

Choose a reason for hiding this comment

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

Looks super nice! Just a few a11y and TS comments :)

@artonge artonge force-pushed the artonge/feat/files_blurhash branch from 7019db8 to 36dfaae Compare August 29, 2024 12:46
@nextcloud-command nextcloud-command force-pushed the artonge/feat/files_blurhash branch from 36dfaae to df60ea0 Compare August 29, 2024 12:47
@artonge artonge force-pushed the artonge/feat/files_blurhash branch from df60ea0 to be57ebb Compare August 29, 2024 14:04
@artonge artonge force-pushed the artonge/feat/files_blurhash branch from be57ebb to b7bdfce Compare August 29, 2024 15:23
@artonge artonge enabled auto-merge August 29, 2024 15:49
artonge and others added 2 commits August 29, 2024 23:08
Signed-off-by: Louis Chemineau <louis@chmn.me>
Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
@susnux susnux force-pushed the artonge/feat/files_blurhash branch from e388b7f to 75a684a Compare August 29, 2024 21:08
@susnux
Copy link
Contributor

susnux commented Aug 29, 2024

/compile

Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
@artonge artonge merged commit 1cc7851 into master Aug 29, 2024
116 checks passed
@artonge artonge deleted the artonge/feat/files_blurhash branch August 29, 2024 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ☑️ Done
Development

Successfully merging this pull request may close these issues.

5 participants