-
Notifications
You must be signed in to change notification settings - Fork 12
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
Plugin broken #3
Comments
Also consider this: reactjs/react-rails#825 |
Seems like this is an deprecation issue. Components like
I know this is not a pretty solution but I'm kinda new to react. Maybe someone with more experience have a look into this and use the mailspring react components instead of local ones? Cheers! |
Fixed on my fork : Striffly@a74fceb |
@Striffly Your fork works for me, although I get this minor layout issue where the unread icon overlaps over the name of the email sender. I'm on the latest Mailspring (Version 1.4.2). |
Fixed ! Striffly@876280b |
Hi, thanks a lot @Striffly for adapting this plugin! It doesn't seem to work with Mailspring 1.6.1 though, I don't have any avatars appear in my email list. Does it work for you? |
Hi @paulgaumer Same thing for me, I don't find solution for the moment 😕 |
Plugin fixed here : https://github.com/Striffly/mailspring-avatars |
The plugin is not working for me (OS: Ubuntu 16.04)
I tracked it down to the
React.Proptype
component which seems to beundefined
inavatar-image.jsx
. I am currently trying to fix it.Console Output:
The text was updated successfully, but these errors were encountered: