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 publisher broken after jquery upgrade #4845

Merged
merged 1 commit into from Mar 13, 2014

Conversation

Flaburgan
Copy link
Member

As indicated by @fabianrbz in #4813 (comment) this commit fixes the publisher after jquery update. I don't know if we want to close #4813 or upgrade the plugin though, but this should be merged at least as a temporary fix.

@jaywink
Copy link
Contributor

jaywink commented Mar 13, 2014

Thanks! If this fixes #4813 personally I would close it and open a feature request for finding a new plugin from one of the forks or fork it ourself. Or we can always rename #4813? Anyway, even if the issue is fixed by this, we should somehow track the upstream plugin requirement.

jaywink added a commit that referenced this pull request Mar 13, 2014
Fix publisher broken after jquery upgrade
@jaywink jaywink merged commit 8f67e1e into diaspora:develop Mar 13, 2014
@jhass jhass added this to the next milestone Mar 13, 2014
@Flaburgan Flaburgan deleted the fix-mention-publisher branch March 15, 2014 15:12
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.

publisher mentions ui glitch
3 participants