viewerjs icon indicating copy to clipboard operation
viewerjs copied to clipboard

add some events(full,exit,render)

Open tanlng opened this issue 1 year ago • 4 comments

Summary

What kind of change does this PR introduce? (check at least one)

  • [ ] Bugfix
  • [x] Feature
  • [ ] Code style update
  • [ ] Refactor
  • [ ] Docs
  • [ ] Build-related changes
  • [ ] Other, please describe:

If changing the UI of the default theme, please provide the before/after screenshot:

Does this PR introduce a breaking change? (check one)

  • [ ] Yes
  • [x] No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

  • [ ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

You have tested in the following browsers: (Providing a detailed version will be better.)

  • [x] Chrome
  • [ ] Firefox
  • [ ] Safari
  • [ ] Edge
  • [ ] IE

If adding a new feature, the PR's description includes:

  • [x] A convincing reason for adding this feature
  • [ ] Related documents have been updated
  • [ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

tanlng avatar Jun 28 '23 01:06 tanlng

Why not use the existing viewed event instead of the adding rendered event?

fengyuanchen avatar Jul 01 '23 09:07 fengyuanchen

Why not use the existing viewed event instead of the adding rendered event? image

I need to synchronously modify the position of the markers after rendering

tanlng avatar Jul 03 '23 12:07 tanlng

The moved, rotated, scaled, and zoomed events would also work in your case.

fengyuanchen avatar Jul 09 '23 04:07 fengyuanchen

I also need to trigger an event after resizing. If there is a render event, I will not need to update my code after updating new features

tanlng avatar Jul 10 '23 01:07 tanlng