feat: more horizontal space on mobile #2722
No reviewers
Labels
No Label
area/internal-code
changes requested
confirmed
dependencies
duplicate
good first issue
help wanted
hosting
invalid
kind/bug
kind/feature
question
wontfix
No Milestone
No project
No Assignees
3 Participants
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: vikunja/frontend#2722
Loading…
Reference in New Issue
No description provided.
Delete Branch "dpschen/frontend:feature/more-horizontal-space-on-mobile"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
By adding less outer padding we keep more screen space. This is important especially for small screens.
I assumed that in the task view the background of the list is not that important
Hi dpschen!
Thank you for creating a PR!
I've deployed the changes of this PR on a preview environment under this URL: https://2722-feature-more-horizontal-space-on--vikunja-frontend-preview.netlify.app
You can use this url to view the changes live and test them out.
You will need to manually connect this to an api running somehwere. The easiest to use is https://try.vikunja.io/.
Have a nice day!
@ -169,3 +165,2 @@
margin-left: 0;
padding-top: 1.5rem;
min-height: calc(100vh - 4rem);
min-height: calc(100dvh - 4rem);
dvh
are a nice solution here but the browser support is really poor. We should at least add a fallback, maybe even not using them.We should, because support should soon be better. And this makes really sense for iPhone. But you're right! I thought that postcss-preset-env adds
vh
as fallback automatically, but probably they decided against it becausedvh
is not 100% the same asvh
, but instead, what you would actually expectvh
to be. :DAfter further thinking I removed
dvh
now. Not because it's wrong, but because it's not used anywhere else. And since some height calculations depend on each other it might break something else if I use it only here.I might add
dvh
as mobile optimization at some point. But then everywhere where it's needed at the same time.ebcc0db1e7
to4c03adb6e2