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

document: fix thumbnail caption #2844

Merged
merged 1 commit into from
Apr 25, 2022

Conversation

zannkukai
Copy link
Contributor

In the document detailed view, the figure caption behind the thumbnail
was vertically displayed. The reason was that the document_types jinja
filter return only a string (splitted on list of char) instead of a list
of string.

Co-Authored-by: Renaud Michotte renaud.michotte@gmail.com

How to test?

Before

image

After

image

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Cypress tests successful?

In the document detailed view, the figure caption behind the thumbnail
was vertically displayed. The reason was that the `document_types` jinja
filter return only a string (splitted on list of char) instead of a list
of string.

Co-Authored-by: Renaud Michotte <renaud.michotte@gmail.com>
@zannkukai zannkukai self-assigned this Apr 21, 2022
rero_ils/modules/documents/views.py Show resolved Hide resolved
rero_ils/modules/documents/views.py Show resolved Hide resolved
@zannkukai zannkukai merged commit 3c8f5f8 into rero:staging Apr 25, 2022
@zannkukai zannkukai deleted the zan-fix-again-issue-2623 branch April 25, 2022 08:15
@PascalRepond PascalRepond added the bug Breaks something but is not blocking label May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Breaks something but is not blocking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants