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

add site logo with fallback (task #1136) #7

Merged
merged 2 commits into from
Feb 25, 2016

Conversation

georgeconstantinou
Copy link
Contributor

No description provided.

<a class="navbar-brand" href="/">Qobo Admin Panel</a>
<?php
$logoImageUrl = 'QoboAdminPanel.logo.png';
if (file_exists(WWW_ROOT . 'img' . DS . 'logos.png')) {
Copy link
Contributor

Choose a reason for hiding this comment

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

@georgeconstantinou typo ... logo.png, not logos.png

}
echo $this->Html->image($logoImageUrl, [
'alt' => 'Site Logo',
'url' => '#'
Copy link
Contributor

Choose a reason for hiding this comment

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

@georgeconstantinou this can be '/' as it will always lead to the site's root. I'll merge for now, but please fix later. Thanks!

mamchenkov added a commit that referenced this pull request Feb 25, 2016
add site logo with fallback (task #1136)
@mamchenkov mamchenkov merged commit b0d6a5b into master Feb 25, 2016
@mamchenkov mamchenkov deleted the task-1136-modules-prototype branch February 25, 2016 16:29
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