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: respect locked content when returning to location (fixes #51) #71

Merged
merged 6 commits into from
Jun 20, 2024

Conversation

chris-steele
Copy link
Contributor

@chris-steele chris-steele commented Jun 19, 2024

#51

Fix

  • respect locked content when returning to location

Notes

If the bookmark is of type contentobject and the location is locked or preceded by locked content then the user is returned to the nearest unlocked contentobject prior to the location.

If the bookmark is of type article/block/component and the location is preceded by locked content then the user is returned to the nearest point prior to the location that is not itself preceded by locked content.

Co-authored-by: Oliver Foster <oliver.foster@kineo.com>
@oliverfoster
Copy link
Member

Would you mind commenting on why offset is needed? @chris-steele

@chris-steele
Copy link
Contributor Author

@oliverfoster sorry I misinterpreted the way trickle would lock - have removed offset

Co-authored-by: Oliver Foster <oliver.foster@kineo.com>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👀

@cahirodoherty-learningpool cahirodoherty-learningpool merged commit d1e8fbb into master Jun 20, 2024
@cahirodoherty-learningpool cahirodoherty-learningpool deleted the issue/51 branch June 20, 2024 14:37
github-actions bot pushed a commit that referenced this pull request Jun 20, 2024
## [4.3.1](v4.3.0...v4.3.1) (2024-06-20)

### Fix

* respect locked content when returning to location (fixes #51) (#71) ([d1e8fbb](d1e8fbb)), closes [#51](#51) [#71](#71)
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.

5 participants