[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #24 from avlas --- I still can reproduce the issue -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #23 from avlas --- (In reply to Laurent Montel from comment #22) > I confirm it. > I will investigate it Thank you -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 Laurent Montel changed: What|Removed |Added Status|RESOLVED|REOPENED Resolution|FIXED |--- Ever confirmed|0 |1 --- Comment #22 from Laurent Montel --- I confirm it. I will investigate it -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #21 from avlas --- (In reply to Laurent Montel from comment #20) > (In reply to avlas from comment #19) > > (In reply to Laurent Montel from comment #13) > > > Git commit 137430312149daa2a95109160d236ad7c70d14a6 by Laurent Montel. > > > Committed on 25/04/2019 at 07:11. > > > Pushed by mlaurent into branch 'master'. > > > > > > Fix Bug 406334 - Markdown addon: embedded images (attached to the email) > > > are > > > not displayed > > > FIXED-IN: 5.12.0 > > > > > > M +6-5 > > > kmail/editorconvertertextplugins/markdown/markdowninterface.cpp > > > > > > https://commits.kde.org/kdepim-addons/ > > > 137430312149daa2a95109160d236ad7c70d14a6 > > > > Not sure there has been a regression of some sort, but attached images are > > not displayed in markdown formatted html emails. > > Test case ? > For me it seems working here. Please see: https://i.imgur.com/moqaQSv.png The image is missing and if I send the email anyway, I cannot find the image in the resulting email -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #20 from Laurent Montel --- (In reply to avlas from comment #19) > (In reply to Laurent Montel from comment #13) > > Git commit 137430312149daa2a95109160d236ad7c70d14a6 by Laurent Montel. > > Committed on 25/04/2019 at 07:11. > > Pushed by mlaurent into branch 'master'. > > > > Fix Bug 406334 - Markdown addon: embedded images (attached to the email) are > > not displayed > > FIXED-IN: 5.12.0 > > > > M +6-5 > > kmail/editorconvertertextplugins/markdown/markdowninterface.cpp > > > > https://commits.kde.org/kdepim-addons/ > > 137430312149daa2a95109160d236ad7c70d14a6 > > Not sure there has been a regression of some sort, but attached images are > not displayed in markdown formatted html emails. Test case ? For me it seems working here. -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #19 from avlas --- (In reply to Laurent Montel from comment #13) > Git commit 137430312149daa2a95109160d236ad7c70d14a6 by Laurent Montel. > Committed on 25/04/2019 at 07:11. > Pushed by mlaurent into branch 'master'. > > Fix Bug 406334 - Markdown addon: embedded images (attached to the email) are > not displayed > FIXED-IN: 5.12.0 > > M +6-5 > kmail/editorconvertertextplugins/markdown/markdowninterface.cpp > > https://commits.kde.org/kdepim-addons/ > 137430312149daa2a95109160d236ad7c70d14a6 Not sure there has been a regression of some sort, but attached images are not displayed in markdown formatted html emails. -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #18 from avlas --- (In reply to Laurent Montel from comment #17) > now we can define the size of image (using =x) Nice! -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #17 from Laurent Montel --- now we can define the size of image (using =x) -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #16 from avlas --- (In reply to Laurent Montel from comment #15) > (In reply to avlas from comment #14) > > (In reply to Laurent Montel from comment #13) > > > Git commit 137430312149daa2a95109160d236ad7c70d14a6 by Laurent Montel. > > > Committed on 25/04/2019 at 07:11. > > > Pushed by mlaurent into branch 'master'. > > > > > > Fix Bug 406334 - Markdown addon: embedded images (attached to the email) > > > are > > > not displayed > > > FIXED-IN: 5.12.0 > > > > > > M +6-5 > > > kmail/editorconvertertextplugins/markdown/markdowninterface.cpp > > > > > > https://commits.kde.org/kdepim-addons/ > > > 137430312149daa2a95109160d236ad7c70d14a6 > > > > Great, thanks! > > > > Do you think there could be a way to apply styling? I think it would be the > > last missing piece > > It's still on my todo list (+ some others fix/improvement) > > See you if I will find a fix for it too :) Very much appreciated! -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #15 from Laurent Montel --- (In reply to avlas from comment #14) > (In reply to Laurent Montel from comment #13) > > Git commit 137430312149daa2a95109160d236ad7c70d14a6 by Laurent Montel. > > Committed on 25/04/2019 at 07:11. > > Pushed by mlaurent into branch 'master'. > > > > Fix Bug 406334 - Markdown addon: embedded images (attached to the email) are > > not displayed > > FIXED-IN: 5.12.0 > > > > M +6-5 > > kmail/editorconvertertextplugins/markdown/markdowninterface.cpp > > > > https://commits.kde.org/kdepim-addons/ > > 137430312149daa2a95109160d236ad7c70d14a6 > > Great, thanks! > > Do you think there could be a way to apply styling? I think it would be the > last missing piece It's still on my todo list (+ some others fix/improvement) See you if I will find a fix for it too :) -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #14 from avlas --- (In reply to Laurent Montel from comment #13) > Git commit 137430312149daa2a95109160d236ad7c70d14a6 by Laurent Montel. > Committed on 25/04/2019 at 07:11. > Pushed by mlaurent into branch 'master'. > > Fix Bug 406334 - Markdown addon: embedded images (attached to the email) are > not displayed > FIXED-IN: 5.12.0 > > M +6-5 > kmail/editorconvertertextplugins/markdown/markdowninterface.cpp > > https://commits.kde.org/kdepim-addons/ > 137430312149daa2a95109160d236ad7c70d14a6 Great, thanks! Do you think there could be a way to apply styling? I think it would be the last missing piece -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 Laurent Montel changed: What|Removed |Added Version Fixed In||5.12.0 Resolution|--- |FIXED Status|REPORTED|RESOLVED Latest Commit||https://commits.kde.org/kde ||pim-addons/137430312149daa2 ||a95109160d236ad7c70d14a6 --- Comment #13 from Laurent Montel --- Git commit 137430312149daa2a95109160d236ad7c70d14a6 by Laurent Montel. Committed on 25/04/2019 at 07:11. Pushed by mlaurent into branch 'master'. Fix Bug 406334 - Markdown addon: embedded images (attached to the email) are not displayed FIXED-IN: 5.12.0 M +6-5kmail/editorconvertertextplugins/markdown/markdowninterface.cpp https://commits.kde.org/kdepim-addons/137430312149daa2a95109160d236ad7c70d14a6 -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #12 from avlas --- (In reply to Laurent Montel from comment #11) > Thanks I will continue to implement it today or tomorrow Thank you very much! If you implement all these enhancements, it's going to be amazing! -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #11 from Laurent Montel --- Thanks I will continue to implement it today or tomorrow -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #10 from avlas --- (In reply to avlas from comment #7) > (In reply to Laurent Montel from comment #5) > > (In reply to avlas from comment #1) > > > I tested this in Discount and works as expected, embedding the image > > > within > > > other markdown content. Could you please consider adding this option? > > > > How did you test it ? > > Could you give me your command line ? > > thanks > > I do: > > mkd2html -G -css github.css mainTest.md mainTest_github.html Forgot to say about the image, which I didn't attach (as it may have copyright). Please change it by one that you have to test. -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #9 from avlas --- In addition to the github css style (that I attached) I have another css file from markdown-here. The one I use (and like the most) is the github one, with single exception, the background color that is used in the title row of tables, which is taken from markdown-here. It would be ideal to, for instance, use the github style by default but allow the final user to add a specific css file in settings, if desired. -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #8 from avlas --- Created attachment 119433 --> https://bugs.kde.org/attachment.cgi?id=119433=edit File of github style for markdown conversion -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #7 from avlas --- (In reply to Laurent Montel from comment #5) > (In reply to avlas from comment #1) > > I tested this in Discount and works as expected, embedding the image within > > other markdown content. Could you please consider adding this option? > > How did you test it ? > Could you give me your command line ? > thanks I do: mkd2html -G -css github.css mainTest.md mainTest_github.html -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #6 from avlas --- Created attachment 119432 --> https://bugs.kde.org/attachment.cgi?id=119432=edit File to test markdown format conversion -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #5 from Laurent Montel --- (In reply to avlas from comment #1) > I tested this in Discount and works as expected, embedding the image within > other markdown content. Could you please consider adding this option? How did you test it ? Could you give me your command line ? thanks -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #4 from Laurent Montel --- Git commit 767d21f55ef2447f67790c033762c51a4d6e1ff1 by Laurent Montel. Committed on 16/04/2019 at 11:27. Pushed by mlaurent into branch 'master'. Add method for generating image markdown string. For the moment it doesn't add image in mail M +2-0kmail/editorconvertertextplugins/markdown/CMakeLists.txt M +4-2 kmail/editorconvertertextplugins/markdown/autotests/CMakeLists.txt A +46 -0 kmail/editorconvertertextplugins/markdown/autotests/markdowncreateimagedialogtest.cpp [License: LGPL (v2+)] A +35 -0 kmail/editorconvertertextplugins/markdown/autotests/markdowncreateimagedialogtest.h [License: LGPL (v2+)] A +59 -0 kmail/editorconvertertextplugins/markdown/autotests/markdowncreateimagewidgettest.cpp [License: LGPL (v2+)] A +36 -0 kmail/editorconvertertextplugins/markdown/autotests/markdowncreateimagewidgettest.h [License: LGPL (v2+)] A +52 -0 kmail/editorconvertertextplugins/markdown/markdowncreateimagedialog.cpp [License: LGPL (v2+)] A +36 -0 kmail/editorconvertertextplugins/markdown/markdowncreateimagedialog.h [License: LGPL (v2+)] A +52 -0 kmail/editorconvertertextplugins/markdown/markdowncreateimagewidget.cpp [License: LGPL (v2+)] A +38 -0 kmail/editorconvertertextplugins/markdown/markdowncreateimagewidget.h [License: LGPL (v2+)] M +14 -0kmail/editorconvertertextplugins/markdown/markdowninterface.cpp M +1-0kmail/editorconvertertextplugins/markdown/markdowninterface.h https://commits.kde.org/kdepim-addons/767d21f55ef2447f67790c033762c51a4d6e1ff1 -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #3 from avlas --- (In reply to Laurent Montel from comment #2) > option for what ? > Generating a image.png ? I meant for adding images within context. Imaging you write a markdown email and you refer to one or more images in the body of the email. Discount allows to generate an html version of it that embeds the referenced image in its right place. It would be great if Kmail would benefit from this, so referenced images are embedded in the right places of the email body. For this, Kmail would need to be smart enough to know where to find the image (that needs of course to be attached to the email). I hope I was clearer this time :) -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 Laurent Montel changed: What|Removed |Added CC||mon...@kde.org --- Comment #2 from Laurent Montel --- option for what ? Generating a image.png ? -- You are receiving this mail because: You are watching all bug changes.
[kdepim] [Bug 406334] Markdown addon: embedded images (attached to the email) are not displayed
https://bugs.kde.org/show_bug.cgi?id=406334 --- Comment #1 from avlas --- I tested this in Discount and works as expected, embedding the image within other markdown content. Could you please consider adding this option? -- You are receiving this mail because: You are watching all bug changes.
