runcitadel / dashboard

Other
11 stars 5 forks source link

Add search feature to the app store #36

Closed AaronDewes closed 2 years ago

AaronDewes commented 2 years ago

As the app store grows, a search feature should make using it much easier. This uses fuse.js, so it doesn't search for exact matches, but allows for small differences.

image

image

pwltr commented 2 years ago

Very nice, this was much needed. I'm not a fan of the full-width text field though. Just as a small suggestion, we can wrap the input in a column grid to make it smaller.

<b-row>
  <b-col cols="4"
    ><b-input
      v-model="searchQuery"
      class="neu-input my-4"
      placeholder="Search for apps"
      type="text"
      size="lg"
      autofocus
    ></b-input
  ></b-col>
  <b-col></b-col>
  <b-col></b-col>
</b-row>
WilliamConnatser commented 2 years ago

I kind of agree with Phil that the input is too big. I would probably just put a max width on it on desktop, and keep it full width on mobile. I don't care enough to reject this though so Ima approve this. Great work as always guys! Ya'll are killing it @pwltr @AaronDewes !!! 🚀🚀🚀

AaronDewes commented 2 years ago

Merging now, we can redesign later.