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

Demo page improvements and restyling #81

Merged
merged 5 commits into from
Aug 9, 2022

Conversation

paolo-cargnin
Copy link
Contributor

So I read the tweet of Michele, and I tried to improve the demo page as I could without having any specification about the needs.

I aligned better the form elements and changed the header of the page (imho this style is cleaner and more modern, maybe we could adopt it even in the homepage?).
I also added autofocus on the search input.
Of course if it is not as you guys were thinking it, I can improve it.

The main part now that should be improved is the search results, but about this... Can I suggest to use a better structured database? The categories in this example are not easy to display and organize since they are not always the same. Maybe using some more "mainstream" data could be nice (I was thinking movie quotes) ? Dunno.

Let me know!

@ShogunPanda
Copy link
Contributor

Hi Paolo! (nice name XD)
In #82 I also redesigned and improved the demo page. Do you mind reapplying your changes after the current main?

@paolo-cargnin
Copy link
Contributor Author

Yeah of course! I'll do it this evening ;)

@paolo-cargnin
Copy link
Contributor Author

Just to be sure, I added this heading to the demo page:
image

Do you prefer this style or the one that there is right now?

@ShogunPanda
Copy link
Contributor

@paolo-cargnin What about having that gradient in lieu of the current pink background in the hero and use regular text?

@paolo-cargnin
Copy link
Contributor Author

Ok done. Most of the usability features were already done by you. I list here the super small things that I have done:

  • added a progress bar for the indexing
  • heading as described by Paolo
  • Fixed a "category 2" entry on the events.ts file (so it will render "&" instead of &amp)
  • Aligned a bit better the result block
    Let me know!

Copy link
Contributor

@ShogunPanda ShogunPanda left a comment

Choose a reason for hiding this comment

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

Just a minor thing. Rest LGTM, thanks!

packages/docs/src/components/LyraDemo/style.module.css Outdated Show resolved Hide resolved
@ShogunPanda ShogunPanda merged commit 4c8b73f into askorama:main Aug 9, 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.

3 participants