-
Notifications
You must be signed in to change notification settings - Fork 12
Added GitHub contribution link in the header #54
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the unrelated changes. I'll try this out locally soonish to see how it looks.
{%endif%} | ||
</li> | ||
{%if header_activity%} <li class="pull-right active"><a href="{{header_activity_link}}">{{header_activity}}</a></li>{%endif%} | ||
</ul> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unrelated change
{%block extrahead%}{%endblock%} | ||
{%if rss_alternate%} | ||
<link rel="alternate" type="application/rss+xml" title="{{rss_alternate_title}}" href="{{rss_alternate}}" />{%endif%} | ||
</head> | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated change
<script src="/media/commitfest/js/bootstrap.js"></script> | ||
<script src="/media/commitfest/js/commitfest.js?{% static_file_param %}"></script> | ||
{%block morescript%}{%endblock%} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated change
Logged in as {{user}} (<a href="/account/profile/">edit profile</a> | <a href="/account/logout/">log out</a>{%if | ||
user.is_staff%} | <a href="/admin/">administration</a>{%endif%}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
unrelated change
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some of your unrelated changes actually broke the rendering. It now throws an error when loading any page. I tried out your original commit to see what it looked like and left a comment about removing the custom styling.
closes #38
This PR adds a "Contribute on GitHub" link in the header, making it easier for users to find the repository and contribute.
Fixes: #38