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

[Micropub] Editing a post in Known can have destructive side effects on content #11

Open
cleverdevil opened this issue Feb 12, 2018 · 2 comments

Comments

@cleverdevil
Copy link

There is a bug in Known which causes HTML posts published via Micropub to be changed (usually in bad ways) when "editing" the post, even when you don't actually make any changes to the post. I discovered this issue when publishing via Sunlit 2.0, which supports Micropub.

I published two stories:

Because Sunlit doesn't yet support syndication via Micropub, I clicked "edit" on one of the posts, and toggled on syndication to Twitter and Facebook, and then clicked "save." The result was that the post's content was changed (in a destructive way, resulting in visual regressions), even though I hadn't actually edited the content, or even clicked into the content editor.

Seems like this is a bug.

(Originally published at: https://cleverdevil.io/2018/editing-a-post-in-known-can-have-destructive-side-effects)

@mapkyca mapkyca changed the title Editing a post in Known can have destructive side effects on content [Micropub] Editing a post in Known can have destructive side effects on content Feb 13, 2018
@cleverdevil
Copy link
Author

Did some investigation, and it appears that the editor is basically stripping out (some? all?) inline styles.

@mapkyca
Copy link
Member

mapkyca commented Apr 20, 2018

Is this still a thing? I seem to remember doing something in this domain a little while ago...

@mapkyca mapkyca transferred this issue from idno/known Aug 11, 2019
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 a pull request may close this issue.

2 participants