Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix composer remaining open (minimised) when creating PD from user card #121

Merged
merged 1 commit into from
Jun 2, 2020

Conversation

davwheat
Copy link
Member

@davwheat davwheat commented Jun 2, 2020

Fixes #120

This is a super simple change which seems to fix the issue.

Moving the composer hide until after we've navigated to the new discussion makes the composer hide correctly.

Possibly some super weird and barely reproducible upstream bug?

ezgif com-video-to-gif

@imorland
Copy link
Member

imorland commented Jun 2, 2020

So simple it's almost embarrasing, thank you for this @davwheat

@davwheat
Copy link
Member Author

davwheat commented Jun 2, 2020

@imorland it still doesn't make sense to me though... Surely this should break in both instances of creating a PD?

@imorland
Copy link
Member

imorland commented Jun 2, 2020

Checking this out now! 🕺

@imorland
Copy link
Member

imorland commented Jun 2, 2020

You're a 🌟 Thank you so much for this, I will merge now!

@imorland imorland merged commit 2b06952 into FriendsOfFlarum:master Jun 2, 2020
@davwheat davwheat deleted the davwheat/fix-120 branch June 2, 2020 15:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Creating a new PD from the hover user card does not close the composer
2 participants