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/dropdown component and navbar bottom line #800

Conversation

momodaadaa99
Copy link
Contributor

The changes include adding new dependencies and refactoring the dropdown component to use @floating-ui so that it fixes the dropdown problem and add automatic position of the dropdown in case of edges.

Dependencies:

  • Added @floating-ui/react to the dependencies in package.json.

Dropdown Component Enhancements:

  • Refactored Dropdown.component.tsx to use @floating-ui for positioning and interactions.

Additional Changes:

  • Removed the border-bottom style from Navbar.component.tsx to fix the difference in height in navbar when there is a dropdown.

@momodaadaa99 momodaadaa99 self-assigned this Nov 20, 2024
@JBWatenbergScality JBWatenbergScality changed the base branch from development/1.0 to improvement/status-icon-component November 20, 2024 16:44
@momodaadaa99 momodaadaa99 force-pushed the fix/dropdown-component-and-navbar-bottom-line branch from 24d92a3 to d160dde Compare November 20, 2024 16:59
@@ -109,19 +113,30 @@ function Dropdown({
items,
itemToString: (item) => item?.label || '',
});

const { refs, floatingStyles } = useFloating({
// open: isOpen,
Copy link
Contributor

Choose a reason for hiding this comment

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

can be deleted

@momodaadaa99
Copy link
Contributor Author

/approve

Base automatically changed from improvement/status-icon-component to development/1.0 November 21, 2024 13:56
@JBWatenbergScality
Copy link
Contributor

/status

@momodaadaa99 momodaadaa99 deleted the fix/dropdown-component-and-navbar-bottom-line branch November 21, 2024 14:00
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.

2 participants