-
-
Notifications
You must be signed in to change notification settings - Fork 634
feat: Mantine v8 upgrade #2028
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: Mantine v8 upgrade #2028
Changes from all commits
589c6f2
52e0fd9
7dedd5f
034ff21
6fa5f92
d3db99d
b46968f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -82,62 +82,51 @@ | |
| @import url("@mantine/core/styles/Timeline.css"); | ||
| @import url("@mantine/core/styles/Title.css"); | ||
| @import url("@mantine/core/styles/Tooltip.css"); | ||
| @import url("@mantine/core/styles/TypographyStylesProvider.css"); | ||
| @import url("@mantine/core/styles/Typography.css"); | ||
|
|
||
| /* Mantine baseline styles, scoped to .bn-mantine element. */ | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. are you sure we don't need the other styles from global.css?
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, the The other styles in |
||
| /* https://github.com/mantinedev/mantine/blob/8.3.1/packages/%40mantine/core/src/core/MantineProvider/baseline.css */ | ||
| .bn-mantine { | ||
| color-scheme: var(--mantine-color-scheme); | ||
| } | ||
|
|
||
| /* Mantine global styles, scoped to bn-container */ | ||
| /* Based on @mantine/core/styles/global.css | ||
| (src: https://github.com/mantinedev/mantine/blob/master/packages/%40mantine/core/src/core/MantineProvider/global.css) | ||
| but with styles set on `body` and `html` instead set on `bn-container`, as | ||
| well as other minor changes. */ | ||
| .bn-mantine *, | ||
| .bn-mantine :after, | ||
| .bn-mantine :before { | ||
| .bn-mantine *::before, | ||
| .bn-mantine *::after { | ||
| box-sizing: border-box; | ||
| } | ||
|
|
||
| /* Fonts are already set in `style.css`, so not needed here. */ | ||
| /* .bn-mantine input, | ||
| .bn-mantine button, | ||
| .bn-mantine textarea, | ||
| .bn-mantine select { | ||
| text-transform: none; | ||
| } | ||
|
|
||
| @media screen and (max-device-width: 500px) { | ||
| .bn-mantine { | ||
| -webkit-text-size-adjust: 100%; | ||
| } | ||
| } | ||
|
|
||
| @media (prefers-reduced-motion: reduce) { | ||
| .bn-mantine [data-respect-reduced-motion] [data-reduce-motion] { | ||
| animation: none; | ||
| transition: none; | ||
| } | ||
| } | ||
| font: inherit; | ||
| } */ | ||
|
|
||
| .bn-mantine [data-mantine-color-scheme="dark"] .mantine-dark-hidden, | ||
| .bn-mantine [data-mantine-color-scheme="light"] .mantine-light-hidden { | ||
| display: none; | ||
| } | ||
|
|
||
| .bn-mantine .mantine-focus-auto:focus-visible { | ||
| outline: calc(0.125rem * var(--mantine-scale)) solid | ||
| var(--mantine-primary-color-filled); | ||
| outline-offset: calc(0.125rem * var(--mantine-scale)); | ||
| } | ||
|
|
||
| .bn-mantine .mantine-focus-always:focus { | ||
| outline: calc(0.125rem * var(--mantine-scale)) solid | ||
| var(--mantine-primary-color-filled); | ||
| outline-offset: calc(0.125rem * var(--mantine-scale)); | ||
| .bn-mantine button, | ||
| .bn-mantine select { | ||
| text-transform: none; | ||
| } | ||
|
|
||
| .bn-mantine .mantine-focus-never:focus { | ||
| outline: none; | ||
| } | ||
| .bn-mantine { | ||
| /* Defaults for font, text color, etc are already set in `style.css` so they | ||
| aren't needed here. The `.bn-mantine` element also has a margin that we don't | ||
| want to remove, so that too is commented out (would normally just remove the | ||
| margin on `body`). */ | ||
| /* margin: 0; | ||
matthewlipski marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| font-family: var(--mantine-font-family); | ||
| font-size: var(--mantine-font-size-md); | ||
| line-height: var(--mantine-line-height); | ||
| background-color: var(--mantine-color-body); | ||
| color: var(--mantine-color-text); */ | ||
|
|
||
| .bn-mantine .mantine-active:active { | ||
| transform: translateY(calc(0.0625rem * var(--mantine-scale))); | ||
| -webkit-font-smoothing: var(--mantine-webkit-font-smoothing); | ||
| -moz-osx-font-smoothing: var(--mantine-moz-font-smoothing); | ||
| } | ||
|
|
||
| .bn-mantine[dir="rtl"] .mantine-rotate-rtl { | ||
| transform: rotate(180deg); | ||
| @media screen and (max-device-width: em(500px)) { | ||
| .bn-mantine { | ||
| -webkit-text-size-adjust: 100%; | ||
| } | ||
| } | ||


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.
I think you need to update the .bnblock config file or whatever it is called. So that pnpm run gen doesn't overwrite these changes
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.
Is it not correct as is? In the
.bnexample.jsonfile, I updated the Mantine package version underdependencies, whichpnpm genuses to update the dependencies in the correspondingpackage.json. Or am I misunderstanding?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.
Ah maybe I missed it then