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

Fix mobile view #393

Closed
wants to merge 2 commits into from
Closed

Fix mobile view #393

wants to merge 2 commits into from

Conversation

Dennis1993
Copy link
Contributor

Fix for #385 and #361

Signed-off-by: Dennis1993 Dennis1993@users.noreply.github.com

Fix for #385 and #361

Signed-off-by: Dennis1993 <Dennis1993@users.noreply.github.com>
Fix for #385 and #361

Signed-off-by: Dennis1993 <Dennis1993@users.noreply.github.com>
@Dennis1993
Copy link
Contributor Author

Before:
image

After:
image

@Dennis1993 Dennis1993 requested a review from come-nc September 5, 2022 11:52
@come-nc come-nc requested a review from szaimen September 5, 2022 12:51
@szaimen szaimen added this to the Nextcloud 25 milestone Sep 6, 2022
@szaimen
Copy link
Contributor

szaimen commented Sep 6, 2022

Hm... not really fixed if you have multiple storages in my testing:

Click here

image

For my own testing
docker run -it \
-p 8444:443 \
-e TRUSTED_DOMAIN=10.6.0.1 \
-e SERVERINFO_BRANCH=Dennis1993-patch-1 \
-v /:/host/ \
--name nextcloud-easy-test \
ghcr.io/szaimen/nextcloud-easy-test:latest

@Dennis1993
Copy link
Contributor Author

@szaimen Thanks, I've created a new patch, a css code is missing here.
Thanks for your support!

@Dennis1993 Dennis1993 closed this Sep 12, 2022
@kesselb kesselb deleted the Dennis1993-patch-1 branch June 10, 2024 11:06
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.

2 participants