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.

Sign up for GitHub

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

Jump to bottom

[#54503] replaced certain icons in xeokit toolbar #15414

Merged
merged 1 commit into from May 7, 2024
Merged

[#54503] replaced certain icons in xeokit toolbar #15414

merged 1 commit into from May 7, 2024

Conversation

Kharonus
Copy link
Member

54503

WAT?

  • replaced some xeokit toolbar icons with new icons from new octicon package

@Kharonus Kharonus requested a review from a team April 29, 2024 14:36
@Kharonus Kharonus self-assigned this Apr 29, 2024
wielinde
wielinde approved these changes May 6, 2024
View reviewed changes
Copy link
Member

@wielinde wielinde left a comment

Choose a reason for hiding this comment

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

Love the new icons.

@Kharonus Could you please make sure that the documentation team updates screenshots, etc?

@Kharonus Kharonus merged commit 9c46880 into dev May 7, 2024
6 checks passed
@Kharonus Kharonus deleted the feature/54503-include-new-ifc-toolbar-octicons branch May 7, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wielinde wielinde wielinde approved these changes

Assignees

@Kharonus Kharonus

Labels
None yet
Milestone
No milestone
2 participants
@Kharonus @wielinde