Skip to content

Correct stying of search preview list #137

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

Closed
Tychodewaard opened this issue Feb 28, 2019 · 5 comments
Closed

Correct stying of search preview list #137

Tychodewaard opened this issue Feb 28, 2019 · 5 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@Tychodewaard
Copy link

Describe the bug

If you start to type in the search box, the preview is quite messy.

To Reproduce

Type something that is used in content in the search box and wait for a sec :-)

Expected behavior

What I'd prefer:

  1. Not show searchSkinObjectPreview_group as it is the name of the module like 'HTML' of 'Content' (2SXC) which is not of help in any of my scenarios.
  2. Have the module title above the search result instead of left
@david-poindexter david-poindexter added the bug Something isn't working label Mar 2, 2019
@david-poindexter
Copy link
Member

david-poindexter commented Mar 2, 2019

Thanks for reporting this @Tychodewaard. We'd like to maintain as much of the core features of the SEARCH skin object as possible, but it could definitely use some styling TLC! 😄

@Tychodewaard
Copy link
Author

Are you saying it is core related? by default the preview of search results is:
image

In Xcillion it looks like:
image

@david-poindexter
Copy link
Member

@Tychodewaard thanks for the additional insight. We'll do an A/B comparison between nvQuickTheme and Xcillion and see if we can tell what is wrong. I agree that the presentation on Xcillion looks MUCH better and this should be fixed.

@david-poindexter david-poindexter added this to the 2.1.0 milestone Jun 22, 2019
@david-poindexter
Copy link
Member

Good news...I have isolated the issue. I will discuss solutions with our team and we'll get a fix in for this upcoming 2.1.0 release.

@david-poindexter david-poindexter changed the title List preview search results Correct stying of search preview list Jul 5, 2019
@david-poindexter david-poindexter self-assigned this Jul 5, 2019
@david-poindexter
Copy link
Member

Closing per #169

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants