The logo option should link to the configured site_url

[msudol]

Description

The site logo currently has html source that looks like this

<a href="../.." title="My Docs" class="md-logo md-header-nav__button">

Expected behavior

Expected behavior would be that the yml configuration for site_url would be here if it is configured.

Actual behavior

The site_url configuration is not made available anywhere within the material theme.

Steps to reproduce the bug

1. Set a logo in config
2. Set a site_url in config
3. Serve site in material theme

Package versions

• Python: 3.6.0
• MkDocs: 0.16.2
• Material: 1.5.0

Project configuration

# Project information
site_name: My Docs Example
site_url: https://www.mysite.com
site_description: My Test Documentation Site
site_author: John Do
repo_name: 'My Docs Example'
repo_url: 'https://www.mysite.com'

# Copyright
copyright: 'Copyright © 2016 - 2017 John Doe'

# Theme
theme: 'material'
theme_dir: 'theme'

extra_css:
  - 'css/extra.css'
  
# Options
extra:
  logo: 'assets/images/logo.svg'
  palette:
    primary: 'blue grey'
    accent: 'deep orange'
  font:
    text: 'Roboto'
    code: 'Roboto Mono'

# Settings
docs_dir: 'docs'
site_dir: 'site'

# Extensions
markdown_extensions:
    - admonition
    - smarty
    - codehilite
    - footnotes
    - meta
    - toc(permalink=true)
    - pymdownx.arithmatex
    - pymdownx.betterem(smart_enable=all)
    - pymdownx.caret
    - pymdownx.critic
    - pymdownx.inlinehilite
    - pymdownx.magiclink
    - pymdownx.mark
    - pymdownx.smartsymbols
    - pymdownx.superfences
    - pymdownx.tasklist(custom_checkbox=true)
    - pymdownx.tilde  

# Pages    
pages:
    - Home: 'index.md'
    - User Guide: 
        - 'Step 1': 'user-guide/step1.md'
        - 'Step 2': 'user-guide/step2.md'
        - 'Step 3': 'user-guide/step3.md'
    - About:
        - 'License': 'about/license.md'
        - 'Release Notes': 'about/release-notes.md'    

System information

• OS: Windows 10
• Browser: Firefox

[squidfunk]

That's actually a good point and interesting that nobody noted it up to now. Makes sense to me, because sometimes you may want to link to another page and not to the index page of the docs.

[msudol]

Thanks, and amazing work on this theme by the way. I've been trying to settle on a framework for documentation and as a node.js developer wanted something node based.. but this theme along with how MkDocs is just so easy to get setup, it won me over.

[squidfunk]

Fixed in 1.5.1 which was just released.

[borekb]

I'm not sure this still works in the latest release – the icon links to 0.0.0.0:8000 in my case (I'm running the Docker container).

[squidfunk]

It should:
https://github.com/squidfunk/mkdocs-material/blob/master/src/partials/header.html#L32

[borekb]

Strange, this is the generated HTML:

<div class="md-flex__cell md-flex__cell--shrink">
        <a href="http://0.0.0.0:8000/" title="Example Docs" class="md-header-nav__button md-logo">
          
            <i class="md-icon">school</i>
          
        </a>
      </div>

Wondering if this has something to do with Docker: I'm on Windows where 0.0.0.0 is not even valid. I can browse the docs in the browser at http://localhost:8000/ but when I click the home icon, the site doesn't load:

This is from my mkdocs.yml:

site_name: Example Docs
site_url: 'https://example.com'

(tried also without quotes).

[borekb]

For reference:

mkdocs-material/Dockerfile

Line 51 in dbd0006

CMD ["serve", "--dev-addr=0.0.0.0:8000"]

[squidfunk]

I will investigate.