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

Dropdown triangle fixes #14028

Merged
merged 4 commits into from
Dec 17, 2020
Merged

Dropdown triangle fixes #14028

merged 4 commits into from
Dec 17, 2020

Conversation

silverwind
Copy link
Member

Fomantic's CSS assumes the dropdown triangles are fonts but we've recently swapped them for SVG so we need to wrestle a bit with Fomantic to get it to display properly and not cause buttons to resize.

Fixes: #14014

Screen Shot 2020-12-16 at 22 10 24

Screen Shot 2020-12-16 at 22 10 34

Screen Shot 2020-12-16 at 22 11 07

Screen Shot 2020-12-16 at 22 13 08

Screen Shot 2020-12-16 at 22 15 48

Fomantic's CSS assumes the dropdown triangles are fonts but we've recently
swapped them for SVG so we need to wrestle a bit with Fomantic to get it
to display properly and not cause buttons to resize.

Fixes: go-gitea#14014
@silverwind silverwind added type/bug topic/ui Change the appearance of the Gitea UI labels Dec 16, 2020
@silverwind silverwind added this to the 1.14.0 milestone Dec 16, 2020
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Dec 16, 2020
@silverwind
Copy link
Member Author

CI failure is unrelated.

@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 Dec 17, 2020
@6543 6543 merged commit a8fa380 into go-gitea:master Dec 17, 2020
@silverwind silverwind deleted the btnheight branch December 17, 2020 14:42
silverwind added a commit to silverwind/gitea that referenced this pull request Dec 17, 2020
go-gitea#14028 had improved dropdown
triangle display for buttons with text but had regressed it for buttons
without. Fix this by adding a class to prevent horizontal margin adjustment.
techknowlogick added a commit that referenced this pull request Dec 19, 2020
#14028 had improved dropdown
triangle display for buttons with text but had regressed it for buttons
without. Fix this by adding a class to prevent horizontal margin adjustment.

Co-authored-by: Lauris BH <lauris@nix.lv>
Co-authored-by: techknowlogick <techknowlogick@gitea.io>
Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
@go-gitea go-gitea locked and limited conversation to collaborators Jan 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/ui Change the appearance of the Gitea UI type/bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Different size on pull request review UI
5 participants