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/aut 3923/textreader image figure support #2613

Merged
merged 7 commits into from
Nov 14, 2024

Conversation

olga-kulish
Copy link
Contributor

@olga-kulish olga-kulish commented Nov 8, 2024

Related to https://oat-sa.atlassian.net/browse/AUT-3923

Changes

  1. htmlEditor.js & elementSupport.js & Img.js
    When you position image as left/right/center,

  2. Widget.js
    Explicitly set contenteditable=false, because otherwise <figcaption> is editable in TextReaderInteraction PCI. It is not editable in A-block or QTI interaction prompt/choice.
    I have no idea what's the difference and how to implement this properly.

  3. item-creator.scss
    When you wrap image inside Choice of ChoiceInteraction, it goes outside Choice borders.

Test

On unit05. Related PRs:

Copy link

github-actions bot commented Nov 8, 2024

Front-end summary Node 18

💯 Total ✅ Passed ⏭️ Skipped ❌ Failed
287 287 0 0

Copy link
Contributor

@oatymart oatymart left a comment

Choose a reason for hiding this comment

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

Looks fine.

@olga-kulish olga-kulish self-assigned this Nov 13, 2024
Copy link

Version

Target Version 30.22.4
Last version 30.22.3

There are 0 BREAKING CHANGE, 0 feature, 3 fixes

@olga-kulish olga-kulish merged commit f4ac324 into develop Nov 14, 2024
5 checks passed
@olga-kulish olga-kulish deleted the fix/AUT-3923/textreader-image-figure-support branch November 14, 2024 10:16
wazelin added a commit to oat-sa/tao-community that referenced this pull request Nov 21, 2024
…eases/tag/v54.20.1.1)**

- _Fix_ [AUT-3923](https://oat-sa.atlassian.net/browse/AUT-3923) : textreader image figure support [#4125](oat-sa/tao-core#4125) ( by [olga-kulish](https://github.com/olga-kulish) - validated by [olga-kulish](https://github.com/olga-kulish)
**[oat-sa/extension-tao-itemqti v30.20.0.2](https://github.com/oat-sa/extension-tao-itemqti/releases/tag/v30.20.0.2)**
 - _Fix_ [AUT-3923](https://oat-sa.atlassian.net/browse/AUT-3923) : textreader image figure support [#2613](oat-sa/extension-tao-itemqti#2613) ( by [olga-kulish](https://github.com/olga-kulish) - validated by [olga-kulish](https://github.com/olga-kulish) )
 - _Fix_ [AUT-3876](https://oat-sa.atlassian.net/browse/AUT-3876) : editor passage remove [#2605](oat-sa/extension-tao-itemqti#2605) ( by [olga-kulish](https://github.com/olga-kulish) - validated by [olga-kulish](https://github.com/olga-kulish) )
**[oat-sa/extension-pcisample v3.9.2](https://github.com/oat-sa/extension-pcisample/releases/tag/v3.9.2)**
 - _Fix_ [AUT-3923](https://oat-sa.atlassian.net/browse/AUT-3923) : textreader image figure support [#172](oat-sa/extension-pcisample#172) ( by [olga-kulish](https://github.com/olga-kulish) - validated by [olga-kulish](https://github.com/olga-kulish) )
**[oat-sa/extension-pcisample v3.9.1](https://github.com/oat-sa/extension-pcisample/releases/tag/v3.9.1)**
 - _Fix_ [AUT-3876](https://oat-sa.atlassian.net/browse/AUT-3876) : editor passage remove [#171](oat-sa/extension-pcisample#171) ( by [olga-kulish](https://github.com/olga-kulish) - validated by [olga-kulish](https://github.com/olga-kulish) )
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

Successfully merging this pull request may close these issues.

2 participants