-
Notifications
You must be signed in to change notification settings - Fork 260
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
Improve loading experience for message on thread #7562
Conversation
4243dcc
to
f958e7f
Compare
Signed-off-by: greta <gretadoci@gmail.com>
Signed-off-by: greta <gretadoci@gmail.com>
Signed-off-by: greta <gretadoci@gmail.com>
Signed-off-by: greta <gretadoci@gmail.com>
7b9a3f3
to
4997fd3
Compare
src/components/ThreadEnvelope.vue
Outdated
@@ -490,6 +491,7 @@ export default { | |||
margin-right: 10px; | |||
background-color: var(--color-main-background); | |||
padding-bottom: 28px; | |||
min-height: 150px; |
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 doesn't seem to work. the message to iframe loader still causes a jump
the loading starts at sec 16 |
|
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.
👍 🐘
@@ -46,5 +46,16 @@ export default { | |||
<style scoped> | |||
#mail-content { | |||
height: 450px; | |||
animation: show 200ms 80ms cubic-bezier(0.38, 0.97, 0.56, 0.76) forwards; | |||
opacity: 0.3; |
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.
👀
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.
Next to the jumping loading icon, 2 other things (possibly for other PRs, your call):
- The "Loading thread" message in the beginning is not needed, just show the loading thread directly
- The height of the container is still very small. Only mails which are just 1 sentence or word long will fit in there. So the default message container height (when the loading spinner is shown) can be increased. E.g. let's say that a mail with 5-7 lines of text could fit.
Signed-off-by: greta <gretadoci@gmail.com>
Superseded by #7615 |
ref #6045
Add minimum height to element so the spinner looks less broken. This should be less than an action message
After the loading finished, animate that the element grows to the actual message size (especially text messages)