Skip to content

More arc-green fixes (#13247) #13253

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

Merged
merged 2 commits into from
Oct 22, 2020

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented Oct 22, 2020

Backport of #13247, was a clean cherry-pick.

- Fix various white borders
- Tweak basic button style to have more contrast
- Add more contrast to hover styles
- Invert Matrix webhook icon

May backport to 1.13.

Co-authored-by: zeripath <[email protected]>
@GiteaBot GiteaBot added the lgtm/need 1 This PR needs approval from one additional maintainer to be merged. label Oct 22, 2020
@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 Oct 22, 2020
@zeripath zeripath added topic/ui Change the appearance of the Gitea UI and removed lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. labels Oct 22, 2020
@zeripath zeripath added this to the 1.13.0 milestone Oct 22, 2020
Copy link
Member

@6543 6543 left a comment

Choose a reason for hiding this comment

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

blocking until #13263 got resolved

@GiteaBot GiteaBot added the lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. label Oct 22, 2020
@6543 6543 added the status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR label Oct 22, 2020
Copy link
Member

@techknowlogick techknowlogick left a comment

Choose a reason for hiding this comment

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

blocking via review so I don't accidentally merge

@silverwind
Copy link
Member Author

Linked issue is not really related here, should be good to go.

@6543
Copy link
Member

6543 commented Oct 22, 2020

@silverwind sorry mixed up css fixes somehow @techknowlogick ☝️

@6543 6543 removed the status/blocked This PR cannot be merged yet, i.e. because it depends on another unmerged PR label Oct 22, 2020
@techknowlogick techknowlogick merged commit ad4dde1 into go-gitea:release/v1.13 Oct 22, 2020
@silverwind silverwind deleted the backport-13247 branch October 23, 2020 03:37
@6543 6543 added the skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. label Nov 10, 2020
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
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. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants