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.

Sign up for GitHub

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

Jump to bottom

Change <r:site_name /> to <r:site:name /> #3

Closed
adamjohnson opened this issue Oct 9, 2014 · 3 comments
Closed

Change <r:site_name /> to <r:site:name /> #3

adamjohnson opened this issue Oct 9, 2014 · 3 comments
Labels
enhancement

Comments

@adamjohnson
Copy link
Contributor

As per @nreckart's request at wvuweb/cleanslate#63.

It's used in 4 places in the toolkit.

Searching 14 files for "r:site_name"

/Users/adamjohnson/Sites/cleanslate_themes/cleanslate-toolkit/views/blog_index.rss:
    2  <rss version="2.0">
    3     <channel>
    4:       <title><r:page:name /> | <r:site_name /> | West Virginia University</title>
    5        <link></link>
    6:       <description>RSS feed for the <r:site_name /> blog.</description>
    7        <language>en-us</language>
    8        <pubDate><r:date_format format="httpdate" value="now" /></pubDate>

/Users/adamjohnson/Sites/cleanslate_themes/cleanslate-toolkit/views/layouts/_header.html:
    1  <header class="header sixteen columns">
    2:   <a class="site-logo" href="/"><r:site_name /></a>
    3  
    4    <nav class="nav" role="navigation">

/Users/adamjohnson/Sites/cleanslate_themes/cleanslate-toolkit/views/layouts/default.html:
    4    <head>
    5      <r:comment><!-- meta charset gets set automatically by CleanSlate --></r:comment>
    6:     <title><r:page:first_non_blank_attr names="title, name" /> | <r:site_name /> | West Virginia University</title>
    7      <meta name="description" content="<r:page:meta_description />" />
    8      <meta name="viewport" content="width=device-width, initial-scale=1.0" />

4 matches across 3 files
@adamjohnson adamjohnson mentioned this issue Oct 27, 2014
Closed
@adamjohnson
Copy link
Contributor Author

Adding a note here to update the Hammer mock_data.yml example file to be like this:

site:
  name: This is my site name.

...when I get to this issue.

@zeroedin
Copy link
Contributor

added to mock_data.yml example

@adamjohnson
Copy link
Contributor Author

90e4ac5 fixes this issue. I've also added a mock_data.yml file to this repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@zeroedin @adamjohnson