hypothesis / product-backlog

Where new feature ideas and current bugs for the Hypothesis product live
118 stars 7 forks source link

[VitalSource] Discovery: Constrain annotation to assignment #1256

Closed chrisshaw closed 7 months ago

chrisshaw commented 2 years ago

Restricting user access to the content is a common theme when talking with instructors. But this limits some of the potential engagement of the students. We'd like to thread the needle between those needs.

We also have multiple possible implementation ideas that have various trade-offs

Question:

What we should learn

How we hope to learn it

Notes

chrisshaw commented 2 years ago

Candidate for Lyza

dwhly commented 2 years ago

A few potential issues w/ this: 1- It adds a UX complexity. Having to define an "assignment", from one place in a text to another. 2- It has the potential to create "hidden" and unexplained behaviors (why does the group control mysteriously change when I go from here to there.) 3- It removes agency from students. For instance, why can I not ask a question of my peers in a different place in the text? (Note that we're 4- It can be "solved" simply by adding instructions to assignments. (You're expected to create X annotations and reply to Y others on Chapter 3). If they do more than that or "work ahead" of where they're supposed to be, shouldn't that be ok?

Of course it may be useful for us to more precisely allow instructors to indicate the beginning of (and possibly the end) of an assignment (e.g. not just at the beginning of Chapter 3, but 5 pages in, on page 135.) Possibly, one solution is (at some point in the future) to allow instructors to define these points, and provide visual clues for them, but not necessarily to constrain annotation functions around them.

robertknight commented 7 months ago

We've implemented a feature that addresses this. See https://github.com/hypothesis/lms/pull/6111 for a summary.