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 Safari spinner rendering #29801

Merged
merged 2 commits into from
Mar 14, 2024
Merged

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Mar 14, 2024

Fixes: #29041
Fixes: #29713

Any of the width: *-content properties seem to workaround this Webkit bug, this one seemed most suitable.

Before:
Screenshot 2024-03-14 at 22 29 58

After:
Screenshot 2024-03-14 at 22 30 30

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Mar 14, 2024
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Mar 14, 2024
@silverwind silverwind added type/bug backport/v1.21 This PR should be backported to Gitea 1.21 and removed modifies/frontend labels Mar 14, 2024
Copy link
Member

@denyskon denyskon left a comment

Choose a reason for hiding this comment

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

This change is straight-forward, but I have no ability to test on Safari, trusting you that it fixes the issue😆

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Mar 14, 2024
@silverwind
Copy link
Member Author

Tested in Safari 17.4 😉

@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Mar 14, 2024
@techknowlogick techknowlogick enabled auto-merge (squash) March 14, 2024 21:56
@techknowlogick techknowlogick added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Mar 14, 2024
@techknowlogick techknowlogick merged commit 35def31 into go-gitea:main Mar 14, 2024
26 checks passed
@GiteaBot GiteaBot added this to the 1.23.0 milestone Mar 14, 2024
GiteaBot pushed a commit to GiteaBot/gitea that referenced this pull request Mar 14, 2024
Fixes: go-gitea#29041
Fixes: go-gitea#29713

Any of the `width: *-content` properties seem to workaround this Webkit
bug, this one seemed most suitable.
@GiteaBot GiteaBot added backport/done All backports for this PR have been created and removed reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. labels Mar 14, 2024
@silverwind silverwind added the backport/v1.20 This PR should be backported to Gitea 1.20 label Mar 14, 2024
@silverwind silverwind deleted the safarispin branch March 14, 2024 22:07
@silverwind
Copy link
Member Author

Not sure if 1.20 will see more releases, but this should be a clean merge 😆

silverwind added a commit that referenced this pull request Mar 14, 2024
Backport #29801 by @silverwind

Fixes: #29041
Fixes: #29713

Any of the `width: *-content` properties seem to workaround this Webkit
bug, this one seemed most suitable.

Before:
<img width="184" alt="Screenshot 2024-03-14 at 22 29 58"
src="https://github.com/go-gitea/gitea/assets/115237/6effc5f0-bc64-4752-be74-9c43b3974407">

After:
<img width="177" alt="Screenshot 2024-03-14 at 22 30 30"
src="https://github.com/go-gitea/gitea/assets/115237/5de244d7-6b46-428e-957c-4b10f53e2441">

Co-authored-by: silverwind <me@silverwind.io>
@silverwind
Copy link
Member Author

Ah, I think I may have confused the bot. Well, I assume 1.20 isn't going to get any more releases.

@silverwind silverwind removed the backport/v1.20 This PR should be backported to Gitea 1.20 label Mar 14, 2024
zjjhot added a commit to zjjhot/gitea that referenced this pull request Mar 15, 2024
* giteaofficial/main:
  Update JS dependences (go-gitea#29797)
  Unify search boxes (go-gitea#29530)
  Fix document error about 'make trans-copy' (go-gitea#29710)
  Remove jQuery AJAX from the diff functions (go-gitea#29743)
  Fix Safari spinner rendering (go-gitea#29801)
  Remove jQuery AJAX from the `repo-issue.js` file (go-gitea#29776)
  Improve commit record's ui in comment list (go-gitea#26619)
  enable tailwind nesting (go-gitea#29746)
@6543 6543 modified the milestones: 1.23.0, 1.22.0 Mar 16, 2024
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Jun 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created backport/v1.21 This PR should be backported to Gitea 1.21 lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Circular loading spinner is an oval on mobile
5 participants