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

Sample models #187

Merged
merged 8 commits into from May 4, 2022
Merged

Sample models #187

merged 8 commits into from May 4, 2022

Conversation

OlegMoshkovich
Copy link
Member

@OlegMoshkovich OlegMoshkovich commented Apr 28, 2022

  • Add sample model dialog
  • Fix Shortcut Dialog
  • Change Open Dialog icon
  • Fix Open Dialog tooltip
  • Adjusted Open Dialog styles

image

https://olegmoshkovich.github.io/Share/share/v/p/index.ifc#c:-111.37,14.94,90.63,-43.48,15.73,-4.34

@pablo-mayrgundter
Copy link
Member

Looking good! Ready for review? Please add a unit test that checks the element renders.

headerText='Model Access'
isDialogDisplayed={isDialogDisplayed}
setIsDialogDisplayed={setIsDialogDisplayed}
content={
<div className={classes.content}>
<LocalFileOpen/>
<GitHubIcon style = {{width: '50px', height: '50px'}}/>
Copy link
Member Author

Choose a reason for hiding this comment

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

Need to use inline styles here.
We are styling all of the svgs in the dialog which over writes the class styles.

@OlegMoshkovich OlegMoshkovich merged commit b6856de into bldrs-ai:main May 4, 2022
@OlegMoshkovich OlegMoshkovich deleted the sampleModels branch May 5, 2022 08:56
OlegMoshkovich added a commit to OlegMoshkovich/Share that referenced this pull request May 19, 2022
* add sample models Dialog

* add sample model

* add description to the sample model dialog

* adjust shortcut models

* clean up

* clean up

* add test + style details

* clean up
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