Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.The way the JS has been written implies a dependency on the bartik theme.
I have identified two problems, maybe there are more:
- You need the block template because it adds a "content" class that is used to refresh the inbox block.
<div{{ content_attributes.addClass('content') }}>
- The position of the blocks in the page matters. It is not possible to put the inbox block before the content because of a JS selector.
The code should not be based on the use of ":first".
threadWrapper = $(".private-message-thread").parent();
Issue fork private_message-3042276
Show commands
Start within a Git clone of the project using the version control instructions.
Or, if you do not have SSH keys set up on git.drupalcode.org:
changes
Comments
Comment #2
phjouComment #3
phjouSince you can use this module with only one theme (or if you spend time to debug like and find out what templates you need), I think this is pretty critical.
Comment #4
phjouComment #5
phjouJust added this issue related #3038271 Inbox not working properly
The second problem is basically this issue. There is a fix that is needed quickly, but I think it will still be theme dependent.
The selector "$(".private-message-thread-full")" will not be there in every theme. We have to use selectors with a naming like "js-class-name" in order to work with every theme and it will split properly the JS logic and the templating.
Comment #6
phjouChange name of the issue to be distinct from #3073058
Comment #9
snig commentedSimple wrapping block content and changed class selector in js should fix this issue.
Comment #12
artem_sylchukLooks good enough for me.