-
-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
Feat/optimize span svg layout #34185
Feat/optimize span svg layout #34185
Conversation
I agree, this seems to be too prone to false-positives. I still think the best course of actions is to make Vue not output the wrapper in first place: |
The screenshots are not from Vue ...... |
yes, screenshots from gitea/templates/explore/repo_list.tmpl Line 47 in 946fb78
There is a lot of legacy code in the system, such as The goal of this PR is to make the SVG and the subsequent text vertically centered in all such scenarios across the entire system. |
See #34185 (comment), it won't work in all scenarios, instead, it introduces more different cases. The root problem is that you can't assume the parent has "flex" layout. |
946fb78
to
2beacc5
Compare
I did a global search, and it turns out the situation is much more complicated than I thought. I believe there might not be a universal solution. It seems that we really have no other choice but to address each issue as it arises. |
before:

after:
