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

feat(organizations): add theming/story for Mantine Modal TASK-1423 #5432

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

pauloamorimbr
Copy link
Contributor

🗒️ Checklist

  1. run linter locally
  2. update all related docs (API, README, inline, etc.), if any
  3. draft PR with a title <type>(<scope>)<!>: <title> TASK-1234
  4. tag PR: at least frontend or backend unless it's global
  5. fill in the template below and delete template comments
  6. review thyself: read the diff and repro the preview as written
  7. open PR & confirm that CI passes
  8. request reviewers, if needed
  9. delete this section before merging

📣 Summary

Theming for Mantine's Modal was added to match Kobo's current KoboModal used for general purpose.

📖 Description

  • We don't have a direct reference of an existing Kobo component, so we've came up a middle ground between the existing modals using though the application.
  • We have added a Story with a big set of configurations to showcase the possibilities of the Modal
  • The story also displays a way of
  • We added a theme and style to the Modal component
  • Title is using the default heading weight

👀 Preview steps

  • Use storybook to check the visual of the component.

@pauloamorimbr pauloamorimbr self-assigned this Jan 20, 2025
@pauloamorimbr pauloamorimbr force-pushed the pamorim/task-1423-mantine-theming-modal branch from 3750706 to e068d41 Compare January 21, 2025 20:21
@pauloamorimbr pauloamorimbr force-pushed the pamorim/task-1423-mantine-theming-modal branch from e068d41 to 27e8429 Compare January 21, 2025 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant