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

chore(general): allow empty model initialization #126

Closed
wants to merge 43 commits into from

Conversation

sergiulucaci
Copy link
Contributor

This PR aims to resolve #124 ticket.

Since this is my first contact with all packages, there might be some default values that you expect to see but ignored by me, so please review carefully default values chosen.

Besides this, there are two UI panels that seem not to work properly:

and I'm not sure if those are known issues or WIP, but if not, we could create tickets for those.

For some packages I avoided to use arrow-body-style for consistency purposes.

@sergiulucaci sergiulucaci requested a review from edeustace March 7, 2019 16:18
@edeustace
Copy link
Contributor

thanks @sergiulucaci will look in the morning!

@evaneus
Copy link
Contributor

evaneus commented Mar 7, 2019

thanks @sergiulucaci can you check with @elodszopos about math-inline?

@elodszopos
Copy link
Contributor

Verified with @sergiulucaci that the math-inline textarea in question is working as intended. No problems with the interaction.

@sergiulucaci sergiulucaci deleted the chore/elements-model-initialization branch March 11, 2019 16:34
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.

All config Elements should be initializeable with an empty model object.
4 participants