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

[Feature request] More cards per row on larger screens (for items result list) #1401

Closed
arittner opened this issue Feb 21, 2022 · 6 comments
Closed

Comments

@arittner
Copy link

Hi!

Currently, the responsive layout shows 4 cards in a row (for documents) and reduce it to 1 card (for smaller windows widths).

It would be very nice, if it's possible to define a higher count of cards for a row (e.g. up to 8). The gold standard would be to have 2 different config values: One for "full-height-view" and one for the "top-page" preview. :-)

I guess many users will have monitors with 1920px resolution and stay with the 32dpi preview image settings. And 4 cards only on a full window shows only cluttered images. Anyway, a 1920px resolution is able to show up to 8 cards in a row without big issues. If a user can configure this, it would be a very useful option to accommodate a lot of information in really wide windows.

@eikek
Copy link
Owner

eikek commented Feb 21, 2022

Ah yes, agree. I think for a first quick win, I can add more breakpoints and set different columns for larger screens. The default dpi size could also be increased maybe. I was rather conservative here, thinking it's just a thumbnail :-)

I just tried 8 cards on 1920px wide and it looked quite narrow to me. I found it better with 6 cards? But I would increase it for larger widths as well, I think very wide monitors are also quite popular. Here are some screenshots with width 1920 (simulated in firefox on a larger screen) with 5,6,7 and 8 cards. I think when items have some number of tags and custom fields, a bit more extra width is quite nice.

I think two different options is a bit too much, no? Might be ok to add later a config for a column count per screen size maybe.

Screen Shot 2022-02-21 at 22 09 11
Screen Shot 2022-02-21 at 22 09 39
Screen Shot 2022-02-21 at 22 09 58
Screen Shot 2022-02-21 at 22 10 14

@arittner
Copy link
Author

I just tried 8 cards on 1920px wide and it looked quite narrow to me. I found it better with 6 cards?

Yes 6 cards looks ok for me. I compared it wit paperless-ng, but docspell shows much more information. So 6 cards are fine. :)

I think two different options is a bit too much, no? Might be ok to add later a config for a column count per screen size maybe.

I think with 6 cards it works for both page view modes.

Thank you for your efforts!

@eikek
Copy link
Owner

eikek commented Feb 22, 2022

I tried a simple improvement which could be tried using the nightly release (but use a separate database, it contains db migrations that can't be rolled back).

@arittner
Copy link
Author

Ok, I'll try it :-) - thank you!

@arittner
Copy link
Author

Phew, after struggling with some config issues, I put together a docker compose script that got me the nightlies working with a h2 DB together. Somehow, I can't cope with the idea that Joex and Docspell rest server need the same DB (but I come from the microservice world)

However, now it's working, I was able to test it, and it looks fine. The cards are responding to browser size changes and zoom changes. In both cases I get more than 4 cards. Perfect :-)

@eikek
Copy link
Owner

eikek commented Feb 24, 2022

Nice, thanks for testing it! Sorry for the trouble 😬 . I have a bash script for this that I could check in (not using docker, just starting it up).

Somehow, I can't cope with the idea that Joex and Docspell rest server need the same DB (but I come from the microservice world)

The database is used as a synchronization point. It's possible to run multiple instances up to a point. It is a simpler setup, given it's some personal tool after all, not "enterprise ready". I usually hear the opposite opinion: that it is already too much to setup :)

@eikek eikek added this to the Docspell 0.33.0 milestone Feb 24, 2022
@eikek eikek changed the title [Feature request] Add a layout option for cards per row (for items result list) [Feature request] More cards per row on larger screens (for items result list) Feb 26, 2022
@eikek eikek closed this as completed Mar 1, 2022
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

No branches or pull requests

2 participants