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

Add possibility to search for elements #77

Closed
klues opened this issue Nov 2, 2020 · 16 comments
Closed

Add possibility to search for elements #77

klues opened this issue Nov 2, 2020 · 16 comments
Labels

Comments

@klues
Copy link
Contributor

klues commented Nov 2, 2020

User feedback: it would be nice to have a global search function in order to quickly find existing elements and the place/grid where it is.

Possibliy it's best to implement it somewhere in the Manage Grids view, and do this while also working on #14

@klues klues added enhancement New feature or request priority user feedback and removed priority labels Nov 2, 2020
@klues
Copy link
Contributor Author

klues commented Nov 9, 2020

Idea / user feedback:
Search for existing elements in current gridset, show all search results next to each other, actions (e.g. speech) can be directly triggered from clicking on elements in search results.

Below search results maybe also show results from opensymbols.org

@klues klues added the priority label Nov 9, 2020
@klues
Copy link
Contributor Author

klues commented Feb 14, 2023

Also see discussion #239

@klues
Copy link
Contributor Author

klues commented Dec 14, 2023

I've just released a new version to grid.asterics.eu/latest/.

Changes / new features:

  • Ctrl + F on the keyboard opens the search modal in normal grid view, edit view and manage grids view
  • correctly showing the path to the element in search results
  • click on the image directly shows the element, click on the path highlights the path, starting from home grid:
    grafik
  • keyboard shortcuts: Enter directly shows first search result, Ctrl + Enter shows the path the the first search result
  • always shows shortest path to the element, if multiple paths
  • tried to optimize performance for big grid configurations, thanks to @jamjolu for your quite complex test case (Quick_Say20) ;)
  • added possibilities to open search dialog in action Navigate to other grid, including options to directly search for collected or custom text:
    grafik
  • added search icon to the default global grid - get it by clicking here in "Manage grids":
    grafik
    grafik

To do / open for discussion:

  • currently there is no search button in manage grids and edit grid, only the Ctrl + F is possible. Do we need a button there? Where? I would assume that "power users" will quickly adapt to Ctrl + F and don't need an additional button. If we need it, I think the only place to add it would be within the "More" button.
  • I think we would need an additional pictogram for searching, in the same style as the others in the global grid, @arasaac-dga
  • New strings in crowdin should be translated, @arasaac-dga
  • Should all default gridsets be updated to contain the new global grid?

Please test and comment. @arasaac-dga @jamjolu @tiekoide @deinhofer @sabicalija @msantos-87

@jamjolu
Copy link

jamjolu commented Dec 14, 2023 via email

@arasaac-dga
Copy link
Collaborator

Good morning, Benjamin. First of all, great job as always. Congrats!!!! We are going to reply to your questions and share the issues that we have found. 

TO DO / OPEN FOR DISCUSSION:

  1. "Currently there is no search button in manage grids and edit grid, only the Ctrl + F is possible. Do we need a button there Where? I would assume that "power users" will quickly adapt to Ctrl + F and don't need an additional button. If we need it, I think the only place to add it would be within the "More" button. We have to think in tablet users too where is impossible to use Ctr+F. So we need a search button al most in Manage Grids page (in edit mode is enough with Ctrl+F). We atttach you the poisition that we think is the best (addtionaly we have change the position of number of grids that we think is more suitable)"

search_button_position

You can add too in More menú but we think it is more accessible in the position we have suggested. 

  1. "I think we would need an additional pictogram for searching, in the same style as the others in the global grid, @arasaac-dga"

We have made and upload one new pictogram here: https://arasaac.org/pictograms/es/39110/

  1. "New strings in crowdin should be translated, @arasaac-dga"

Translated

  1. "Should all default gridsets be updated to contain the new global grid?"

We don't think that you must add the new search button in Global Grid. We are more comfortable with the actual global grid. Search option is only for advanced users. So it will not be commonly used by users and they have the upper button in the header to access this option. On the other hand Global Grid has many buttons and adds one more supposed less space for accumulated phrase. So, please come back to the old global grid version (of course with the new features for home and come back buttons).

ISSUES:

In general the search option works as expected in most of the searches but we have found some issues. We attach some videos to see them:

  1. In Edit Cell mode, when you press Ctrl+F the search dialog window is placed behind the Edit Cell Window

https://app.box.com/s/r1ml6alkqbo918tm2g40kzjf5zok4wim

  1. Using our ARASAAC Predefined Communicator we have found issues searching elements placed in Core 50 grid when you use the option to follow the route (with direct access there is no problem). As you can see always go to the same pictogram and not the selected one (goes to "Se acabo"). 

https://app.box.com/s/vauckqff5ahgr71naeivs69do65o7s0jhttps://app.box.com/s/wnso4z438hf9gfz30zxjp01wpr0n6jkdhttps://app.box.com/s/wahpxuj1kp22hygxp1a41tz6bsbnscdvhttps://app.box.com/s/f6qsof8qmfyu3otbdimlixv5ylgbj07ohttps://app.box.com/s/leejqr392stn39e23h17pd6ifnbo5wjqhttps://app.box.com/s/rbksx7l38lqqp476uimx2l058borhcsj

As you can see there are errors with the initial cell of the route that blinks continuously when you come back to the initial grid. 

The new options in "Navigate to" action work as expected. No problems detected. 

@jamjolu
Copy link

jamjolu commented Dec 15, 2023 via email

@arasaac-dga
Copy link
Collaborator

@jamjolu Benjamin has added as sub-action inside "Navigation to other grid" action. Look the capture that Bejamin share with us. In "Navigation type" selecting "Open search dialog" you can assign this action to any cell you wish (with or without custom text).

290552106-d08d5705-82fe-45cf-8cf4-35f50ea6bf2b

klues added a commit that referenced this issue Dec 18, 2023
klues added a commit that referenced this issue Dec 18, 2023
…event errors if there are duplicated grid ids, see #77
klues added a commit that referenced this issue Dec 18, 2023
@klues
Copy link
Contributor Author

klues commented Dec 18, 2023

I've just released a new version to https://grid.asterics.eu/latest/

The flashing elements are not leading me correctly to the destination elements

Thanks for finding this bug - it should be fixed.

You can add too in More menú but we think it is more accessible in the position we have suggested.

Do you think many people will use the button or at all the search function within "Manage grids"? Or should we just provide the keyboard shortcut as now for power users? What about "edit grid view"?

So, please come back to the old global grid version (of course with the new features for home and come back buttons).

OK, I've removed the search button from the default global grid again.

In Edit Cell mode, when you press Ctrl+F the search dialog window is placed behind the Edit Cell Window

Now it's in the foreground - however I think it's a rare case that someone goes to the search dialog while editing an element.

Using our ARASAAC Predefined Communicator we have found issues searching elements placed in Core 50 grid when you use the option to follow the route (with direct access there is no problem).

I think it was the same issue also @jamjolu reported - should be resolved, please test.

Fun fact: @jamjolu 's Quick Say 20 has a total of 28038 possible paths to all other grids starting from the home grid, I've further optimized performance, so that searches in this complex network now take below 500ms.

@ms-mialingvo
Copy link
Collaborator

Hi Benjamin

I just tested after the latest update:
Safari 17.0, Macbook as well as Samsung Android. The leading-through-the-full-path is not working, it skips directly to the page, for both, the fullscreen search and the not-fullscreen search option.
Safari, iPad 16.7. The fullscreen search exits fullscreen once you click on the search button. The leading-through-the-full-path is not working either.
I think this was working before the latest update.

@klues
Copy link
Contributor Author

klues commented Dec 19, 2023

Strange, please describe step by step what you're doing. If I test:

  1. Add new user and import AsTeRICS Grid default
  2. Search for german, I see this result:
    image
  3. After clicking on the path (red marked) it leads me step-by-step to the correct element. Clicking on the image directly shows the element.

Tested in Chrome on Win10 and Safari on iPad Pro.
Edit: on Firefox / Win 10 I can reproduce the issue, I'll investigate.

@klues
Copy link
Contributor Author

klues commented Dec 19, 2023

I've tested again in Firefox, it seems like it wasn't the latest version running there. Now everything works in all browsers for me, so I cannot reproduce the issue.

@ms-mialingvo
Copy link
Collaborator

ms-mialingvo commented Dec 19, 2023

After clearing cache and re-importing everything the step-by-step-function now works again on the Macbook, Android and iPad :) The full screen issue on the iPad remains, though: For Safari and Chrome it exits full screen when I want to search. For Firefox the full screen isn't working at all. All apps are updated.

@arasaac-dga
Copy link
Collaborator

I've just released a new version to https://grid.asterics.eu/latest/

The flashing elements are not leading me correctly to the destination elements

Thanks for finding this bug - it should be fixed.

Yes, it's fixed

You can add too in More menú but we think it is more accessible in the position we have suggested.

Do you think many people will use the button or at all the search function within "Manage grids"? Or should we just provide the keyboard shortcut as now for power users? What about "edit grid view"?

We are discussing again and we agree that perhaps the Search button is not needed in Manage Grid page. Thinking in people that edit boards from a tablet (with no keyboard) perhaps it would be more suitable in More menu as you initialy proposed (of Edit Grid page only, not in more menu of Manage Grids).

So, please come back to the old global grid version (of course with the new features for home and come back buttons).

OK, I've removed the search button from the default global grid again.

Thanks

In Edit Cell mode, when you press Ctrl+F the search dialog window is placed behind the Edit Cell Window

Now it's in the foreground - however I think it's a rare case that someone goes to the search dialog while editing an element.

Yes it's a rare cae but always is better that is in the correct position.

Using our ARASAAC Predefined Communicator we have found issues searching elements placed in Core 50 grid when you use the option to follow the route (with direct access there is no problem).

I think it was the same issue also @jamjolu reported - should be resolved, please test.

Yes, it works fine now.

Fun fact: @jamjolu 's Quick Say 20 has a total of 28038 possible paths to all other grids starting from the home grid, I've further optimized performance, so that searches in this complex network now take below 500ms.

@jamjolu
Copy link

jamjolu commented Dec 20, 2023 via email

@klues
Copy link
Contributor Author

klues commented Jan 2, 2024

The full screen issue on the iPad remains, though: For Safari and Chrome it exits full screen when I want to search.

Thanks for the hint. I've tried it and could reproduce it. I assume that maybe Safari automatically stops fullscreen at showing the keyboard?! However if you "install" the app (add it to home screen and start it from there), for me staying in fullscreen and searching works. However if there's anything left to be done regarding this, please just open a new issue.

Thinking in people that edit boards from a tablet (with no keyboard) perhaps it would be more suitable in More menu as you initialy proposed (of Edit Grid page only, not in more menu of Manage Grids).

Ok, I'll add it there and then release everything to main version.

@klues
Copy link
Contributor Author

klues commented Jan 3, 2024

@klues klues closed this as completed Jan 3, 2024
klues added a commit that referenced this issue Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants