The theme of the /user/inbox page and the new private message page is broken. The message box is overlapping the user suggestion list where we select the username for sending the message.

  1. Enabled the Social Private Message module
  2. Create a new message
  3. Try to select the user whom who want to send the message and send the message
  4. Go to /user/inbox

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

navneet0693 created an issue. See original summary.

Kingdutch’s picture

Version: 8.x-9.x-dev » 8.x-8.x-dev
Priority: Normal » Major

This looks like a pretty serious issue even though it's sort of still working.

Corn696’s picture

Hmm on my install the autocomplete is not working for the first element. For the second one autocomplete works but the css is broken.

Edit: autocomplete is working. Just could not see it ...

robertragas’s picture

First issue has to do (create private message) with the social_private_message_thread_member_widget not being picked up, set on admin/structure/private_message/private_message_thread/form-display where it takes the default instead social.

Since 8 has the features removal, it must be something in the update path.

robertragas’s picture

Update:

the social core update 8802 which does the feature removal gets the config files from the original contrib private_message and reverts to those, ignoring our feature-removal folder, thus breaking it.

robertragas’s picture

robertragas’s picture

Status: Active » Needs review
Kingdutch’s picture

Status: Needs review » Needs work
michaelschutz’s picture

I ran this update on a test site (using the very latest PR commit from yesterday), and the update hook said it ran fine with drush (8805: update completed), but after a cache clear, the inbox page is still broken with the weird formatting. I'm willing to push this out to my live site using the PR code if needed, since it's a pretty major bug, but need to know if there are further steps required to fix the display issue.

Is there something else I need to do?

ronaldtebrake’s picture

Status: Needs work » Fixed

So we have changed the update hook and moved it in to social_private_message and ensure it runs after social core its update hook. Hope that it helps. Feel free to let us know if it didnt.

michaelschutz’s picture

Thanks - I've successfully updated both my test and live site. After running the update, nothing changed. I uninstalled social_private_message, deleted 4 config keys, then re-installed it. But after that, it seems to work fine. The formatting is fixed and didn't lose any messages.

  • ronaldtebrake committed 71f99e5 on 8.x-8.x
    Issue #3113150 by robertragas: since the feature removal failed for the...
  • ronaldtebrake committed 86fc1e0 on 8.x-8.x
    Issue #3113150 by ronaldtebrake: ensure correct dependency with...
  • ronaldtebrake committed 8dae6d8 on 8.x-8.x
    Issue #3113150 by robertragas: relocate the update hook to the...
  • ronaldtebrake committed d628a14 on 8.x-8.x
    Issue #3113150 by robertragas: check if social_private_message is...
  • ronaldtebrake committed ec09ef1 on 8.x-8.x
    Issue #3113150 by ronaldtebrake: ensure Yaml is used
    
Corn696’s picture

The theme is still broken but the other issue is fixed.

navneet0693’s picture

Status: Fixed » Active

The autocomplete is working on creating a new private message but /user/inbox theme is broken when coming from an existing environment.

navneet0693’s picture

Status: Active » Needs review
navneet0693’s picture

Version: 8.x-8.x-dev » 8.x-9.x-dev

  • 2ac0f51 committed on 8.x-9.x
    Issue #3113150 by robertragas: check if social_private_message is...
  • f0099ba committed on 8.x-9.x
    Issue #3113150 by robertragas: since the feature removal failed for the...
  • navneet0693 committed 4e28f00 on 8.x-9.x
    Issue #3113150 by navneet0693: Reverted remaining configuration by...
  • ronaldtebrake committed 6990f5e on 8.x-9.x
    Issue #3113150 by ronaldtebrake: ensure Yaml is used
    
  • navneet0693 authored 868cdb5 on 8.x-9.x
    Merge pull request #1745 from goalgorilla/issue/3113150-user-inbox-theme...
  • ronaldtebrake committed 86d012e on 8.x-9.x
    Issue #3113150 by ronaldtebrake: ensure correct dependency with...
  • ronaldtebrake authored b735679 on 8.x-9.x
    Merge pull request #1727 from goalgorilla/bugfix/issue-3113150-social-...
  • d513949 committed on 8.x-9.x
    Issue #3113150 by robertragas: relocate the update hook to the...
ronaldtebrake’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.