Remove support for custom logos
Summary: - These don't fit anywhere in the new design. - Even if we figure out how to fit them in, 220px logos definitely won't fit on the 320px iPhone screen so anyone who has a custom logo will have to rework them anyway. - Kill it for now, and once we get the new design in and working maybe we can restore it somehow. Test Plan: Loaded local install, no logo. Grepped for config. Reviewers: btrahan, vrana Reviewed By: vrana CC: aran Maniphest Tasks: T1569 Differential Revision: https://secure.phabricator.com/D3101
This commit is contained in:
@@ -381,20 +381,6 @@ final class PhabricatorStandardPageView extends AphrontPageView {
|
||||
$admin_class = 'phabricator-admin-page-view';
|
||||
}
|
||||
|
||||
$custom_logo = null;
|
||||
$with_custom = null;
|
||||
$custom_conf = PhabricatorEnv::getEnvConfig('phabricator.custom.logo');
|
||||
if ($custom_conf) {
|
||||
$with_custom = 'phabricator-logo-with-custom';
|
||||
$custom_logo = phutil_render_tag(
|
||||
'a',
|
||||
array(
|
||||
'class' => 'logo-custom',
|
||||
'href' => $custom_conf,
|
||||
),
|
||||
' ');
|
||||
}
|
||||
|
||||
$notification_indicator = '';
|
||||
$notification_dropdown = '';
|
||||
$notification_container = '';
|
||||
@@ -481,8 +467,7 @@ final class PhabricatorStandardPageView extends AphrontPageView {
|
||||
$header_chrome =
|
||||
'<table class="phabricator-standard-header">'.
|
||||
'<tr>'.
|
||||
'<td class="phabricator-logo '.$with_custom.'">'.
|
||||
$custom_logo.
|
||||
'<td class="phabricator-logo">'.
|
||||
'<a class="logo-standard" href="/"> </a>'.
|
||||
'</td>'.
|
||||
$notification_indicator.
|
||||
|
||||
Reference in New Issue
Block a user