Merge pull request #5041 from brave/bookmark-icon-position

Bookmark Icon position in title mode looks bad
brave · Oct 22, 2016 · e20eb28 · e20eb28
2 parents 394d1e3 + 77cc262
commit e20eb28
Showing 1 changed file with 13 additions and 9 deletions.
diff --git a/js/components/navigationBar.js b/js/components/navigationBar.js
@@ -148,15 +148,19 @@ class NavigationBar extends ImmutableComponent {
             onClick={this.onHome} />
           : null
         }
-        <Button iconClass={this.bookmarked ? 'fa-star' : 'fa-star-o'}
-          className={cx({
-            navbutton: true,
-            bookmarkButton: true,
-            removeBookmarkButton: this.bookmarked,
-            withHomeButton: getSetting(settings.SHOW_HOME_BUTTON)
-          })}
-          l10nId={this.bookmarked ? 'removeBookmarkButton' : 'addBookmarkButton'}
-          onClick={this.onToggleBookmark} />
+        {
+          !this.titleMode
+          ? <Button iconClass={this.bookmarked ? 'fa-star' : 'fa-star-o'}
+            className={cx({
+              navbutton: true,
+              bookmarkButton: true,
+              removeBookmarkButton: this.bookmarked,
+              withHomeButton: getSetting(settings.SHOW_HOME_BUTTON)
+            })}
+            l10nId={this.bookmarked ? 'removeBookmarkButton' : 'addBookmarkButton'}
+            onClick={this.onToggleBookmark} />
+          : null
+        }
       </div>
       <UrlBar ref='urlBar'
         sites={this.props.sites}