Home page shows "index.html" in menu bar #658

Closed
opened 2017-10-10 21:50:32 +02:00 by tzarebczan · 3 comments
tzarebczan commented 2017-10-10 21:50:32 +02:00 (Migrated from github.com)

The Issue

Home page menu bar shows index.html. Probably related to some recent search changes which put the search icon in the address bar instead of home. It should show "Home" or possibly "Discover".
homepage

<!-- Thanks for reporting an issue to LBRY and helping us improve! To make it possible for us to help you, please fill out below information carefully. Before reporting any issues, please make sure that you're using the latest version. - App releases: https://github.com/lbryio/lbry-app/releases - Standalone daemon: https://github.com/lbryio/lbry/releases We are also available on Slack at https://slack.lbry.io --> ## The Issue Home page menu bar shows index.html. Probably related to some recent search changes which put the search icon in the address bar instead of home. It should show "Home" or possibly "Discover". ![homepage](https://user-images.githubusercontent.com/8120721/31407408-db965df8-add2-11e7-81e7-a4662b3d8177.PNG)
sircodesalittle commented 2017-10-10 21:57:54 +02:00 (Migrated from github.com)

I'll do some investigating.

I'll do some investigating.
tzarebczan commented 2017-10-10 22:03:52 +02:00 (Migrated from github.com)

It was introduced here: 18450209b7 (diff-9a6a461f504e63bae49576c6db1d1592) because we wanted to add the search icon/text on the home page, but it also tied to the menu bar. Correct me if I am wrong, but the fix would be to independently set the Title bar in the case of Home.

It was introduced here: https://github.com/lbryio/lbry-app/commit/18450209b77e63ddb047c805ceb8edbaea1d2fd2#diff-9a6a461f504e63bae49576c6db1d1592 because we wanted to add the search icon/text on the home page, but it also tied to the menu bar. Correct me if I am wrong, but the fix would be to independently set the Title bar in the case of Home.
btzr-io commented 2017-10-11 03:46:21 +02:00 (Migrated from github.com)

I'm not sure if it needs to show "Home" or "Discover", the title it's already handle on the search-bar / wonder-bar...

I'm not sure if it needs to show "Home" or "Discover", the title it's already handle on the `search-bar` / `wonder-bar`...
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#658
No description provided.