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

fr-115_Toolbar #120

Merged

Conversation

OlegMoshkovich
Copy link
Member

@OlegMoshkovich OlegMoshkovich commented Feb 21, 2022

  • Subtract the toolbar from the Cad view
  • Add new logo
  • Add a new favicon
  • Move logo to the bottom left
  • Create base button group

https://olegmoshkovich.github.io/Share/share/v/p/index.ifc

@OlegMoshkovich OlegMoshkovich changed the title Fix for #91, #93. Branched from upstream with rebase again (#112) fr-115_Toolbar Feb 21, 2022
@OlegMoshkovich OlegMoshkovich marked this pull request as ready for review February 21, 2022 17:33
@OlegMoshkovich
Copy link
Member Author

@pablo-mayrgundter
PTAL!

Copy link
Member

@pablo-mayrgundter pablo-mayrgundter left a comment

Choose a reason for hiding this comment

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

Can't wait :). Mostly just spaces

src/Components/AboutPanel.jsx Outdated Show resolved Hide resolved
src/Components/AboutPanel.jsx Outdated Show resolved Hide resolved
src/Components/AboutPanel.jsx Outdated Show resolved Hide resolved
src/Components/BaseGroup.jsx Show resolved Hide resolved
src/Components/ItemPanel.jsx Outdated Show resolved Hide resolved
src/Containers/CadView.jsx Outdated Show resolved Hide resolved
src/Containers/CadView.jsx Outdated Show resolved Hide resolved
src/Containers/CadView.jsx Outdated Show resolved Hide resolved
src/Containers/CadView.jsx Outdated Show resolved Hide resolved
src/Containers/CadView.jsx Outdated Show resolved Hide resolved
@OlegMoshkovich
Copy link
Member Author

PTAL!

src/Components/ItemPanel.jsx Outdated Show resolved Hide resolved
src/Components/ItemPanel.jsx Outdated Show resolved Hide resolved
src/Components/ItemProperties.jsx Show resolved Hide resolved
src/Components/ItemPropertiesDrawer.jsx Outdated Show resolved Hide resolved
src/Components/ShortcutsPanel.jsx Show resolved Hide resolved
@OlegMoshkovich
Copy link
Member Author

PTAL!

@pablo-mayrgundter pablo-mayrgundter merged commit bec50aa into bldrs-ai:main Feb 23, 2022
@pablo-mayrgundter pablo-mayrgundter mentioned this pull request Mar 6, 2022
@OlegMoshkovich OlegMoshkovich deleted the fr-115_Toolbar-kill-clean branch March 7, 2022 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Completed
Development

Successfully merging this pull request may close these issues.

None yet

2 participants