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

As an admin/content editor, I want to see all the images associated with a document so that I can determine whether I need to associate more images, clone the record, etc. #1109

Closed
2 tasks done
mrustow opened this issue Sep 20, 2022 · 1 comment
Assignees

Comments

@mrustow
Copy link

mrustow commented Sep 20, 2022

testing notes (qa)

On the admin site (QA), edit a document with multiple images, some of which do not belong on the document, i.e. are not selected. (Or make some image selections in the form yourself, in the Fragment section of the form—then save if you made any changes.)

  • In the side-by-side image/transcription viewer, confirm all images are equal size, with no gray overlays, and the selected one(s) have a yellow border.
  • on the public view for the same document, confirm that unselected images are grayed out and smaller as before

... but the greyed-out images are hard to see. So once they're no longer greyed out, the chosen images could be marked in some other way, perhaps with the yellow frames we now use for the thumbnails (or another idea you may have).

Then there's the question of do we need to zoom in on them. I'm not as worried about that, since we're just talking about eyeballing the fragment to decide what to do with it, so I say no, not at this stage.

Thanks!

@blms blms self-assigned this Sep 20, 2022
blms added a commit that referenced this issue Sep 20, 2022
Also pass new iiif dict to admin (ref #1109)
blms added a commit that referenced this issue Sep 20, 2022
blms added a commit that referenced this issue Sep 21, 2022
Co-authored-by: Rebecca Sutton Koeser <rlskoeser@users.noreply.github.com>
blms added a commit that referenced this issue Sep 21, 2022
Revise admin display of non-document images (#1109)
@rlskoeser rlskoeser changed the title As an admin/content editor, I want to be able to see all the images associated with a document so that I can determine whether I need to associate more images, clone the record, etc. As an admin/content editor, I want to see all the images associated with a document so that I can determine whether I need to associate more images, clone the record, etc. Sep 22, 2022
@blms blms added the 🗜️ awaiting testing Implemented and ready to be tested label Sep 22, 2022
@kseniaryzhova
Copy link

@rlskoeser works perfectly, closing!

@rlskoeser rlskoeser removed the 🗜️ awaiting testing Implemented and ready to be tested label Oct 3, 2022
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

No branches or pull requests

4 participants