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.

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

fix(sheets-hyper-link): hyper-link operator issues #3628

Merged
merged 18 commits into from
Oct 8, 2024

Conversation

weird94
Copy link
Contributor

@weird94 weird94 commented Sep 28, 2024

close https://github.com/dream-num/univer-pro/issues/2708
close https://github.com/dream-num/univer-pro/issues/2719
close https://github.com/dream-num/univer-pro/issues/2631
close https://github.com/dream-num/univer-pro/issues/2691
close https://github.com/dream-num/univer-pro/issues/2504

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).

Copy link

github-actions bot commented Sep 28, 2024

View Deployment

📑 Examples 📚 Storybook
🔗 Preview link 🔗 Preview link

Copy link

github-actions bot commented Sep 28, 2024

Playwright test results

failed  2 failed
passed  9 passed

Details

stats  11 tests across 5 suites
duration  3 minutes, 55 seconds
commit  37facab
info  For more information, see full report

Failed tests

chromium › visual-comparison/docs/docs-visual-comparison.spec.ts › diff default doc content
chromium › visual-comparison/sheet/sheets-visual-comparison.spec.ts › diff demo sheet content

@univer-bot univer-bot bot added the qa:untested This PR is ready to be tested label Sep 28, 2024
Copy link

codecov bot commented Sep 28, 2024

Codecov Report

Attention: Patch coverage is 1.56250% with 63 lines in your changes missing coverage. Please review.

Project coverage is 31.80%. Comparing base (71229b3) to head (37facab).
Report is 1 commits behind head on dev.

Files with missing lines Patch % Lines
...docs/data-model/text-x/build-utils/text-x-utils.ts 0.00% 16 Missing ⚠️
packages/sheets-ui/src/common/utils.ts 0.00% 11 Missing ⚠️
...ui/src/components/range-selector/RangeSelector.tsx 0.00% 9 Missing ⚠️
packages/design/src/components/popup/RectPopup.tsx 0.00% 8 Missing ⚠️
...ets-hyper-link-ui/src/views/CellLinkEdit/index.tsx 0.00% 7 Missing ⚠️
...re/src/docs/data-model/text-x/build-utils/parse.ts 20.00% 4 Missing ⚠️
...ackages/docs-ui/src/basics/custom-range-factory.ts 0.00% 4 Missing ⚠️
...sheets-hyper-link-ui/src/services/popup.service.ts 0.00% 2 Missing ⚠️
...services/selection/doc-selection-render.service.ts 0.00% 1 Missing ⚠️
...ers/render-controllers/scroll.render-controller.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              dev    #3628      +/-   ##
==========================================
- Coverage   31.81%   31.80%   -0.02%     
==========================================
  Files        2225     2225              
  Lines      115601   115648      +47     
  Branches    25447    25462      +15     
==========================================
+ Hits        36776    36777       +1     
- Misses      78825    78871      +46     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@oumomomo
Copy link

oumomomo commented Sep 29, 2024

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

1、选中链接点击取消链接,左键点击其他单元格时面板不消失
2024_9_29 10_46_41 video.webm

2、选中链接点击取消链接,使用快捷键crtl+B,加粗不生效
2024_9_29 16_28_28 video.webm

@univer-bot
Copy link

univer-bot bot commented Sep 29, 2024

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

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

  1. Select a link and click to cancel the link. The panel will not disappear when you left-click on other cells.
    2024_9_29 10_46_41 video.webm

  2. Select the link, click to cancel the link, and use the shortcut key crtl+B. Bold will not take effect.
    2024_9_29 16_28_28 video.webm

@univer-bot univer-bot bot removed the qa:untested This PR is ready to be tested label Sep 29, 2024
@weird94
Copy link
Contributor Author

weird94 commented Sep 29, 2024

dream-num/univer-pro#2719 选中单元格点击添加链接,左键点击其他单元格时面板不消失 2024_9_29 10_46_41 video.webm

fixed

@univer-bot
Copy link

univer-bot bot commented Sep 29, 2024

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

dream-num/univer-pro#2719 Select a cell and click Add Link. The panel does not disappear when you left-click on other cells. 2024_9_29 10_46_41 video.webm

fixed

@weird94 weird94 added qa:untested This PR is ready to be tested and removed qa:rejected labels Sep 29, 2024
@oumomomo oumomomo added the qa:verified This PR has already by verified by a QA and is considered good enough to be merge label Sep 29, 2024
@univer-bot univer-bot bot removed the qa:untested This PR is ready to be tested label Sep 29, 2024
@weird94 weird94 changed the title fix(sheets-hyper-link): hyper-link range-selector fix(sheets-hyper-link): hyper-link operator issues Sep 29, 2024
@oumomomo
Copy link

https://github.com/dream-num/univer-pro/issues/2504
在文档内取消链接后使用上下左右键,光标移动的位置不对
操作步骤:
1、在文档内添加链接
2、将光标移动到链接最后方
3、取消超链接后使用上下左右键
image

@univer-bot
Copy link

univer-bot bot commented Sep 29, 2024

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

https://github.com/dream-num/univer-pro/issues/2504
After canceling the link in the document, the cursor moves to the wrong position when using the up, down, left and right keys.
Operation steps:

  1. Add a link to the document
  2. Move the cursor to the end of the link
  3. Use the up, down, left and right keys after canceling the hyperlink.
    image

@oumomomo oumomomo added qa:rejected and removed qa:verified This PR has already by verified by a QA and is considered good enough to be merge labels Sep 29, 2024
@weird94
Copy link
Contributor Author

weird94 commented Oct 8, 2024

dream-num/univer-pro#2504 在文档内取消链接后使用上下左右键,光标移动的位置不对 操作步骤: 1、在文档内添加链接 2、将光标移动到链接最后方 3、取消超链接后使用上下左右键 image

fixed

@univer-bot
Copy link

univer-bot bot commented Oct 8, 2024

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

dream-num/univer-pro#2504 When using the up, down, left and right keys after canceling the link in the document, the cursor moves to the wrong position. Steps: 1. Add a link in the document 2. Move the cursor to the end of the link 3. Use the up, down, left and right keys after canceling the hyperlink![image](https://private-user-images.githubusercontent.com/167171356/371836427 -99f7bbbe-a236-4468-8363-295f826201f2.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb2 50ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjgzNzAyNDcsIm5iZiI6MTcyODM2OTk0N ywicGF0aCI6Ii8xNjcxNzEzNTYvMzcxODM2NDI3LTk5ZjdiYmJlLWEyMzYtNDQ2OC04MzYzLTI5 NWY4MjYyMDFmMi5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQxMDA4JTJGdXMtZWFzdC0xJ TJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MTAwOFQwNjQ1NDdaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1lMGE0NTlhNjI4MGEyZWNmMDg1MDAxNzEzM zZhZjdjNDc1YTQ2N2RkNjE2MWVhMDhiYjM2YzZiYWQ4M2RkMjc2JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCJ9.v5b8DyNCUALKIRbl_WCQjrKYObv8Sw0rFwWgSbOFK3E)

fixed

@weird94 weird94 added qa:untested This PR is ready to be tested and removed qa:rejected labels Oct 8, 2024
@weird94
Copy link
Contributor Author

weird94 commented Oct 8, 2024

/update-snapshots

3 similar comments
@weird94
Copy link
Contributor Author

weird94 commented Oct 8, 2024

/update-snapshots

@weird94
Copy link
Contributor Author

weird94 commented Oct 8, 2024

/update-snapshots

@weird94
Copy link
Contributor Author

weird94 commented Oct 8, 2024

/update-snapshots

@weird94 weird94 removed the qa:untested This PR is ready to be tested label Oct 8, 2024
@weird94 weird94 merged commit e27bf5e into dev Oct 8, 2024
8 of 9 checks passed
@weird94 weird94 deleted the fix/hyper-link-range-selector branch October 8, 2024 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants