Skip to content

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