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

fix: refactor loading of main page, use new Sidebar in main page, fix small UI bugs #4451

Merged
merged 50 commits into from
Nov 8, 2024

Conversation

lucaseduoli
Copy link
Collaborator

@lucaseduoli lucaseduoli commented Nov 7, 2024

This pull request includes several updates to the frontend components and styles, particularly focusing on the SideBarFoldersButtonsComponent and related files. Key changes include refactoring import statements, modifying component props and states, and updating UI elements for better user experience and code maintainability.

Component Refactoring and State Management:

UI and Styling Updates:

API and Query Adjustments:

Minor Fixes and Improvements:

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@lucaseduoli lucaseduoli enabled auto-merge (squash) November 7, 2024 17:45
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Nov 7, 2024
@github-actions github-actions bot added bug Something isn't working and removed bug Something isn't working labels Nov 7, 2024
Copy link
Contributor

@anovazzi1 anovazzi1 left a comment

Choose a reason for hiding this comment

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

lgtm

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Nov 7, 2024
@lucaseduoli lucaseduoli merged commit a09652c into main Nov 8, 2024
27 checks passed
@lucaseduoli lucaseduoli deleted the fix/home_nav_sweep branch November 8, 2024 17:33
diogocabral pushed a commit to headlinevc/langflow that referenced this pull request Nov 26, 2024
… small UI bugs (langflow-ai#4451)

* Tighten space between main page sidebar buttons

* Fixed skeleton size

* Added playground button back

* Updated no components and flows state

* Update default icon to Workflow

* Fixed size of list cards

* Removed browse store

* Removed playground button

* Removed black background from empty folder state

* Update empty state color

* Fix color of empty state

* fix text not selectable

* updated border color

* added shadow only on hover

* Remove JSON from Download JSON

* Fixed colors and weight of tabs on home page

* Fixed padding on list and grid components

* Update icons that take long to load

* Fixed icon and bg color for home tiles

* Removed unused code

* removed placeholder data for skeleton to not appear on first load

* Make onSuccess refetch the queries so that the loading waits for it

* Removed unused divs on foldersidebarnav

* Refactor sidebar buttons to use new shadcn sidebar

* Created skeletons for folder, grid and list

* Added new sidebar size

* Use new sidebar button on header and implemented animation

* Changed icon to getIcon

* Added sidebar provider and fixed loading states of the main page

* Removed folder buttons on emptyPage

* Fixed foldername to appear immediatly, and fixed loading states for the folders

* Removed unused state from folders store

* Removed unused states from folders store type

* Added new icon

* fixed modals component to not show a trash icon

* Changed icons to load immediatly

* Added empty folder condition to not display header info

* Added conditions to show loading state until everything loads

* Created empty folder state

* Changed empty page to correct colors

* Added skeletons while flows of the folder are loading

* Removed shadow from text

* Fixed font chivo taking long time to load

* Fix adding new folder not redirecting

* [autofix.ci] apply automated fixes

* Fixed colors and paddings on list and grid components

* Re added tooltips to upload and create folders

* fix input for name editing

* Fix tests

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants