Navigation Menu

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

edit-server-htmlize.el #76

Merged
merged 1 commit into from Mar 19, 2013
Merged

edit-server-htmlize.el #76

merged 1 commit into from Mar 19, 2013

Conversation

frobtech
Copy link
Contributor

This adds the edit-server-htmlize.el package, which offers hooks that are handy for GMail (and perhaps other things).

stsquad added a commit that referenced this pull request Mar 19, 2013
@stsquad stsquad merged commit 89737e5 into stsquad:master Mar 19, 2013
@stsquad
Copy link
Owner

stsquad commented Mar 19, 2013

Awesome, thanks.

@stsquad
Copy link
Owner

stsquad commented Mar 20, 2013

I've just been doing some testing with Gmail in it's plain text mode. It seems it's dealing with newlines using

with a default css of display: block for newlines. As a result the dehtmlize looks a bit funny.

Can you confirm it's doing this for you as well?

@frobtech
Copy link
Contributor Author

I've just been doing some testing with Gmail in it's plain text mode. It
seems it's dealing with newlines using

with a default css of
display: block for newlines. As a result the dehtmlize looks a bit funny.

Can you confirm it's doing this for you as well?

Confirmed. It looks like they've just changed it in the last day or two.
I'll see what I can do.

It may get to the point that the simple dehtmlization is insufficient and
we need to employ something that actually understands HTML. Sigh.

Thanks,
Roland

@stsquad
Copy link
Owner

stsquad commented Mar 20, 2013

@frobtech before you go too deep down the rabit hole I have pushed a hacky change which seems to work. However I fully expect it could be cleaner.

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.

None yet

3 participants