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

Preset and add creation #1015

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Preset and add creation #1015

wants to merge 1 commit into from

Conversation

SteveMicroNova
Copy link
Contributor

@SteveMicroNova SteveMicroNova commented Jan 9, 2025

What does this change intend to accomplish?

Closes #707

Moved stream and preset creation workflows to the components folder, then spread them to relevant places (preset creation on presets settings page and presets modal on home page, stream creation in all places streams can be selected)

Checklist

  • Have you tested your changes and ensured they work?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • If applicable, have you updated the documentation/manual?
  • If applicable, have you updated the CHANGELOG?
  • Does your submission pass linting & tests? You can test on localhost using ./scripts/test
  • If this is a UI change, have you tested it across multiple browser platforms on both desktop and mobile?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

TypeSelectModal and StreamModal are always used as a pair, so I packaged them as a pair for ease of use

Copy link
Contributor Author

@SteveMicroNova SteveMicroNova Jan 9, 2025

Choose a reason for hiding this comment

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

I liked the style of the add stream button on the home page and wanted to use it elsewhere, so I've packaged it into this component

Comment on lines 27 to 29
<>
<Modal className="streams-modal" onClose={onClose}>
<Modal onClose={onClose}>
<Card className="type-select-card">
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This class wasn't declared in any CSS so I removed it

…ls into the components folder and reuse them on the home page
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.

Add "Add Stream" and "Add Preset" buttons, modals to relevant modals on Home screen
1 participant