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

Side Navigation customization #1458

Merged
merged 28 commits into from Aug 14, 2023
Merged

Side Navigation customization #1458

merged 28 commits into from Aug 14, 2023

Conversation

xman343
Copy link
Contributor

@xman343 xman343 commented Jul 28, 2023

Changes

#1368 - Improving customization of SideNavigation component.

Testing

Will be tested thru unit & visual testing.

Docs

TBD

@xman343 xman343 self-assigned this Jul 28, 2023
@xman343 xman343 mentioned this pull request Jul 28, 2023
31 tasks
@xman343 xman343 marked this pull request as ready for review August 2, 2023 15:58
@xman343 xman343 requested review from a team as code owners August 2, 2023 15:58
@xman343 xman343 requested review from mayank99 and adamhock and removed request for a team August 2, 2023 15:58
mayank99

This comment was marked as outdated.

@mayank99 mayank99 dismissed their stale review August 2, 2023 19:10

pending api discussion

mayank99

This comment was marked as outdated.

@mayank99 mayank99 dismissed their stale review August 10, 2023 16:00

broken again

@xman343
Copy link
Contributor Author

xman343 commented Aug 10, 2023

Reverted the changes that were meant for another PR. Should work now

@mayank99 mayank99 merged commit d7b9a50 into dev Aug 14, 2023
13 of 16 checks passed
@mayank99 mayank99 deleted the xander/side-navigation--x]Props branch August 14, 2023 17:31
@imodeljs-admin imodeljs-admin mentioned this pull request Oct 23, 2023
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.

None yet

4 participants