Skip to content

Feat/scrollbar always visible space correction #3776

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

Conversation

xxxace
Copy link
Contributor

@xxxace xxxace commented Apr 21, 2025

[中文版模板 / Chinese template]

🤔 This is a ...

  • New feature
  • Bug fix
  • TypeScript definition update
  • Bundle size optimization
  • Performance optimization
  • Enhancement feature
  • Refactoring
  • Update dependency
  • Code style optimization
  • Test Case
  • Branch merge
  • Site / documentation update
  • Demo update
  • Workflow
  • Chore
  • Release
  • Other (about what?)

🔗 Related issue link

#3742

💡 Background and solution

📝 Changelog

Language Changelog
🇺🇸 English scrollbar always visible space correction
🇨🇳 Chinese scrollStyle.visible = 'always' 时没触发滚动条高度则不占用宽度

☑️ Self-Check before Merge

⚠️ Please check all items below before requesting a reviewing. ⚠️

  • Doc is updated/provided or not needed
  • Demo is updated/provided or not needed
  • TypeScript definition is updated/provided or not needed
  • Changelog is provided or not needed

🚀 Summary

copilot:summary

🔍 Walkthrough

copilot:walkthrough

@fangsmile fangsmile changed the base branch from develop to feat/scrollbar-always-visible-space-correction May 7, 2025 06:20
@fangsmile fangsmile merged commit 28237eb into VisActor:feat/scrollbar-always-visible-space-correction May 7, 2025
2 checks passed
@fangsmile
Copy link
Contributor

@xxxace 这个优化可以适用到其他的各种设置情况 不局限于always。希望可以再一起把其他的情况一起考虑下,谢谢! 目前已经合入#3833 可以再提个pr 合到分支 feat/scrollbar-always-visible-space-correction

@fangsmile
Copy link
Contributor

@xxxace 这个功能有较多bug 请各种情况测试后在重写提交pr吧

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants