Merge pull request #1076 from mozilla/i1059

don't make title a link on android
This commit is contained in:
Danny Coates 2019-01-08 14:14:45 -08:00 committed by GitHub
commit 1bb1e8123f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 21 additions and 6 deletions

View File

@ -1,6 +1,7 @@
const html = require('choo/html'); const html = require('choo/html');
const Component = require('choo/component'); const Component = require('choo/component');
const Account = require('./account'); const Account = require('./account');
const { browserName } = require('../utils');
class Header extends Component { class Header extends Component {
constructor(name, state, emit) { constructor(name, state, emit) {
@ -16,16 +17,27 @@ class Header extends Component {
} }
createElement() { createElement() {
const title =
browserName() === 'android-app'
? html`
<a class="header-logo">
<h1 class="text-white md:text-black font-normal">
Firefox <b>Send</b>
</h1>
</a>
`
: html`
<a class="header-logo" href="/">
<h1 class="text-white md:text-black font-normal">
Firefox <b>Send</b>
</h1>
</a>
`;
return html` return html`
<header <header
class="relative flex-none flex flex-row items-center justify-between bg-blue md:bg-white w-full px-6 h-16 md:shadow z-20" class="relative flex-none flex flex-row items-center justify-between bg-blue md:bg-white w-full px-6 h-16 md:shadow z-20"
> >
<a class="header-logo" href="/"> ${title} ${this.account.render()}
<h1 class="text-white md:text-black font-normal">
Firefox <b>Send</b>
</h1>
</a>
${this.account.render()}
</header> </header>
`; `;
} }

View File

@ -146,6 +146,9 @@ function browserName() {
if (/safari/i.test(navigator.userAgent)) { if (/safari/i.test(navigator.userAgent)) {
return 'safari'; return 'safari';
} }
if (/send android/i.test(navigator.userAgent)) {
return 'android-app';
}
return 'other'; return 'other';
} catch (e) { } catch (e) {
return 'unknown'; return 'unknown';