Skip to content

fix: label支持自动换行- #3409

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.

Sign up for GitHub

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

Open
yangyanAurora wants to merge 1 commit into
base: feat_v3.x
Choose a base branch
from
Open

fix: label支持自动换行- #3409

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions src/packages/formitem/formitem.scss
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,13 +133,13 @@
.nut-form-item-label-right {
justify-content: flex-end;
padding-right: 24px;
white-space: nowrap;
word-wrap: break-word;
Copy link

@coderabbitai coderabbitai bot Dec 25, 2025

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

# Check if file exists and get basic info
if [ -f "src/packages/formitem/formitem.scss" ]; then
  echo "File found"
  wc -l "src/packages/formitem/formitem.scss"
else
  echo "File not found, searching for similar files"
  fd "formitem.*scss" -t f
fi

Repository: jdf2e/nutui-react

Length of output: 111

🏁 Script executed:

# Read the relevant sections of the file to verify the changes
# Reading around lines 136 and 142 (the changed lines)
sed -n '10,25p' src/packages/formitem/formitem.scss | cat -n
echo "---"
sed -n '130,155p' src/packages/formitem/formitem.scss | cat -n

Repository: jdf2e/nutui-react

Length of output: 1244

🏁 Script executed:

# Search for all occurrences of word-wrap, white-space, and overflow-wrap in the file
echo "=== word-wrap occurrences ==="
rg -n "word-wrap" src/packages/formitem/formitem.scss
echo "=== white-space occurrences ==="
rg -n "white-space" src/packages/formitem/formitem.scss
echo "=== overflow-wrap occurrences ==="
rg -n "overflow-wrap" src/packages/formitem/formitem.scss

Repository: jdf2e/nutui-react

Length of output: 271

🏁 Script executed:

# Get the git diff for this file
git diff HEAD -- src/packages/formitem/formitem.scss | head -100

Repository: jdf2e/nutui-react

Length of output: 43

🏁 Script executed:

# If that doesn't show recent changes, check git log
git log --oneline -n 20 -- src/packages/formitem/formitem.scss

Repository: jdf2e/nutui-react

Length of output: 87

🏁 Script executed:

# Show the actual commit with changes
git show 68aa0853 -- src/packages/formitem/formitem.scss

Repository: jdf2e/nutui-react

Length of output: 4240

移除 136 行和 142 行的冗余 word-wrap 声明

这是一个新文件创建。第 18 行的父选择器 .nut-form-item-label 已经设置了 word-wrap: break-word，子选择器 .nut-form-item-label-right（第 136 行）和 .nut-form-item-label-left（第 142 行）无需重复声明该属性——它们会自动继承父样式。

建议删除第 136 行和 142 行的 word-wrap: break-word;，保持样式表的整洁。同时可考虑使用更现代的 CSS 属性 overflow-wrap: break-word 替代第 18 行的 word-wrap（两者功能相同，但 overflow-wrap 是标准属性）。

第 148 行的 .nut-form-item-top 中的 white-space: nowrap 是独立的样式设置，用于列布局的特定场景，与换行功能无直接关系。

🤖 Prompt for AI Agents 
In src/packages/formitem/formitem.scss around lines 136 and 142, remove the
redundant "word-wrap: break-word;" declarations from the child selectors
.nut-form-item-label-right (line 136) and .nut-form-item-label-left (line 142);
instead rely on the parent .nut-form-item-label rule (line 18) for wrapping;
additionally, replace the parent rule's non-standard "word-wrap: break-word" at
line 18 with the standard "overflow-wrap: break-word" to modernize the
stylesheet while keeping .nut-form-item-top (line 148) untouched.

}

.nut-form-item-label-left {
position: relative;
padding-left: 12px;
white-space: nowrap;
word-wrap: break-word;
}

.nut-form-item-top {
Expand Down
Loading