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

[EuiCollapsibleNav] Right direction for Collapsible nav #4303

Closed
sjp-jamalian opened this issue Nov 24, 2020 · 5 comments
Closed

[EuiCollapsibleNav] Right direction for Collapsible nav #4303

sjp-jamalian opened this issue Nov 24, 2020 · 5 comments

Comments

@sjp-jamalian
Copy link

It would be convenient if a user could select direction of Collapsible nav. (Left or Right)
Thank you
1

@thompsongl thompsongl changed the title [Collapsible nav] Right direction for Collapsible nav [EuiCollapsibleNav] Right direction for Collapsible nav Nov 24, 2020
@kshitij86
Copy link

kshitij86 commented Nov 26, 2020

@sjpsjp Can this not be done by defining rules in the style prop for the CollapsibleNav component? Adding your own positioning rules should be able to achieve this. Or do you mean adding the direction as a prop for convenience?

@sjp-jamalian
Copy link
Author

I mean adding direction as a prop for convenience.
Thank you

@cchaos
Copy link
Contributor

cchaos commented Dec 9, 2020

Hi @sjpsjp . We don't currently have any plans to add this option, trivial as it may be. This component is our opinionated global navigation pattern for Elastic specifically. However, it is just made up of lots of smaller EUI components. For instance you can easily acheive a right side nav by putting the same contents within a regular EuiFlyout and just adding your own toggle button to EuiHeader.

@github-actions
Copy link

github-actions bot commented Jun 8, 2021

👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

@cchaos
Copy link
Contributor

cchaos commented Jun 9, 2021

I missed this associated issue when merging #4713. EuiCollapsibleNav now accepts a side prop.

@cchaos cchaos closed this as completed Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants