fix(sheets-ui): after unhiding row or col icon doesn't disappear

[kovsu]

close #2063

before:

2024-04-25.18.46.51.mov

after:

2024-04-25.18.45.52.mov

[github-actions]

View Deployment

📑 Examples	📚 Storybook
🔗 Preview link	🔗 Preview link

[oumomomo]

同时设置多个隐藏行时，还是有问题

Univer.Sheets.-.-.Microsoft.Edge.2024-04-25.19-55-14.mp4

[univer-bot]

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

There is still a problem when setting multiple hidden rows at the same time

Univer.Sheets.-.-.Microsoft.Edge.2024-04-25.19-55-14.mp4

[wzhudev]

@kovsu Thanks for opening this PR!

Univer uses a model -> rendering approach. After some investigation, I noticed that _clearShapes is not invoked in _update when a related mutation is triggered:

univer/packages/sheets-ui/src/controllers/header-unhide.render-controller.ts

Line 103 in 1f03b7a

this._update(workbook, worksheet);

If you would like to make a more proper fix, we would be very happy to merge this PR. Thanks for your effort! ❤️

[wzhudev]

Should use another approach. Thanks!

[kovsu]

Should use another approach. Thanks!

明天修 ：）

[univer-bot]

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

Should use another approach. Thanks!

Will fix it tomorrow :)

[kovsu]

@oumomomo 这样是正常的吧

2024-04-26.08.29.56.mov

[univer-bot]

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

@oumomomo This is normal, right?

2024-04-26.08.29.56.mov