cheshire-cat-ai / admin-vue

Admin panel of the Cheshire Cat AI
GNU General Public License v3.0
32 stars 28 forks source link

Update packages and fix CSS #68

Closed zAlweNy26 closed 8 months ago

zAlweNy26 commented 8 months ago

Description

Just some tweaks and updates,

Checklist:

zAlweNy26 commented 8 months ago

@kodaline feel free to test it too :)

EugenioPetulla commented 8 months ago

The PR is generally good! :D

The only think I'm noticing is that the"upload" and "create" plugin buttons in the plugins page are too big for the scope, we should make them xs in order to better fit the rest of UX.

Did you swapped the order of the Wipe buttons in the memory page? I prefer have it at the bottom.

zAlweNy26 commented 8 months ago

The "upload" and "create" plugin buttons are medium size, they seem fine to me, but making them smaller could be a little too much.

For the wipe button on the memory page, it is @kodaline's fault 🤣 I'll move it to bottom, I agree with you.

kodaline commented 8 months ago

The "upload" and "create" plugin buttons are medium size, they seem fine to me, but making them smaller could be a little too much.

For the wipe button on the memory page, it is @kodaline's fault 🤣

I am sorry, when I did the heavy endpoints, I was wiping the memory without using the search bar, so without seeing the 2D memory graph. What was happening was that the wipe buttons where going down as if there was the graph but there wasn't, so I interpreted it as "bug".

Sorry again, @zAlweNy26 I can fix it if you want

zAlweNy26 commented 8 months ago

No problem, I fixed it :)