| Age | Commit message (Collapse) | Author |
|
If the chat window was initially folded and when a new message is received
the chat window automatically unfolds. it is annoying having a window that
pops up each time a new message is coming while having folded it once.
So, It should remain folded when a new message is received if the chat
window is already folded.
Task : 2522141
closes odoo/odoo#70737
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
Display the default name (the one displayed in the kanban view) for followers replacing
the False value for "other address" partners, this way the user won't get
confused, furthermore, this fix won't require the name to be mandatory for this
type of contacts.
Task-2514244
closes odoo/odoo#70391
X-original-commit: 4c1271738daa039e79a11646fef3f4264e4d96d2
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
|
|
What are the steps to reproduce your issue ?
1. Create a record and add two attachments
2. Navigate quickly between these attachments
What is currently happening ?
Traceback
TypeError: Cannot read property 'complete' of undefined
at AttachmentViewer._handleImageLoad (https://www.odoo.com/mail/static/src/components/attachment_viewer/attachment_viewer.js:163:20)
opw-2521901
closes odoo/odoo#70309
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
In the chatter, the attachment preview image size is fixed with 160*160
but the actual preview area is 200*200px so the image is not filling correctly.
It appears padding around the image, especially visible when hovering it.
So with this commit, We should make sure to use a properly sized image in the
preview. we have to also provide the perfect resolution and preview for
the small images without upscaling incorrectly. For large images, it should be
display based on the aspect ratio.
Task : 2483885
closes odoo/odoo#68696
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
Task-2486731
closes odoo/odoo#68405
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
PURPOSE:
command suggestion should only open if command is the first char.
SPECIFICATION:
command suggestion should only open if command is the first char.
Task-2487518
closes odoo/odoo#68376
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
task-2412150
closes odoo/odoo#64072
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
PURPOSE:
Currently typing status is sent when typing a command.
SPECIFICATION:
Typing status should not be sent when typing a command.
Task-2363267
closes odoo/odoo#66821
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
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) <seb@odoo.com>
|
|
The code from v13 used to look for partners in JS before making the RPC, which
was much faster, and also gave better results by returning partners that were
already known and therefore more likely to be selected.
The same logic is reintroduced here, and further improved to take into account
all known partners and sort them according to the likeliness they will be
selected based on various criteria.
task-2413776
closes odoo/odoo#64161
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
Co-authored-by: Sébastien Theys <seb@odoo.com>
|
|
Currently, In channels listing, the order of the channels is case sensitive.
The Uppercase names are listing first and then all the Lowercase ones.
but it should be displayed in the alphabetical case insensitive order.
With this commit, we have resolved this issue and now channels are listing
in case insensitive alphabetical manner.
Task : 2453592
closes odoo/odoo#65579
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
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) <seb@odoo.com>
|
|
When formatting a message, a message could be with an empty or obsolete
record_name field. It's better to rely on the relation between the message and
the thread and fetch the current thread name.
task-2411715
closes odoo/odoo#63000
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
task-2438828
closes odoo/odoo#65183
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
When we search for a name in the chat window having a very long name
of the user will overflowing the screen. The autocomplete dropdown list is
overflowing and rendering out of the screen.
With this commit, we have resized the width of autocomplete dropdown list with
the width of the outer element's width as it's standard behavior.
Task : 2449115
closes odoo/odoo#66238
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Before this commit:
For chatter in window, the name of the record is not in an appropriate place in
header as there is an empty space before name.
After this commit:
The name of the record is aligned to the left, and there is no empty space
before the name.
LINKS
Task- 2442652
PR https://github.com/odoo/odoo/pull/65600
closes odoo/odoo#65600
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Before this commit:
When a new conversation is initiated, the 'new messages' separator is displayed.
This looks a bit weird as there is no history of conversation and thus no older
messages from which new messages should be separated.
After this commit:
The 'new messages' separator will not be displayed in new conversations.
LINKS
PR https://github.com/odoo/odoo/pull/65952
Task-2453511
closes odoo/odoo#65952
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
Before this commit:
When chat window is focused odoo shortcuts (with alt) did not work because of
there is stop propagation in _onKeydown method.
After this commit:
Remove stop propagation. Now odoo shortcuts (with alt) work
properly.
Task-2388829
closes odoo/odoo#65478
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
Before this commit:
If a next activity is "triggered", the discard button is not visible.
After this commit:
Even if a next activity is "triggered", the discard button will visible.
Task-2459645
closes odoo/odoo#66164
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
This commit prevents the opening/closure of a channel on mobile when
the action is performed on desktop and reciprocally
closes odoo/odoo#64599
Tasks-id: 2267593
Related: odoo/enterprise#16296
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
Current behavior before PR:
When looking for a user through the new message chat window, if there is already
a chat open for this user, the new message window is still open on selecting the
user.
Desired behavior after PR is merged:
when opening a chat from a new message, if the chat with this user is already
open then it will close the new message chat window on selecting the user.
LINKS:
PR https://github.com/odoo/odoo/pull/65851
Task-2449118
closes odoo/odoo#65851
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
Due to asynchronous rendering, the attachment template can be rendered
when its attachment is undefined, this was causing an issue with an
unguarded getter call that was attempting to access the attachment.
This commit fixes this issue.
opw-2457983
closes odoo/odoo#65944
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
PURPOSE
Displaying the subject when the name of the record is exactly the same is
redundant and crowds the interface needlessly
SPECIFICATION
Subject should only be displayed if it is different than the name of the
record
LINKS
closes odoo/odoo#61044
Taskid: 2363130
Pr: https://github.com/odoo/odoo/pull/61044
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
- Send the new Inbox counter when marking as read, to guarantee the JS has the
correct value (instead of having to guess it which was impossible when an
arbitrary domain was given to the method).
NOTE: Manual increment is kept on receiving a new Inbox message because that
one was reliable and it would have been more tricky to compute the new counter
for each target partner (mark as read is one user doing an action for himself,
whereas new message is from another user to potentially many different users).
Generic counter for threads is also not sent on every change for similar
reasons.
- Refresh Inbox automatically when some messages are marked as read and there
are more message on the server than currently loaded.
- Remove confusing (and impossible to maintain) counter on non-origin threads
(in particular channel followers). This behavior in JS was not consistent with
the server which always counted only origin thread (model/res_id).
task-2446302
closes odoo/odoo#65161
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Task-2447922
closes odoo/odoo#65071
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
task-2447939
closes odoo/odoo#65074
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
task-2426357
task-2426366
closes odoo/odoo#64009
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
* = 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) <aku@odoo.com>
|
|
task-2347640
closes odoo/odoo#63313
Related: odoo/enterprise#15320
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Task-2444878
closes odoo/odoo#64857
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
task-2388600
closes odoo/odoo#62446
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
task-2333535
task-2388676
closes odoo/odoo#64505
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
find is O(n) and findFromIdentifyingData is O(1)
task-2440693
closes odoo/odoo#64712
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Before this commit, messages in the chatter that contained non wrapping
content could indefinitely extend the width of the chatter.
opw-2425351
closes odoo/odoo#64542
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
task-2414811
closes odoo/odoo#63785
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
task-2397404
closes odoo/odoo#62878
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
`t-raw` is too slow due to parsing the HTML in JS, converting it to VDOM and
comparing to existing VDOM. Here we can make a custom comparison based on actual
string, and directly replace in DOM if needed.
Part of task-2399731
closes odoo/odoo#63505
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
The style of discuss new message (+) and chat window new message
should be consistent, as they both cover the exact same feature.
closes odoo/odoo#63496
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
This was the case before the refactoring.
task-2411114
closes odoo/odoo#62942
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Currently, scheduling the activity from the document chatter does not
appear on the kanban record on activity button.
and the 'onchange' action receives the ID of the activity that was
marked as done and thus the 'Missing record' error
So in this commit, on creation and done of activity reload the view to
update the record on kanban card.
The opportunity is taken to make a pass on all of the "reload" to add the
`keepChanges` flag to prevent from losing pending changes on reload, especially
useful in form view.
closes odoo/odoo#60397
Taskid: 2362964
Related: odoo/enterprise#14235
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Before this commit, an event listener was still active after the removal
of the attachment viewer, which caused the the onLoad function to be
called on an removed component if the component was unmounted before
the image was loaded. This commit fixes this issue by adding the
eventListener through the owl template.
Additionally, when a new image was loading, the old image was kept in
the dom until the end of the loading of the new image, this commit
also fixes this issue by adding a t-key on the image element.
part of task-2363522
closes odoo/odoo#61560
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Before this commit, quickly clicking on the unlink button of an
attachment could result in a traceback if the click happened right
after the deletion of attachment but before the rendering of the DOM.
part of task-2363522
closes odoo/odoo#63366
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
task-2330277
closes odoo/odoo#63235
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
When the topbar goes to multiple line due to too much text content to fit its
currently available space (depending for example on length of text after
translations), it will look bad before this commit because the "send message"
and "log note" have a visual effect when they are open that requires them to be
directly above the composer to look good.
Part of task-2413814
closes odoo/odoo#63320
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
closes odoo/odoo#63298
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
Before this commit, attempting to delete attachments multiple times
could result in errors, especially on slow networks.
This commit fixes this issue.
task-2363522
closes odoo/odoo#61017
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
Before this commit, removing attachments that are being uploaded
could lead to a traceback if the removed attachment was still
awaiting for the upload of another attachment.
part of task-2363522
closes odoo/odoo#61376
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
task-2391170
closes odoo/odoo#62981
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|
|
This avoid text flickering when a multiline channel name is hovered in the
discuss sidebar.
task-2274991
closes odoo/odoo#63084
Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
|
|
STEPS:
* Created partner with following name: Putin's spokesman
* Tag the partner in a message: @Putin's spokesman
BEFORE: tag is not converted into a link
AFTER: the link is created
WHY: _generateMentionsLinks uses ``owl.utils.escape`` which is different from ``_.escape``
https://github.com/odoo/odoo/blob/880555b38dee0c3e3b364c19bb6b0db983bf8aa0/addons/mail/static/src/models/composer/composer.js#L659
So following replace didn't work:
https://github.com/odoo/odoo/blob/880555b38dee0c3e3b364c19bb6b0db983bf8aa0/addons/mail/static/src/models/composer/composer.js#L667
---
opw-2408863
closes odoo/odoo#62893
Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
|