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

Generated Markdown is not markdown spec compatible #55

Closed
niderhoff opened this issue Feb 19, 2020 · 5 comments
Closed

Generated Markdown is not markdown spec compatible #55

niderhoff opened this issue Feb 19, 2020 · 5 comments

Comments

@niderhoff
Copy link

niderhoff commented Feb 19, 2020

The generated markdown is not compatible with the markdownlint default settings (see here).

Maybe spacing after headings etc. should be configurable?

I know I can change my markdownlint settings, but it should really be the other way round I think and a lot of people might be using markdownlint anyway so we should give an option to make vscode journal compatible.

image

image

Solutions I could suggest:

A. make spacing after headings configurable
B. make a setting ("fix markdown automatically") that triggers the "fix all all violations in documents" (markdownlint.fixAll) of markdownlint if markdownlint is installed

@tisseurdetoile
Copy link

Hello

does your "journal.tpl-entry":

match the default one

"journal.tpl-entry":# ${weekday}, ${localDate}\n\n## Tasks\n\n## Notes\n\n

It seem your config lack the "\n\n" after ${localDate} and Tasks and note note.

You must edit the vscode settings.json directly.

@niderhoff
Copy link
Author

I changed it manually at some point and added the \n\n, so I cannot answer this question anymore. I will close this issue until somebody else has this problem.

@niderhoff
Copy link
Author

niderhoff commented Apr 28, 2020

Hello,

I just installed vscode journal on a different PC and the same thing happened. Note that I have NOT set ANY options whatsoever. This is default behaviour after install.
grafik

Note that there is no "journal.tpl-entry" in my settings.

{
    "workbench.iconTheme": "vscode-icons",
    "workbench.colorTheme": "One Dark Pro",
    "terminal.integrated.shell.windows": "cmd.exe",
    "window.zoomLevel": 0,
    "editor.fontSize": 13,
    "journal.base": "C:/Users/Nicolas/Documents/Journal",
    "editor.tokenColorCustomizations": {
        "textMateRules": [
            {
                "scope": "text.html.markdown.journal.task.open.bullet",
                "settings": {
                    "foreground": "#FFFF00"
                }
            },
            {
                "scope": "text.html.markdown.journal.task.open.marker",
                "settings": {
                    "foreground": "#FFFF00"
                }
            },
            {
                "scope": "text.html.markdown.journal.task.open.keyword",
                "settings": {
                    "fontStyle": "italic"
                }
            },
            {
                "scope": "text.html.markdown.journal.task.open.text",
                "settings": {}
            },
            {
                "scope": "text.html.markdown.journal.task.completed.keyword",
                "settings": {
                    "fontStyle": "italic"
                }
            },
            {
                "scope": "text.html.markdown.journal.task.completed.marker",
                "settings": {
                    "foreground": "#AAAAAA"
                }
            },
            {
                "scope": "text.html.markdown.journal.task.completed.text",
                "settings": {
                    "foreground": "#AAAAAA"
                }
            },
            {
                "scope": "text.html.markdown.journal.task.completed.bullet",
                "settings": {
                    "foreground": "#FFFF00"
                }
            },
            {
                "scope": "text.html.markdown.journal.memo.keyword",
                "settings": {
                    "fontStyle": "italic"
                }
            },
            {
                "scope": "text.html.markdown.journal.memo.bullet",
                "settings": {
                    "foreground": "#FFFF00"
                }
            },
            {
                "scope": "text.html.markdown.journal.scope",
                "settings": {
                    "foreground": "#FFFF00"
                }
            },
            {
                "scope": "text.html.markdown.journal.link.keyword",
                "settings": {
                    "fontStyle": "italic"
                }
            },
            {
                "scope": "text.html.markdown.journal.link.bullet",
                "settings": {
                    "foreground": "#FFFF00"
                }
            }
        ]
    }
}

Interestingly enough, if I add said settings-entry, I get the following suggestion default:

grafik

@niderhoff niderhoff reopened this Apr 28, 2020
@pajoma
Copy link
Owner

pajoma commented Apr 30, 2020

Taken your example, the issue seems to be where the memo is placed (directly after the header in this case), correct?

I am looking into it (hopefully in the weekend, quite busy atm)

pajoma added a commit that referenced this issue May 18, 2020
@pajoma
Copy link
Owner

pajoma commented May 26, 2020

I have added the current prerelease to this project, can you please download it from here and test it?

To test, use the option "Install from VSIX..." in the Extension View of Visual Studio Code.

@pajoma pajoma closed this as completed Dec 3, 2020
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

No branches or pull requests

3 participants