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

Update the centering layout of the previous section in the README file #5520

Closed
wants to merge 2 commits into from

Conversation

lmliheng
Copy link

Before

ada8bb0245c610cd4265baa34edc83c

After

image

将部分md写法改成HTML

Signed-off-by: liheng <[email protected]>
Copy link

How to test

git clone -b master https://github.com/lmliheng/hexo.git
cd hexo
npm install
npm test

@uiolee
Copy link
Member

uiolee commented Jul 19, 2024

Thank you for your efforts.

Is there any reason to use complex html instead of simple markdown?
Personally, I don't think i can accept it.

@lmliheng
Copy link
Author

谢谢你的努力。

有什么理由使用复杂的 html 而不是简单的 Markdown 吗?就个人而言,我认为我不能接受。

Okay, but it's definitely more beautiful, and it's a one-time deal

@yoshinorin
Copy link
Member

Thank you for submitting the PR.

However, we do not see a sufficient reason to accept this PR. Aesthetic judgments are subjective and generally do not constitute a solid basis for acceptance. In this case, it seems to fall within the personal preference. Additionally, no one can ensure that this will be a one-time change.

@yoshinorin yoshinorin closed this Jul 21, 2024
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.

3 participants