{"action":"created","issue":{"url":"https://api.github.com/repos/plone/plone.app.mosaic/issues/413","repository_url":"https://api.github.com/repos/plone/plone.app.mosaic","labels_url":"https://api.github.com/repos/plone/plone.app.mosaic/issues/413/labels{/name}","comments_url":"https://api.github.com/repos/plone/plone.app.mosaic/issues/413/comments","events_url":"https://api.github.com/repos/plone/plone.app.mosaic/issues/413/events","html_url":"https://github.com/plone/plone.app.mosaic/issues/413","id":336753393,"node_id":"MDU6SXNzdWUzMzY3NTMzOTM=","number":413,"title":"empy <p> tag gets appended to existing RichtTextTile when inserting new RichTextTile","user":{"login":"petschki","id":511761,"node_id":"MDQ6VXNlcjUxMTc2MQ==","avatar_url":"https://avatars1.githubusercontent.com/u/511761?v=4","gravatar_id":"","url":"https://api.github.com/users/petschki","html_url":"https://github.com/petschki","followers_url":"https://api.github.com/users/petschki/followers","following_url":"https://api.github.com/users/petschki/following{/other_user}","gists_url":"https://api.github.com/users/petschki/gists{/gist_id}","starred_url":"https://api.github.com/users/petschki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/petschki/subscriptions","organizations_url":"https://api.github.com/users/petschki/orgs","repos_url":"https://api.github.com/users/petschki/repos","events_url":"https://api.github.com/users/petschki/events{/privacy}","received_events_url":"https://api.github.com/users/petschki/received_events","type":"User","site_admin":false},"labels":[{"id":103853548,"node_id":"MDU6TGFiZWwxMDM4NTM1NDg=","url":"https://api.github.com/repos/plone/plone.app.mosaic/labels/01%20type:%20bug","name":"01 type: bug","color":"fa0000","default":false},{"id":213995410,"node_id":"MDU6TGFiZWwyMTM5OTU0MTA=","url":"https://api.github.com/repos/plone/plone.app.mosaic/labels/31%20needs:%20help","name":"31 needs: help","color":"00fafa","default":false},{"id":213995428,"node_id":"MDU6TGFiZWwyMTM5OTU0Mjg=","url":"https://api.github.com/repos/plone/plone.app.mosaic/labels/99%20tag:%20Plone%205.x","name":"99 tag: Plone 5.x","color":"f3f3f3","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":8,"created_at":"2018-06-28T19:39:32Z","updated_at":"2018-07-02T11:58:22Z","closed_at":null,"author_association":"MEMBER","body":"I have a strange behavior in the mosaic layout editor. \r\nSteps to reproduce:\r\n\r\n1. make new \"mosaic layout\" page\r\n2. edit and add RichTextTile and save\r\n3. edit again and add a new RichTextTile again\r\n4. the first RichTextTile markup gets an empty `<p>` tag appended to every existing `<p>`\r\n\r\n tags\r\n\r\nsee screenshots. can anybody else confirm this behavior?\r\n\r\nVersions:\r\nPlone=5.1-latest\r\nplone.app.mosaic=2.1.1\r\nplone.app.standardtiles=2.3.1\r\n\r\n<img width=\"1373\" alt=\"bildschirmfoto 2018-06-28 um 21 26 24\" src=\"https://user-images.githubusercontent.com/511761/42056732-76ec9f40-7b1b-11e8-9280-3cc7056f457e.png\">\r\n<img width=\"1288\" alt=\"bildschirmfoto 2018-06-28 um 21 26 46\" src=\"https://user-images.githubusercontent.com/511761/42056733-770b8b76-7b1b-11e8-963d-8c6abd4c8caa.png\">\r\n<img width=\"1246\" alt=\"bildschirmfoto 2018-06-28 um 21 27 01\" src=\"https://user-images.githubusercontent.com/511761/42056735-772dfd1e-7b1b-11e8-8052-aa2a2c653043.png\">\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/plone/plone.app.mosaic/issues/comments/401780831","html_url":"https://github.com/plone/plone.app.mosaic/issues/413#issuecomment-401780831","issue_url":"https://api.github.com/repos/plone/plone.app.mosaic/issues/413","id":401780831,"node_id":"MDEyOklzc3VlQ29tbWVudDQwMTc4MDgzMQ==","user":{"login":"datakurre","id":160447,"node_id":"MDQ6VXNlcjE2MDQ0Nw==","avatar_url":"https://avatars2.githubusercontent.com/u/160447?v=4","gravatar_id":"","url":"https://api.github.com/users/datakurre","html_url":"https://github.com/datakurre","followers_url":"https://api.github.com/users/datakurre/followers","following_url":"https://api.github.com/users/datakurre/following{/other_user}","gists_url":"https://api.github.com/users/datakurre/gists{/gist_id}","starred_url":"https://api.github.com/users/datakurre/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/datakurre/subscriptions","organizations_url":"https://api.github.com/users/datakurre/orgs","repos_url":"https://api.github.com/users/datakurre/repos","events_url":"https://api.github.com/users/datakurre/events{/privacy}","received_events_url":"https://api.github.com/users/datakurre/received_events","type":"User","site_admin":false},"created_at":"2018-07-02T11:58:22Z","updated_at":"2018-07-02T11:58:22Z","author_association":"MEMBER","body":"@petschki Your standardtiles fix gave me a hint what could cause this:\r\n\r\nhttps://github.com/plone/plone.app.blocks/blob/master/plone/app/blocks/layoutbehavior.py#L309\r\n\r\nCouple of years ago we changed default tile storage to be a single HTML-serialized field. That mimics the original Deco implementation where tile configuration and content was within the layout. Similarly to that, HTML of rich text tile is saved as HTML in that field. Attribute based storage has some benefits over annotation based storage in versioning, multilingual, restapi, etc...\r\n\r\nThe issue is probably that I've been naive enough to enable pretty print for that field. It sounds like it might be that pretty print that is causing extra newlines.\r\n\r\nIf you can confirm that, please, provide fix into plone.app.blocks. I have no idea, why I have not seen this issue myself. Possibly because of using older TinyMCE-version...\r\n\r\n"}}
{"actor":{"display_login":"datakurre"}}