Skip to content
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

Jump to bottom

fix: paste needs to cover border #3982

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

fix: paste needs to cover border #3982

wants to merge 1 commit into from
+17 −0

Conversation

ybzky
Copy link
Member

@ybzky ybzky commented Nov 6, 2024

close https://github.com/dream-num/univer-pro/issues/2579

Pull Request Checklist

  • Related tickets or issues have been linked in the PR description (or missing issue).
  • Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
  • Unit tests have been added for the changes (if applicable).
  • Breaking changes have been documented (or no breaking changes introduced in this PR).

@ybzky ybzky requested a review from wzhudev as a code owner November 6, 2024 13:45
@univer-bot univer-bot bot added the qa:untested This PR is ready to be tested label Nov 6, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024 •
edited
Loading

View Deployment

📑 Examples 📚 Storybook
🔗 Preview link 🔗 Preview link

@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 6, 2024

Playwright test results

passed  19 passed

Details

stats  19 tests across 9 suites
duration  4 minutes, 39 seconds
commit  c5fe59f
info  For more information, see full report

@dream-num dream-num deleted a comment from univer-bot bot Nov 7, 2024
@ybzky ybzky requested a review from yuhongz November 7, 2024 06:17
@zhaolixin7
Copy link

@ybzky 复制后撤销，再次粘贴，边框线未被覆盖
2024_11_7 19_35_40 snapshot.json

2024_11_7.19_34_45.video.webm

@univer-bot
Copy link

univer-bot bot commented Nov 7, 2024

Bot detected the issue body's language is not English, translate it automatically. 👯👭🏻🧑‍🤝‍🧑👫🧑🏿‍🤝‍🧑🏻👩🏾‍🤝‍👨🏿👬🏿

@ybzky Undo after copying and paste again, the border line is not covered
2024_11_7 19_35_40 snapshot.json

2024_11_7.19_34_45.video.webm

@univer-bot univer-bot bot removed the qa:untested This PR is ready to be tested label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wzhudev wzhudev Awaiting requested review from wzhudev wzhudev is a code owner

@yuhongz yuhongz Awaiting requested review from yuhongz

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
qa:rejected
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ybzky @zhaolixin7