-
Notifications
You must be signed in to change notification settings - Fork 457
feat: tailwind-poc #6105
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
base: main
Are you sure you want to change the base?
feat: tailwind-poc #6105
Conversation
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Docker builds report
|
.flex { | ||
@include flexbox(); | ||
@include flex-direction(column) | ||
// @include flex-direction(column) |
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.
/> | ||
<JSONReference title={'Invite Links'} json={inviteLinks} /> | ||
|
||
<p>yolo</p> |
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.
p { | ||
color: var(--text-primary) !important; | ||
} | ||
} |
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.
Bug: Global Paragraph Color Override Bug
The global p { color: var(--text-primary) !important; }
rule forces all paragraph text to be red in light mode (yellow in dark mode). The !important
flag overrides existing styles, breaking the design system and semantic text colors across the application. This appears to be debug code.
{overflow.length > 0 && !isMeasuring && ( | ||
<div | ||
className='d-flex align-items-center justify-content-center ms-2 flex-shrink-0' | ||
className='d-flex align-items-center justify-content-center ms-2 flex-shrink-0 btn btn-secondary' |
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.
Bug: Overflow Button Styling Conflict
Adding btn btn-secondary
classes to the overflow button div
conflicts with its existing inline styles (e.g., backgroundColor
, borderRadius
, height
, width
). This changes the visual appearance of the tabs overflow menu and is semantically incorrect for a div
element.
@@ -1,10 +1,54 @@ | |||
@tailwind base; | |||
@import "tailwind.css"; |
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.
Bug: Tailwind Base Styles Duplicated, Variables Undefined
The @tailwind base
directive is included twice: once directly and again via the tailwind.css
import. This duplicates Tailwind's base styles, increasing bundle size and potentially causing CSS specificity issues. Additionally, these Tailwind imports occur before the variables
file, meaning custom SCSS variables are not defined when Tailwind's base styles are processed.
}} | ||
> | ||
<span>{name}</span> | ||
<span className='dark:text-red-500'>{name}</span> |
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.
|
||
// Text colors | ||
text: { | ||
DEFAULT: '#FF0000', |
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.
Thanks for submitting a PR! Please check the boxes below:
docs/
if required so people know about the feature!Changes
Please describe.
How did you test this code?
Please describe.
Note
Add Tailwind CSS and Autoprefixer via PostCSS, wire into webpack, and apply initial Tailwind-based style/class updates.
tailwindcss
,autoprefixer
, andeslint-plugin-tailwindcss
; createpostcss.config.js
.tailwind.config.js
with custom theme (colors, fonts, radii, spacing, dark mode via class, preflight disabled).postcss-loader
forcss
/scss
inlocal
,prod
, and Django builds.web/styles/tailwind.css
and reference inweb/main.js
andstyles.scss
..tabs
base styles.components/_input.scss
commented out.d-flex
→flex
, addbtn btn-secondary
,dark:text-red-500
).FeatureAnalytics.container.tsx
,Tabs.tsx
,OrganisationNavbar.tsx
,UsersAndPermissionsPage.tsx
.Written by Cursor Bugbot for commit 680d781. This will update automatically on new commits. Configure here.