@tiger5226 Were you planning on finishing this up? I can also make the changes. I do really like this change so it would be great to have for the initial release

tiger5226 commented

2018-06-04 23:51:05 +02:00

(Migrated from github.com)

ohh crap. I saw this but forgot about. I will make the change tonight. Sorry about that.

neb-b commented

2018-06-05 05:04:16 +02:00

(Migrated from github.com)

Just merged https://github.com/lbryio/lbry-app/pull/1514

Which just moves the search modal to an actual page with forward/back support. There are some style issues which should be fixed by this PR.

Just merged https://github.com/lbryio/lbry-app/pull/1514 Which just moves the search modal to an actual page with forward/back support. There are some style issues which should be fixed by this PR.

tiger5226 commented

2018-06-05 13:40:59 +02:00

(Migrated from github.com)

ok, I will take care of this.

tiger5226 commented

2018-06-06 04:20:09 +02:00

(Migrated from github.com)

Hi Sean,

I merged master in, but there are now more issues on the search page after doing so. When you type, then click the search row, it brings you to an empty page. I saw a commit for adjusting the search mechanics. I am not good with react or javascript so its not so simple. Also I tried a rebase and there are conflicts that need to be resolved and I am not sure I am the best person to be doing that.

Hi Sean, I merged master in, but there are now more issues on the search page after doing so. When you type, then click the search row, it brings you to an empty page. I saw a commit for adjusting the search mechanics. I am not good with react or javascript so its not so simple. Also I tried a rebase and there are conflicts that need to be resolved and I am not sure I am the best person to be doing that.