-
Notifications
You must be signed in to change notification settings - Fork 3
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 : Inserted images/videos aren't considred as a content in news - EXO-65311 #905
Conversation
0c4a984
to
4992ade
Compare
@@ -395,23 +395,29 @@ export default { | |||
editMode: function() { | |||
return this.activityId && this.activityId !== ''; | |||
}, | |||
hasMediaContent() { | |||
const parsedDocument = new DOMParser().parseFromString(this.news.body, 'text/html'); | |||
const mediaElements = parsedDocument.querySelectorAll('img, video, iframe'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unfortunately this is still insufficient as it is limited to those tags and does not cover all possible HTML tags that could be added (svg, canva, audio, etc ...) even if it not the case now, but everytime a new tag is allowed, we will have to update the fix.
I prefer doing comparison of raw HTML content
2b2d91a
to
88f99b3
Compare
0eb2fcd
to
5bb122a
Compare
Your PR triggers too many exo-ci builds! Please finish your work and then, set your PR ready! Thank you |
…play - EXO-65200 (#906) Prior to this change, when create news in three different space, go to news app, open hamburger drawer besides posted articles unselect all spaces and select only one space , copy the UI url and paste it in a new taw on notice the filter glitches, all news are displayed. After this change, the filter is already mentioned in the url.
5bb122a
to
e330fbc
Compare
e330fbc
to
f42fd66
Compare
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
…XO-65311 (#905) Before this change, when a user inserted an image or a video into the body of the news during the creation or modification process, the post button would be disabled. After this change, the post button is enabled when the news body isn't empty and the update button is enabled when a new content is inserted.
…XO-65311 (#905) Before this change, when a user inserted an image or a video into the body of the news during the creation or modification process, the post button would be disabled. After this change, the post button is enabled when the news body isn't empty and the update button is enabled when a new content is inserted.
Before this change, when a user inserted an image or a video into the body of the news during the creation or modification process, the post button would be disabled. After this change, the post button is enabled when the news body isn't empty and the update button is enabled when a new content is inserted.