Skip to content
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

Add the possibility to add a logo on a finder #189

Merged
merged 1 commit into from
Apr 30, 2015

Conversation

jackscotti
Copy link
Contributor

European Structural and Investment funds must comply with EU legislations and they therefore need to display the EU logo.

Quote from the legislation: "the Union emblem and the reference to the Union shall be visible, when landing on the website, inside the viewing area of a digital device, without requiring a user to scroll down the page".

  • Why is this not a GOV.UK Component? Because we might receive different logos, with different text and with different styles.

@@ -23,6 +23,10 @@ def beta?
content_item.details.beta
end

def logo
content_item.details.logo
Copy link
Contributor

Choose a reason for hiding this comment

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

Since you're just calling content_item.details.logo here, you can add it to the delegate section at the top of the class.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Uhm... will do, thanks Tommy.

Copy link
Contributor

Choose a reason for hiding this comment

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

Delegation is good 👍

Copy link
Contributor

Choose a reason for hiding this comment

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

Done. I quite like that...

@jackscotti jackscotti changed the title [Discuss: it's not pretty, give me feedback! Halp!]Add the possibility to add a logo on a finder [Discuss: feedback!]Add the possibility to add a logo on a finder Apr 23, 2015
@jackscotti jackscotti changed the title [Discuss: feedback!]Add the possibility to add a logo on a finder [do not merge yet]Add the possibility to add a logo on a finder Apr 29, 2015
@jackscotti jackscotti changed the title [do not merge yet]Add the possibility to add a logo on a finder [wip]Add the possibility to add a logo on a finder Apr 29, 2015
@issyl0 issyl0 force-pushed the add_eu_logo_to_esif_funds branch 2 times, most recently from 5a4cd12 to be6ee03 Compare April 29, 2015 15:44
European Structural and Investment funds must comply with EU
legislations and they therefore need to display the EU logo.

Quote from the legislation: "the Union emblem and the reference to the
Union shall be visible, when landing on the website, inside the
viewing area of a digital device, without requiring a user to scroll
down the page".
@boffbowsh boffbowsh changed the title [wip]Add the possibility to add a logo on a finder Add the possibility to add a logo on a finder Apr 30, 2015
tommyp added a commit that referenced this pull request Apr 30, 2015
Add the possibility to add a logo on a finder
@tommyp tommyp merged commit 7a5e331 into master Apr 30, 2015
@tommyp tommyp deleted the add_eu_logo_to_esif_funds branch April 30, 2015 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants