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: gallery view poc #6584

Draft
wants to merge 7 commits into
base: main
Choose a base branch
from
Draft

Conversation

Xazin
Copy link
Collaborator

@Xazin Xazin commented Oct 20, 2024

I built a version/proof of concept of a Gallery View for Databases. This is not on the roadmap, so it is not prioritized at the moment.

Relates: #3827

My current concern is that I am currently using ReorderableWrap, but how will this fare with hundreds of items performance wise?

Some nice to haves:

  • Would be nice for all items in a row fit the largest items height to match.

To-dos:

  • Insert default cards in Gallery on creation (same as grid)
  • Add actions on card show up on hover (similar to in board)
  • Tests

Feature Preview

Screen.Recording.2024-10-20.at.14.1.mp4

PR Checklist

  • My code adheres to AppFlowy's Conventions
  • I've listed at least one issue that this PR fixes in the description above.
  • I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
  • All existing tests are passing.

Copy link

github-actions bot commented Oct 20, 2024

🥷 Ninja i18n – 🛎️ Translations need to be updated

Project /project.inlang

lint rule new reports level link
Missing translation 108 warning contribute (via Fink 🐦)

@Xazin
Copy link
Collaborator Author

Xazin commented Oct 20, 2024

Added the hover action menu

Optagelse.2024-10-20.172651.mp4

@Xazin
Copy link
Collaborator Author

Xazin commented Oct 20, 2024

Requires AppFlowy-IO/AppFlowy-Collab#320

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant