-
Notifications
You must be signed in to change notification settings - Fork 555
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
Load tags dynamically using the API instead of on page first load #386
Comments
Where do you have this problem? In the search, in the article list, in the tag list, in the "add bookmark" dialog? |
I'm facing this in the add new bookmark screen, as well as when I try to edit an existing bookmark. DB is Postgresql. |
I see the tags input getting suggested, but not able to select from the suggested list.
Firefox v97 |
I've just tried this on Firefox 97.0.2 @ macos 12.1 and it's working: Screen.Recording.2022-03-06.at.00.09.05.movI put a few letters and press enter on the suggestion. Do you see any errors on the firefox console? |
After finding this bug report, I guess I finally understand how tag suggestions are supposed to work. It looks like Shiori's suggestion shows only one of the possible tags, and only when the text entered does not match the suggestion anymore, another possible tag is shown. That's not intuitive at all. Usually, when you enter a tag the suggestions show all the tags matching the substring typed so far, with a way to select one of the (possibly multiple) suggested tags, either by navigating with the keyboard or by clicking on it. |
Moving this from bug to enhancement, and while agree that the current selector is not optimal I can't consider this a "bug" since it's working as expected. At least until we improve it :) @thelazyoxymoron if you are experiencing an actual bug please report back so we can take a look at it (and proper triage a new issue for the feature request). |
Apologies for not providing enough information earlier. I did some more testing and looks like this is only happening with Postgres DB backend and only when I try to add a tag in the following window: I did not observe this when trying to edit an existing bookmark and add tags. The auto-suggest + enter was working there. Only observing when adding tags in the "add new bookmar" screen. This is related to #215. |
I've been testing this locally, and after #484 fix for PSQL it now works but there's a caveheat: the frontend loads tags on page load, so if you add a new bookmark with a new tag and then add another bookmark that new tag wont autocomplete unless the page is reloaded. |
I also confirm I can't click on the suggested tags (Docker, sqlite3, Firefox, Windows). It would be great to have the tags suggestion in the browser extension too (go-shiori/shiori-web-ext#64) |
I'm not able to select any tags from the matched list. Using the Arm64 docker image built using the latest codebase.
The text was updated successfully, but these errors were encountered: