-
Notifications
You must be signed in to change notification settings - Fork 28
Conversation
fec4149
to
a0a275c
Compare
src/modals/masq_activating_modal.js
Outdated
|
||
this.modal = new Modal(this, MasqActivatingModalView, false) | ||
|
||
this.opened = false |
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.
this flag is not used. And as this.activating
is true by default, the modal and the video are present in DOM at app start. (Chromium at least tries to load the .mp4 before it is visible)
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.
Fixed !
removed opened flag and made activating false by default
src/views/masq_activating_modal.dot
Outdated
<div class="modal__masq_activating__body"> | ||
{{? this.activating}} | ||
<div class="modal__masq_activating__animation"> | ||
<video autoplay loop src="{{= window.location.origin }}/statics/images/masq/gifs/masq_loading.mp4"></video> |
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.
Using "origin" is incorrect here: the "baseUrl" ( /maps/...) is missing.
Actually an absolute path can directly be used in templates, since a <base>
tag is defined in the html head:
<video autoplay loop src="{{= window.location.origin }}/statics/images/masq/gifs/masq_loading.mp4"></video> | |
<video autoplay loop src="/statics/images/masq/gifs/masq_loading.mp4"></video> |
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.
👍
a0a275c
to
4018f0b
Compare
4018f0b
to
3fb5a2d
Compare
No description provided.