summaryrefslogtreecommitdiff
path: root/addons/mail/static/src/components/message
AgeCommit message (Collapse)Author
2021-03-08[FIX] mail, test_mail: ensure thread name is up to date on messageDidier (did)
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>
2021-03-03[FIX] mail: do not show falsy value in tracking values messagesXavier Dubuc
task-2438828 closes odoo/odoo#65183 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2021-02-11[FIX] mail: improve subject of message in inboxPriyanka Kakadiya
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>
2021-01-27[FIX] mail: show bullet style of li in message contentAlexandre Kühn
Task-2447922 closes odoo/odoo#65071 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2021-01-21[FIX] mail, *: significantly reduce number of useless rendersSébastien Theys
* = 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>
2021-01-21[FIX] mail: reduce spacing between messagesAlexandre Kühn
Task-2444878 closes odoo/odoo#64857 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2021-01-18[FIX] mail, im_livechat, snailmail: use findFromIdentifyingDataSébastien Theys
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>
2020-12-17[FIX] mail: speed up render of Message by inserting content manuallySébastien Theys
`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>
2020-12-10[FIX] mail: mark all needaction as read when opening the threadDidier (did)
This was the case before the refactoring. task-2411114 closes odoo/odoo#62942 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-12-04[FIX] mail: split attachments list to improve styleXavier Dubuc
There is now two lists of attachments : first one with images and second one with non-images. Attachment viewer allows to navigate through previews of all viewable attachments in the order they're shown. task-2272587 task-2401325 closes odoo/odoo#62866 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-12-02[FIX] mail: set href on notification link.Didier (did)
This allow the user to open link in new tabs with middle click task-2387837 closes odoo/odoo#62304 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-11-23[FIX] mail: mark document as read when openDidier (did)
task-2387820 closes odoo/odoo#62166 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-11-20[FIX] mail: avoid empty seen indicatorSébastien Theys
`MessageSeenIndicator` sub-component should not be rendered if the thread does not support seen indicators. The issue is manifesting because the otherwise empty indicator has a margin, which is adding more space than expected before the following element. closes odoo/odoo#62030 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-11-17[FIX] mail,sms: avoid event propagation on avatar click in message listDidier (did)
Before this commit, when a user click on the author avatar inside the message list, it trigger the o-clicked for the message. task-2341603 closes odoo/odoo#61400 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com> Co-authored-by: Sébastien Theys <seb@odoo.com>
2020-11-20[FIX] mail: fix image width inside the chatterDidier (did)
Web editor hard code the width inside the image html. This `max-width` constraint the image inside the chatter and avoid unwanted horizontal scrollbar. task-2389432 closes odoo/odoo#62073 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-11-16[FIX] mail: fix race conditions, especially with scrollSébastien Theys
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) <aku@odoo.com>
2020-10-30[FIX] mail: remove chat window auto-focus on clickSébastien Theys
This causes more trouble than it's worth, especially with mobile keyboards. It also prevents from selecting any text in chat window threads. task-2372553 Part of task-2277543 closes odoo/odoo#61031 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-10-10[FIX] mail: plain-text messages word-wrap in Discuss UIPierre Paridans
When a user sends a plaintext email to a mail-enabled channel, the rendering of this message in Discuss have present a horizontal scroll when the text contains long sentences. This commit fixes it by enabling wrapping of the line's word to fit its container size and prevent horizontal scrolling. Note: the required CSS rules were already present since odoo/odoo@0e7e18ff71f3b490522358fa689ada7635dcd26a but a wrapper element (with the class `.o_thread_message_content`) was introduced in odoo/odoo@4685e4bc7f19f964697bf0dbd7dfdf254b6c5628 and merged between the testing of the fix above and its actual merging in the Odoo codebase, making it ineffective. We only have to properly re-enable it. opw-2341485 Related task ID: 34931 closes odoo/odoo#59689 X-original-commit: a8b64a74ef5575b783d1b314616f0a89bdce3732 Signed-off-by: Adrien Dieudonné (adr) <adr@odoo.com>
2020-10-09[FIX] mail: ask for confirmation before attachment deletionXavier Dubuc
also hide all messages with empty body and no attachments task-2282322 closes odoo/odoo#59672 X-original-commit: 60ae42dcda777cd66e4e421dbc02f9ef12a430ef Related: odoo/enterprise#13971 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-10-07[FIX] mail: clean up relations between chatter and threadSébastien Theys
This will prevent various flicker when using previous/next. task-2243518 closes odoo/odoo#59375 X-original-commit: 1559e82e6b781601286003c2aff364dbb19c9bfc Related: odoo/enterprise#13868 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-29[IMP] mail, website_livechat: bring back visitor banner in discussXavier Dubuc
task-2276589 closes odoo/odoo#58857 X-original-commit: 65dbe1c4760e2ffe104426f9aa41007a10dc32f0 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-09-29[FIX] mail: fix tour to allow public or private channel selectionSébastien Theys
task-2313321 closes odoo/odoo#58778 X-original-commit: f6a561a7e21772c6a25785c8b2428954d4bd152e Signed-off-by: Sébastien Theys (seb) <seb@odoo.com> Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-24[FIX] mail: fix tracking value issue when value is 0Priyanka Kakadiya
Before this commit: When a tracking field is set to 0, the mail tracking value doesn't display 0 in the chatter. It is empty instead i.e: Field Name: 1 -> After this commit: Tracking value will show the changed value even if it is false or 0 i.e: Field name: 1 -> 0 closes odoo/odoo#58419 Task: 2334048 Closes: https://github.com/odoo/odoo/pull/57342 X-original-commit: 57f4018694a895fd804b51e8206503e3d1d3c5d4 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-23[FIX] mail: use compute fields for identifying relationsXavier Dubuc
(in message seen indicators) task-2335647 closes odoo/odoo#58373 X-original-commit: 3d4ff42f4474a5761c9e0737297d973992a2fcc2 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-09-18[FIX] mail: fix notification partners display in popoverXavier Dubuc
Handle notification creation in convertData correctly. task-2324260 closes odoo/odoo#57970 X-original-commit: 7305bcf4edcbaa31b86bdc7b4ece460f30515c27 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-09-17[FIX] mail: fix "New messages" separator and channel_seen behaviourXavier Dubuc
- channel_seen is now called even for own message and only when needed - channel_seen is now called with a message_id which transfer the decision of what is seen to the client which makes more sense task-2282235 task-2282248 closes odoo/odoo#57950 X-original-commit: 04edfa50ccea305050d017c4a1996e561bc7bd57 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-09-11[FIX] mail: avoid star flicker when hovering a messageDidier (did)
task-2278441 closes odoo/odoo#57570 X-original-commit: 35e1a675803e64eda0fdae402cbddb56d5c0b1d8 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-04[FIX] mail, sms, snailmail: prevent message_fetch for hidden threadsXavier Dubuc
Automatically load messages from a thread cache only if the thread cache is used in at least one thread view. Ensure thread viewer don't keep thread view records alive when they are not displayed. This change is particularly important for performance, to avoid spamming RPC at init if many threads are opened in chat windows. task-2310623 closes odoo/odoo#57061 X-original-commit: dd6981c7812134e0588f80a157eaba45512cd68e Related: odoo/enterprise#12936 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-03[FIX] mail: add open chat when clicking on author im status iconSébastien Theys
task-2329538 closes odoo/odoo#57006 X-original-commit: bfb8d99baaadedb2d0492b6f378b13b3330d496a Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-09-02[FIX] mail: display subject of mailing channels in italicSébastien Theys
task-2321974 closes odoo/odoo#56924 X-original-commit: 7b06b094bd4ee4008fa29dd8d3db79275d4cde1f Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-09-01[FIX] mail: message/activity link with dataset oe-id and oe-modelAlexandre Kühn
Task-2328718 closes odoo/odoo#56864 X-original-commit: 430aa0c2ffe242992fb935733d81f6e05a6d2458 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com> Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-08-28[FIX] mail, *: fix opens and redirects, adapt messaging menu closeSébastien Theys
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) <aku@odoo.com> Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-08-28[FIX] mail, *: rename thread viewer to thread viewSébastien Theys
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) <aku@odoo.com> Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-08-27[FIX] mail: prevent "record created" from being twice in the chatterSébastien Theys
task-2280184 closes odoo/odoo#56691 X-original-commit: 1af2cc1b2190623739085de83870fb415cbdccf1 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-08-25[FIX] mail: reduce vertical spacing for tracking messagesDidier (did)
task-2282385 closes odoo/odoo#56487 X-original-commit: c80ed69bb5b06d3df18a6491c1b889fcdb0de741 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-08-25[FIX] mail: fix tracking value alignment in notificationXavier Dubuc
task-2321970 closes odoo/odoo#56465 X-original-commit: 016e36959337bf81ccf1d00733250af563bd605f Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-08-13[IMP] web, *: improve mock server and replace most mail mockRPCSébastien Theys
Improve mock server: - add support for mocked `fetch` - add support for `active_test` - add support for x2m `in` in domains - add support for default values computed from a function - implement a more natural "next id" compute - allow initial data without ids - ensure write and x2m commands integrity - improve bad data/bad commands error messages - always warn for failing RPC, not only in debug mode - fix all existing tests that had inconsistency data Other changes done in mail (or dependents) that are not just related to tests: - remove `direct_partner` from formatter result ->`correspondent` can be computed from other keys, especially `members` - fix `livechat_visitor` convertData -> only process if there is value - add `current_partner` and `current_user_id` as `init_messaging` result -> easier to mock than session - remove usage of `need_moderation` -> that was just a search indirection to `moderation_status` - adapt `partner_id` -> `res_partner_id` key in `_notification_format` -> to be consistent with field name - add name in result of `mail_partner_format` -> sometimes display_name is not the same - remove usage of `is_moderator` -> that was just an indirection to `moderation_channel_ids` Enterprise counterpart: https://github.com/odoo/enterprise/pull/11523 task-2287171 closes odoo/odoo#55854 X-original-commit: 7ba3fecb3377a720d1eb70e7515a0c45da73836d Related: odoo/enterprise#12391 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-08-06[FIX] mail: hide rating button in chatterDidier (did)
Before this commit, the rating buttons were displayed inside the chatter an could be used to "rate" from there. Intend is to display these buttons only on email: the message in chatter should not have these buttons. This commit fixes the problem similarly to what has been done in 13.3. Note that this solution is quite poor, because it could remove content from other messages that inadvertently hit this CSS selector, but at least it works for all existing rating messages. task-2282414 closes odoo/odoo#55591 X-original-commit: 4474adef45be3289728edc289ecc71c38abcf1b8 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-08-03[FIX] mail: speed up insert by using localId instead of findSébastien Theys
Part of task-2308312 X-original-commit: 407dc9796543a27b6695c8d4da922c13dece5b38
2020-07-29[FIX] mail: prevent attachment delete from not authored messageXavier Dubuc
task-2299330 closes odoo/odoo#55164 X-original-commit: c331b6761fbc1e8dd4d554109acab142ab02702d Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-07-29[FIX] mail: fix read more/less span insertionXavier Dubuc
task-2248307 closes odoo/odoo#55163 X-original-commit: a8ea9bc98d114eef056babc0d2639eaddbcf3778 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-07-22[IMP] mail: hide message error notifications if not origin threadXavier Dubuc
task-2282370 closes odoo/odoo#54768 X-original-commit: 4f693105bcaac7546590302cdc25c2b11b0f42cd Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-07-16[FIX] mail: show subject of the message in inbox if setXavier Dubuc
task-2280146 closes odoo/odoo#54599 X-original-commit: 3114c7dab3c7f0f63e59f01fd06237fdc109203a Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-07-08[FIX] mail: bring back extended composer for mailing channelXavier Dubuc
task-2171877 task-2282377 task-2282382 closes odoo/odoo#54249 X-original-commit: acf8eb3e2e49c1ade638ff1f2fed4969f9944053 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com>
2020-06-26[FIX] mail: fix (inbox reply) composer event handlers (and discard)Sébastien Theys
task-2092950 closes odoo/odoo#53731 X-original-commit: c29f5776594c4d2278ce163a78d8eb917fb672c0 Signed-off-by: Sébastien Theys (seb) <seb@odoo.com> Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-06-22[FIX] mail: consistent relations between threads, caches and messagesAlexandre Kühn
task-2171881 closes odoo/odoo#53440 X-original-commit: 80d13bb828ee75a50d8cea29046a8eb49c71948a Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com>
2020-06-15[REF] mail, *: refactor messaging with OWLSébastien Theys
This commit is a significant rewriting of client-side discuss, chatter, chat window, and messaging menu using OWL. The behavior should be broadly the same, with some slight functional changes here and there. From a technical standpoint, the code of messaging is mainly organized in 2 main groups of modules: - models, which are logical entities that depict the client-side state of messaging as a whole. - components, which are in charge of displaying information from models. This refactoring also introduces new JS guidelines regarding folder structure (/static) and naming rules for JS modules. Community PR: https://github.com/odoo/odoo/pull/39023 Enterprise PR: https://github.com/odoo/enterprise/pull/6249 Task-1914207 This PR is a collaborative work by Alexandre, Julien, Sébastien and Xavier, with the precious help of Lucas to speed it up towards the end. closes odoo/odoo#39023 Related: odoo/enterprise#6249 Signed-off-by: Alexandre Kühn (aku) <aku@odoo.com> Co-authored-by: Alexandre Kühn <aku@odoo.com> Co-authored-by: Julien Giannone <jgi@odoo.com> Co-authored-by: Lucas Perais <lpe@odoo.com> Co-authored-by: Sébastien Theys <seb@odoo.com> Co-authored-by: Xavier Dubuc <xdu@odoo.com>