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

Logo alt text should just be site title #1540

Closed
mwainwright opened this issue Feb 25, 2014 · 1 comment · Fixed by #1561
Closed

Logo alt text should just be site title #1540

mwainwright opened this issue Feb 25, 2014 · 1 comment · Fixed by #1561

Comments

@mwainwright
Copy link

The alt and title attributes of the site logo should just be the site title as set in the config, e.g. "CKAN Demo". At present they are both "CKAN Demo Logo".

@rossjones
Copy link
Contributor

@mwainwright Quickest way to get this fixed is if you don't have commit access to CKAN, you can go to https://github.com/ckan/ckan/edit/master/ckan/templates/header.html and edit the file (search for the two occurrences of Logo and delete them) and submit it as a PR (mentioning #1540 as the ticket number).

If you do have commit access, then don't do this :)

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

Successfully merging a pull request may close this issue.

3 participants