Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Redesign Search page with voice search #593

Merged
merged 146 commits into from
Sep 5, 2022

Conversation

candry7731
Copy link
Contributor

Add Voice search while in folders
Also added a spinner to grid items when loading so user know the app is working
This is to resolve #587

@neilsb
Copy link
Member

neilsb commented May 11, 2022

Is this PR intended to replace the Search Redesign (#568) or should this be looked at after 568 has been merged?

@candry7731 candry7731 changed the title Voice functions #587 Voice functions May 11, 2022
@candry7731 candry7731 mentioned this pull request May 11, 2022
@jimdogx
Copy link
Contributor

jimdogx commented May 14, 2022

This is looking really good!!!!

A couple of things:

  • Voice search appears to work everywhere for me except Live TV. If I try to voice search, it doesn't find anything, but the bad part is that you lose focus from the TV guide and can't get it back. Might be worth just seeing if you can remove voice search from the TV guide all together. Because it works great for Movies, TVShows, etc.

  • Roku is spitting out a few warnings / errors (even though it appears to work). Might be worth investigating:

*** ERROR: Missing or invalid PHY: '/RokuOS/Keyboards/MiniKeyboardKDF.json'

Warning occurred while setting a field of an RoSGNode
-- Type mismatch occurred when setting the "results" field of a "SearchTask" node
pkg:/components/search/SearchTask.brs(7)

BRIGHTSCRIPT: ERROR: roUrlTransfer.GetToString: Sync-wait interrupted: pkg:/source/api/baserequest.brs(71)

Fix sort names to add back the "_"
Fix TR File to original
Fix naming round 2
@neilsb
Copy link
Member

neilsb commented Aug 24, 2022

That's the translation stuff sorted. And voice search seems to be working well.

I'm seeing a problem when selecting search from the options menu on the Home Screen. When the search page opens nothing seems to have focus, so I can't do anything. I'll try and have a look at it shortly.

@jimdogx
Copy link
Contributor

jimdogx commented Aug 24, 2022

I'm getting a crash when selecting "*" and then "Search" from the Home Screen:

Screen Shot 2022-08-24 at 5 43 07 PM

@candry7731
Copy link
Contributor Author

candry7731 commented Aug 25, 2022 via email

@jimdogx
Copy link
Contributor

jimdogx commented Aug 25, 2022

Crash is fixed. However, if you click in to any library (Movies, TV Shows, etc), the word "Search" shows up at the top on top of the first thumbnail.

@neilsb
Copy link
Member

neilsb commented Aug 25, 2022

From the main search screen (accessed via Options menu) search results all look good, and seem to return expected results.

From the results, Movies, TV Shows and Episodes all seem to work as expected also.

However, choosing a Person, Music Album or Music Track item causes the app to crash, as it's trying to display them in the Movie screen, rather than going to appropriate page for the type.

Copy link
Contributor Author

@candry7731 candry7731 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Formatting

Copy link
Contributor

@jimdogx jimdogx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't know if it matters, but if you speak your password (one character at a time), your password is shown.

components/ItemGrid/ItemGrid.brs Outdated Show resolved Hide resolved
@neilsb neilsb merged commit 621e53e into jellyfin:unstable Sep 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Voice search
5 participants