-
Notifications
You must be signed in to change notification settings - Fork 8
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
[fix] always show the custom lookup #19
Conversation
@marmoure thanks a lot for this super quick implementation. could you please move the custom look up to appear below the matched results, so that it's easier for the user to select the matched result if they are correct. |
@ttasovac |
thanks, @marmoure. i wonder if that's the only way to go about it. perhaps @adamretter can take a look as well? |
@ttasovac The difficulty we have is that we can only present a list of strings back to OxygenXML, and it is up to Oxygen how they order the list. They appear to sort the results lexicographically. I am not sure I am completely clear on what you are asking for. Is the goal here to place
If the Goal is to place |
Yes, indeed, the goal is for the lookup link to be last in the list. 👉 Custom Entry... looks good — people will think we added it on purpose and not as a hack! 😄 So let's do that. Thanks, Adam! |
i still had to prefix the emoji with a Also do you think we should keep the space in front of the emoji? |
e50180b
to
104186d
Compare
104186d
to
4e6378d
Compare
@marmoure the goal is to have the link appear last, so I'm not sure why your screenshots have it appear between #frd and #bibl elements. we need a please try again — with an understanding that the custom lookup link should appear after all the results. |
Sorry @ttasovac for the misunderstanding, My assumption was that |
4e6378d
to
5c54d09
Compare
Fixes #14
The Custom Look Up will always be shown on top even when we have matches results