-
Notifications
You must be signed in to change notification settings - Fork 0
chore: lib updates #246
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
chore: lib updates #246
Changes from all commits
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 |
|---|---|---|
|
|
@@ -32,27 +32,27 @@ | |
|
|
||
| <SectionTitle :title="task?.name ?? ''" /> | ||
|
|
||
| <div v-if="task?.description" class="w-full text-base/5 font-normal whitespace-pre-wrap break-words"> | ||
| <div v-if="task?.description" class="w-full text-base/5 font-normal whitespace-pre-wrap wrap-break-word"> | ||
|
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. 🧩 Analysis chainVerify that The class Run the following script to check if this custom utility is defined: Also applies to: 41-41, 48-48, 55-55 🏁 Script executed: #!/bin/bash
# Check if wrap-break-word is defined in Tailwind config
rg -n "wrap-break-word" --type-add 'config:*.{js,ts,mjs,cjs}' --type config -g '!node_modules' -g 'tailwind.config.*' -C3
# Also check for any CSS definitions
rg -n "\.wrap-break-word" --type css -g '!node_modules' -C3Length of output: 148 I'll use a different approach to verify this custom utility. Let me search for Tailwind config files and then check for the utility definition. 🏁 Script executed: #!/bin/bash
# Find tailwind config files
echo "=== Finding Tailwind config files ==="
fd -t f "tailwind" --max-depth 5
echo ""
echo "=== Searching for wrap-break-word in config files ==="
rg "wrap-break-word" -g "tailwind*" -g "*.config.*" -C3
echo ""
echo "=== Searching for wrap-break-word in the entire codebase ==="
rg "wrap-break-word" -C2
echo ""
echo "=== Checking for CSS files that might define this class ==="
fd -t f "\.(css|scss|less)$" -g "!node_modules" | head -20Length of output: 5124 Let me search for the Tailwind configuration in more conventional locations and check if 🏁 Script executed: #!/bin/bash
# Find tailwind config in common locations
echo "=== Looking for tailwind.config files in common locations ==="
find . -maxdepth 3 -name "tailwind.config.*" -o -name "tailwind.js" 2>/dev/null
echo ""
echo "=== Check package.json for tailwind references ==="
find . -maxdepth 3 -name "package.json" -exec grep -l "tailwind" {} \;
echo ""
echo "=== Search for 'break-words' (standard Tailwind utility) in codebase ==="
rg "break-words" -C2
echo ""
echo "=== List all CSS/SCSS files in the project ==="
fd "\.(css|scss)$" --max-depth 5 2>/dev/null | grep -v node_modules | head -20Length of output: 3614 Let me check the Tailwind configuration in the UI package: 🏁 Script executed: #!/bin/bash
# Check packages/ui/package.json for tailwind config
echo "=== Content of packages/ui/package.json (tailwind section) ==="
cat ./packages/ui/package.json | head -50
echo ""
echo "=== Check if there's a tailwind config within packages/ui ==="
find ./packages/ui -maxdepth 2 -type f \( -name "tailwind.config.*" -o -name "tailwind.js" -o -name "*.css" -o -name "*.scss" \) 2>/dev/null
echo ""
echo "=== Verify: are wrap-break-word and break-words in the same files? ==="
rg -l "wrap-break-word|break-words" apps/atrium-telegram/app/pages/task/\[taskId\]/index.vue -C1Length of output: 1453 Replace The class 🤖 Prompt for AI Agents |
||
| {{ task.description }} | ||
| </div> | ||
|
|
||
| <div v-if="task?.report" class="flex flex-row gap-2 items-start w-full"> | ||
| <UIcon name="i-lucide-clipboard-pen" class="shrink-0 size-6 text-primary" /> | ||
| <p class="text-base/5 font-semibold whitespace-pre-wrap break-words"> | ||
| <p class="text-base/5 font-semibold whitespace-pre-wrap wrap-break-word"> | ||
| {{ task.report }} | ||
| </p> | ||
| </div> | ||
|
|
||
| <div v-if="task?.date" class="flex flex-row gap-2 items-center w-full"> | ||
| <UIcon name="i-lucide-calendar" class="shrink-0 size-6 text-primary" /> | ||
| <p class="text-base/5 font-semibold whitespace-pre-wrap break-words"> | ||
| <p class="text-base/5 font-semibold whitespace-pre-wrap wrap-break-word"> | ||
| {{ format(new Date(task.date), 'd MMMM yyyy', { locale: ru }) }} | ||
| </p> | ||
| </div> | ||
|
|
||
| <div v-if="taskList" class="flex flex-row gap-2 items-center w-full"> | ||
| <UIcon name="i-lucide-book-marked" class="shrink-0 size-6 text-primary" /> | ||
| <p class="text-base/5 font-semibold whitespace-pre-wrap break-words"> | ||
| <p class="text-base/5 font-semibold whitespace-pre-wrap wrap-break-word"> | ||
| {{ taskList.name }} | ||
| </p> | ||
| </div> | ||
|
|
||
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,7 +1,7 @@ | ||||||
| <template> | ||||||
| <div | ||||||
| v-if="isCatalogPage && !channelStore.isKitchenOpened" | ||||||
| class="tg-safe-area -mb-[calc(var(--tg-viewport-content-safe-area-inset-top)+var(--tg-viewport-safe-area-inset-top))] tg-text-button bg-gradient-to-b from-indigo-950 to-indigo-800 motion-preset-slide-down" | ||||||
| class="tg-safe-area -mb-[calc(var(--tg-viewport-content-safe-area-inset-top)+var(--tg-viewport-safe-area-inset-top))] tg-text-button bg-linear-to-b from-indigo-950 to-indigo-800 motion-preset-slide-down" | ||||||
|
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. Invalid Tailwind class breaks gradient styling.
Apply this diff to restore the correct Tailwind class: - class="tg-safe-area -mb-[calc(var(--tg-viewport-content-safe-area-inset-top)+var(--tg-viewport-safe-area-inset-top))] tg-text-button bg-linear-to-b from-indigo-950 to-indigo-800 motion-preset-slide-down"
+ class="tg-safe-area -mb-[calc(var(--tg-viewport-content-safe-area-inset-top)+var(--tg-viewport-safe-area-inset-top))] tg-text-button bg-gradient-to-b from-indigo-950 to-indigo-800 motion-preset-slide-down"📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| > | ||||||
| <div class="p-4 max-w-[28rem] mx-auto tg-content-safe-area-top"> | ||||||
| <div class="pt-6 flex flex-row items-center gap-2"> | ||||||
|
|
||||||
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.
🧩 Analysis chain
Verify that
wrap-break-wordis a valid utility class.The class
wrap-break-wordis not a standard Tailwind CSS utility. The standard class isbreak-words. Ifwrap-break-wordis not defined as a custom utility in your Tailwind configuration, this change will break the text wrapping behavior.Run the following script to check if this custom utility is defined:
🏁 Script executed:
Length of output: 148
I'll help verify the
wrap-break-wordutility class. The initial script had overly restrictive filters. Let me try a broader search approach.🏁 Script executed:
Length of output: 7041
Now I need to find the Tailwind configuration and check if
wrap-break-wordis defined. Let me search for the config files and custom utility definitions.🏁 Script executed:
Length of output: 8214
Let me check the Nuxt configuration and CSS files where the Tailwind utility might be defined:
🏁 Script executed:
Length of output: 2721
Now let me check the CSS file where the custom utility should be defined:
🏁 Script executed:
Length of output: 4184
Let me check if
wrap-break-wordis defined in the@roll-stack/uipackage or other Tailwind configurations:🏁 Script executed:
Length of output: 9108
I found an important inconsistency. Let me check the
@roll-stack/uipackage to see if it defineswrap-break-word:Critical finding: The standard Tailwind CSS utility iswrap-break-word, which allows line breaks between letters in a word if needed. This is NOT the same asbreak-words(which is a different utility for word-breaking behavior).Let me verify this finding by checking the packages directory for any @roll-stack/ui configuration:
🏁 Script executed:
Length of output: 865
Let me check the UI package's CSS and configuration files:
🏁 Script executed:
Length of output: 3058
Replace
wrap-break-wordwithbreak-words— undefined CSS class.The verification confirms the review comment is correct. The class
wrap-break-wordis not defined anywhere in the codebase, the extended@roll-stack/uipackage, or as a standard Tailwind utility. The correct Tailwind class isbreak-words, which is already used consistently elsewhere in the same app (e.g.,ticket/Card.vue,flow/ItemCard.vue,PartnerAgreementCard.vue).This affects multiple files in
apps/atrium-telegram:app/pages/agreement/[agreementId]/index.vueapp/pages/task/[taskId]/index.vueapp/components/KitchenCard.vueapp/components/TaskInfoCard.vueReplace all instances of
wrap-break-wordwithbreak-wordsto restore correct text wrapping behavior.🤖 Prompt for AI Agents