Skip to content
This repository has been archived by the owner on Mar 15, 2018. It is now read-only.

Bug 1190107: Text Issue in Footer Extras. #1413

Closed
wants to merge 1 commit into from

Conversation

mudithkr
Copy link
Contributor

@mudithkr mudithkr commented Aug 1, 2015

No description provided.

@@ -32,7 +32,7 @@
<div class="footer-content">
<h1 class="footzilla"><a href="https://www.mozilla.org/" target="_blank">mozilla</a></h1>
<p>
{{ _('Except where otherwise <a href="{legal_url}" target="_blank">noted</a>, content on this site is licensed under the <a href="{cc_url}" target="_blank">Creative Commons Attribution Share-Alike License v3.0</a> or any later version.',
{{ _('<span style="color:#717268">Except where otherwise <a href="{legal_url}" target="_blank">noted</a>, content on this site is licensed under the <a href="{cc_url}" target="_blank">Creative Commons Attribution Share-Alike License v3.0</a> or any later version.</span>',
Copy link
Contributor

Choose a reason for hiding this comment

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

inline styles are bad since they add a lot of specificity. preferably we add a class on the p and add the color thru a stylesheet, using one of our Stylus color variables.

@ngokevin
Copy link
Contributor

ngokevin commented Aug 4, 2015

lmk if you can address my comment. thanks!

@ngokevin ngokevin closed this Aug 4, 2015
@mudithkr
Copy link
Contributor Author

mudithkr commented Aug 8, 2015

Thnks for your imp suggestion.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants