Skip to content
This repository has been archived by the owner on Feb 15, 2025. It is now read-only.

feat(ui): remove carbon and replace with Flowbite #862

Merged
merged 101 commits into from
Aug 2, 2024
Merged

feat(ui): remove carbon and replace with Flowbite #862

merged 101 commits into from
Aug 2, 2024

Conversation

andrewrisse
Copy link
Contributor

PR #6 for Flowbite refactor. Final PR for all Flowbite components.

Refactors the File Management table

Updates the copy btn for code blocks

Removes all carbon packages (except for the pictograms, which we plan to continue to use)

Deployed on LF-04

localhost:5173/chat - 31 July 2024 - Watch Video

andrewrisse added 30 commits July 18, 2024 09:14
@andrewrisse
init
b8c466a
@andrewrisse
remove carbon sass preprocessing
c9af3c4
@andrewrisse
add navbar with drawer links, needs styling
f47d0ee
@andrewrisse
finished navbar
760ff2f
@andrewrisse
update header test
52d9636
@andrewrisse
Add sidebar with edit and delete
c65d791
@andrewrisse
fix sidebar menu btn
7506131
@andrewrisse
merge main
480ac2a
@andrewrisse
update sidebar tests
c26748d
@andrewrisse
all unit tests passing
abe58d9
@andrewrisse
playwright in progress
c9a3f9e
@andrewrisse
add sidebar menu open/close btn
4ff9120
@andrewrisse
Merge branch 'main' into flowbite
9be9b07
@andrewrisse
rename sidebar.svelte
b077e1a
@andrewrisse
add select assistant dropdown
b24e1d3
@andrewrisse
add dynamic height textarea
e7d9dc4
@andrewrisse
fix unit tests
925459e
@andrewrisse
message wip
b4f02f4
@andrewrisse
fix message styling
7e96757
@andrewrisse
finish styling for chat page
80da249
@andrewrisse
lint
40881c4
@andrewrisse
Fix e2es
1d03bca
@andrewrisse
Fix e2es 2
24c77c2
@andrewrisse
remove file attach btn
de1255e
@andrewrisse
Remove unnecessary comment
8bf0457
@andrewrisse
merge main
0fffbfd
@andrewrisse
Merge branch 'flowbite' into flowbite-chat
875fdee
@andrewrisse
fix duplicate package
953200d
@andrewrisse
Merge branch 'flowbite' into flowbite-chat
dc0dd5a
@andrewrisse
fix styles due to carbon override
c4d5db0
@andrewrisse andrewrisse marked this pull request as ready for review August 2, 2024 14:51
@andrewrisse andrewrisse requested a review from a team as a code owner August 2, 2024 14:51
gregclark-defenseunicorns
gregclark-defenseunicorns previously approved these changes Aug 2, 2024
View reviewed changes
Copy link
Contributor

@gregclark-defenseunicorns gregclark-defenseunicorns left a comment

Choose a reason for hiding this comment

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

Design Review passed

YrrepNoj
YrrepNoj previously approved these changes Aug 2, 2024
View reviewed changes
Copy link
Member

@YrrepNoj YrrepNoj left a comment

Choose a reason for hiding this comment

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

I looked at EVERY SINGLE LINE and this looks good to me!

@andrewrisse andrewrisse changed the base branch from flowbite-integration to main August 2, 2024 16:49
@andrewrisse andrewrisse dismissed stale reviews from YrrepNoj and gregclark-defenseunicorns August 2, 2024 16:49

The base branch was changed.

@netlify Netlify
Copy link

netlify bot commented Aug 2, 2024 •
edited
Loading

✅ Deploy Preview for leapfrogai-docs canceled.

Name Link
🔨 Latest commit 0c52960
🔍 Latest deploy log https://app.netlify.com/sites/leapfrogai-docs/deploys/66ad3a4a44b3d50008c69bdf

YrrepNoj
YrrepNoj previously approved these changes Aug 2, 2024
View reviewed changes
YrrepNoj
YrrepNoj previously approved these changes Aug 2, 2024
View reviewed changes
@andrewrisse andrewrisse changed the title feat(ui): Flowbite PR #6 (file management and cleanup) feat(ui): remove carbon and replace with Flowbite Aug 2, 2024
@andrewrisse andrewrisse merged commit 921a864 into main Aug 2, 2024
17 checks passed
@andrewrisse andrewrisse deleted the flowbite-6 branch August 2, 2024 20:16
@github-actions github-actions bot mentioned this pull request Aug 2, 2024
Merged
This was referenced Aug 5, 2024
Closed
Closed
Closed
@andrewrisse andrewrisse mentioned this pull request Aug 29, 2024
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@YrrepNoj YrrepNoj YrrepNoj approved these changes

@gregclark-defenseunicorns gregclark-defenseunicorns gregclark-defenseunicorns left review comments

@BillyFigueroa BillyFigueroa Awaiting requested review from BillyFigueroa

Assignees

@andrewrisse andrewrisse

Labels
enhancement New feature or request
Projects
None yet
Milestone
Current - RAG UX Enhancements | Model...
Development

Successfully merging this pull request may close these issues.
4 participants
@andrewrisse @YrrepNoj @gregclark-defenseunicorns @jalling97