8000 Small UI fixes on full width form titles by joubertredrat · Pull Request #3786 · gogs/gogs · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Small UI fixes on full width form titles #3786

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

Closed
wants to merge 9 commits into from
Closed

Small UI fixes on full width form titles #3786

wants to merge 9 commits into from

Conversation

joubertredrat
Copy link

Small fixed with center title, as below
gogs-title-before
gogs-title-after

Copy link
Contributor
@bkcsoft bkcsoft left a comment

Choose a reason for hiding this comment

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

Move all the new-lines to above the comment please.

word-break: break-word;
}
.dont-break-out {
/* These are technically the same, but use both */

Copy link
Contributor

Choose a reason for hiding this comment

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

new-lines should be before the comment, not after (for readability)

Copy link
Author
@joubertredrat joubertredrat Oct 24, 2016

Choose a reason for hiding this comment

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

@bkcsoft I will investigate this, because css is generated by lass.

@bkcsoft bkcsoft added status: needs feedback Tell me more about it 💻 ui You ruined my eyes! labels Oct 24, 2016
Use proper url for libravatar dep
strk and others added 6 commits November 2, 2016 21:00
It is the FROM field in mailer configuration that needs be used,
not the USER field, which is for authentication.

Closes #3615
Fix sender of issue notifications
config lgtm to disable maintainer to LGTM himself's PR
@unknwon
Copy link
Member
unknwon commented Dec 21, 2016

Seems having strange commits mixed in.

@joubertredrat
Copy link
Author

Closed, I removed original repo with this PR

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: needs feedback Tell me more about it 💻 ui You ruined my eyes!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants
0