-
Notifications
You must be signed in to change notification settings - Fork 479
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
Create and Set Up Add level dialog for Activities Editor GUI #36732
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work, Dani! Thank you for separating this out.
|
||
//TODO: Set up creating a new level. Make sure dropdown has all level types | ||
|
||
export default class AddLevelFilters extends Component { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix class name
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, how does the code even work as-is? Is it because of export default
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
exactly. importing the default implicitly sets the name of the import.
{this.state.methodOfAddingLevel === 'Create New Level' && ( | ||
<CreateNewLevelInputs /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you please include a screenshot of what this looks like?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
//TODO: Set up creating a new level. Make sure dropdown has all level types | ||
|
||
export default class AddLevelFilters extends Component { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huh, how does the code even work as-is? Is it because of export default
?
This splits out some of the work that was in #36627. You can watch the gif below to see what happens when you click add level. The dialog that shows when you click add level is all the components in this PR.
Links
Testing story
Added simple unit tests for each component.
Reviewer Checklist: