Odoo/addons/mail/static/src/components/thread_view/thread_view.js, branch 14.0

[FIX] mail: chatter autoloading loads more messages too quickly

2021-03-26T04:34:04+00:00

if you check on any chatter with a lot of messages, you will see that it starts
loading all the messages as soon as you scroll down even just 1 pixel.
This makes a lot of loading of the messages for no reason unnecessarily.
What should happen instead is that it should load more message only when
scrolling all the way down to the bottom.This works fine in chatwindow & discuss.

With this commit, resolved this issue by providing the exact scrollable element
to the message list from parent component to the child. so with proper element
we can have proper scrollable heights and other dimensions as well to handle
load more messages properly.

Task : 2382735

closes odoo/odoo#66584

Related: odoo/enterprise#16767
Signed-off-by: Sébastien Theys (seb)

[FIX] mail: fix keyboard shortcut issue for small screen

2021-03-10T13:30:18+00:00

Current behavior before PR:
When having a small screen, the composer in a chat window has no actual
shortcut to send the message.

Desired behavior after PR is merged:
'CTRL-Enter' keyboard shortcut will work for small screen size.

LINKS:
PR https://github.com//pull/65241
Task-2446076

closes odoo/odoo#65241

Signed-off-by: Sébastien Theys (seb)

[FIX] mail, *: significantly reduce number of useless renders

2021-01-21T17:43:49+00:00

* = hr_holidays, im_livechat, website_livechat

A huge amount of components where rendered for no reason, notably when typing a
new message (at every key press), which significantly slowed down the interface.

There were 2 causes to this number of renders:
- OWL rendering all children automatically, even if their props didn't change,
  unless `shouldUpdate` is overridden, which is now done.
- `useStore` selectors being way too generous in what they observed

task-2399731

closes odoo/odoo#62703

Signed-off-by: Alexandre Kühn (aku)

[FIX] mail: avoid marking hint as resolved before message list render

2020-11-18T13:52:26+00:00

This change highlighted issues with the async behavior of `useUpdate`, which
was no longer necessary and therefore reverted.

This in turn highlighted issue with animation of chat window unfolding, making
restoring of scroll position not work as intended due to the message list not
having its whole height available during some time after render. This animation
being minor (and even annoying in some cases) it was decided to remove it.

task-2387729

closes odoo/odoo#61881

Signed-off-by: Alexandre Kühn (aku)

[FIX] mail: fix race conditions, especially with scroll

2020-11-16T12:36:47+00:00

The `_update` of some components must take place before the `_update`
of other (usually sibling) components that might directly influence their
layout. This is in particular the case for `_update` that are relying on height.

Prevent force save of scroll position if there are pending hints to adjust it,
and prevent pending hints from overriding new scroll value.

Prevent flicker with message date.

This should fix all known issues related to incorrectly saving or restoring
scroll position.

task-2344226
task-2358066
task-2369332
task-2372339
task-2373741

closes odoo/odoo#61054

Related: odoo/enterprise#14736
Signed-off-by: Alexandre Kühn (aku)

[FIX] mail: make only composer as fixed in chatter

2020-10-30T14:26:29+00:00

task-2282325

closes odoo/odoo#60524

Related: odoo/enterprise#14481
Signed-off-by: Sébastien Theys (seb)

[FIX] mail: do not return members of channels in channel_info

2020-10-23T11:03:10+00:00

This PR has 3 parts:

The members fix itself, which is necessary for performance reasons especially
at init messaging, as per described in the task.

Adding the partner name in the typing payload, because due to not knowing all
members anymore, it is necessary to properly display who is typing.

Restrict restoring scroll position only if the corresponding scroll height is
available, to make the scroll feature and particularly the corresponding tests
more robust.

task-2360020

closes odoo/odoo#60091

Signed-off-by: Alexandre Kühn (aku)

[FIX] mail, *: fix opens and redirects, adapt messaging menu close

2020-08-28T16:15:15+00:00

task-2244279
task-2262862
task-2274993
task-2277537
task-2282238

closes odoo/odoo#56745

X-original-commit: 109f202
Related: odoo/enterprise#12761
Signed-off-by: Alexandre Kühn (aku) 
Signed-off-by: Sébastien Theys (seb)

[FIX] mail, *: rename thread viewer to thread view

2020-08-28T13:43:54+00:00

The "viewer" should be the "parent" not the view itself. This distinction will
become important in a future commit to allow viewers to not display the view in
some situations.

Rename done as a standalone commit to ease diff readability.

Part of task-2310623

closes odoo/odoo#56737

X-original-commit: 1b811e6a972d72de02965adda67c5b039bee023f
Related: odoo/enterprise#12756
Signed-off-by: Alexandre Kühn (aku) 
Signed-off-by: Sébastien Theys (seb)