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

WIP: Fixes #2182 with a more compact display of file browser items: #2205

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

halfdane
Copy link

  • Introduces a preference to enable "compact view" of file browser items
  • if preference is enabled, filename an description are layed out horizontally instead of vertically

- Introduces a preference to enable "compact view" of file browser items
- if preference is enabled, filename an description are layed out
  horizontally instead of vertically
@harshad1
Copy link
Collaborator

Could you add a before and after screenshot here?

Comment on lines +51 to +53
<Space
android:layout_width="8dp"
android:layout_height="wrap_content" />
Copy link
Owner

@gsantner gsantner Feb 16, 2024

Choose a reason for hiding this comment

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

Makes default vertical layout 8dp bigger by adding this spacer. If it's really needed for horiziontal, hide it in vertical please.

Generally I'm not a fan to add a own settings option just so everyone can disable every tiny bit they personally dislike. This is bloat.

Copy link
Collaborator

Choose a reason for hiding this comment

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

To add to this.

IMO if we can come up with a slightly more compact clean layout which doesn't compromise anything we should just switch to that as the new default.

@gsantner
Copy link
Owner

gsantner commented Mar 27, 2024

Sorry to say, but I don't want to merge/implement this. I know some people would like every tiny bit finetuned to their personal taste, but I want to add to-my-feeling unnecessary things.

There is a certain size that works fine for fingers/thumbs (""minimum viable button touch width/height""). The current size seems quite fitting, it shouldn't decrease too much then it's currently. If you find additional ways to improve it, feel free to. But it should be a general improvement that fits without a settings option (and not slams everything into one line). Like adjusting margins or so.

@gsantner gsantner changed the title Fixes #2182 with a more compact display of file browser items: WIP: Fixes #2182 with a more compact display of file browser items: Mar 27, 2024
@gsantner gsantner marked this pull request as draft March 27, 2024 03:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants