-
Notifications
You must be signed in to change notification settings - Fork 169
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
Aesthetics improvements #424
Conversation
pipe01
commented
Jun 5, 2023
- Disabled autocorrection in the login form's instance field.
- Added some padding to the top of the login button.
- Set the status and navigation bar colors depending on the chosen theme.
- Also set the navigation color depending on the bottom app bar visibility.
Could you please re-trigger the build? We can now see the logs. |
Not sure how to do that, sorry. |
Just force push after having made a change to the commit (with |
Well, now you get a proper error output to work with :P Good luck! |
I don't see any errors though, only warnings... It seems like gradle crashed? |
From the logs:
Yeah you're right. I suppose the build is too heavy/long for the current infra and it was either OOM killed or terminated because of execution time. Edit: It turns out it crashed because of exhausted drive space (or inodes?) |
Oops lol, I'm just not sure what the issue is. It builds just fine on my machine. |
I deleted my previous comment because you re-triggering the build actually disclosed what the problem is: But for now, until someone goes fix it on the host, nobody can pass any CI checks. |
@twizmwazin Hey, do you think you could re-trigger this job on the CI please? :) |
I don't think I have any special access to do that. |
There we go, now it's working |
Yeah I suspect that someone in the infra team for Lemmy.ml logged on the VPS and made some room on the drive. PS: you might want to re-sync your PR with upstream... $ git remote add upstream https://github.com/dessalines/jerboa.git
$ git fetch upstream
$ git rebase upstream/main
$ git push -f |
Sorry about the CI issues all, our runners have been pretty overloaded. Testing this now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, thanks!
Yaay! :) |