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

change the first level html to markdown #60

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mountain
Copy link

Since markdown is widely used, and github support markdown well but html badly, so let's change the html files in first level directory to markdown.

About htmls in doc directory, I currently just hold on, because these htmls could be integrated with the IDE in the future.

@mountain
Copy link
Author

in issue #39 , people are also complain about html, maybe we need consider change all html to markdown? there are pandoc tool can convert them automatically. But I am not sure about this. These htmls could be integrated as manual with the IDE in the future.

@digama0
Copy link
Owner

digama0 commented Feb 15, 2022

I wouldn't mind having all the html files in the doc/ folder converted to markdown. I certainly have no plans to incorporate an HTML viewer in mmj2, that sounds like something we should certainly outsource to an actual browser. If you want to add a help menu, it can just open the markdown docs in github by calling the default browser.

@digama0
Copy link
Owner

digama0 commented Feb 15, 2022

Not sure about the LicenseEnclosure.java move. Java files can't usually just be moved around, did you update references to it?

@mountain
Copy link
Author

After a search, I only found LicenseEnclosure.java was referenced in the CHGLOG.TXT which record its changes in past revisions. So CHGLOG.TXT should not be modified, otherwise I don't found any usage in the project.

@mountain
Copy link
Author

OK, I will consider change all html to markdown.

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.

2 participants