Make CSS simpler using macro

[stevenjoezhang]

PR Checklist

Please check if your PR fulfills the following requirements:

• The commit message follows our guidelines.
• Tests for the changes was maked (for bug fixes / features).
  • Muse | Mist have been tested.
  • Pisces | Gemini have been tested.
• Docs in NexT website have been added / updated (for new features).

PR Type

What kind of change does this PR introduce?

• Bugfix.
• Feature.
• Code style update (formatting, local variables).
• Refactoring (no functional changes, no api changes).
• Build related changes.
• CI related changes.
• Documentation content changes.
• Other... Please describe:

What is the current behavior?

Issue resolved: N/A

What is the new behavior?

• Screenshots with this changes: N/A
• Link to demo site with this changes: N/A

How to use?

In NexT _config.yml:

...

Does this PR introduce a breaking change?

• Yes.
• No.

[ivan-nginx]

Need to recheck here, because values in base without !imortant.

hexo-theme-next/source/css/_mixins/base.styl

Lines 71 to 77 in 9876b88

	hide() {
	display: none;
	}
	show() {
	display: block;
	}

[stevenjoezhang]

Yes, I have tested, this is an exception. All other ‘display: none’ style can be replaced with macro ‘none()’

[ivan-nginx]

So, need to fix it somehow.

[stevenjoezhang]

No better solution, I think.

Or we can remove show, hide and circle macros. Currently, they are only used a few times.

[ivan-nginx]

So, need any solution. For now you suggest Improvment (Optimization), but give 1 additional Bug with this.

Mb u can see why was !important there? And try to resolve this.

[stevenjoezhang]

display: none !important is used to overwrite the display: block style in .post-eof in scheme Gemini. To make sure it works on all browsers, an !important is needed. It won't cause additional bug.

[stevenjoezhang]

Remove show, hide and circle macros is another way. What's your opinion?

[ivan-nginx]

display: none !important is used to overwrite the display: block style in .post-eof in scheme Gemini. To make sure it works on all browsers, an !important is needed. It won't cause additional bug.

I know it, why u say me what I already know? Question is: why there are used !important? What rewrited? Why only for Gemini? How to make without rewriting (without !important)? Is it possible? If possible, need to try it, yeah?

[stevenjoezhang]

Oh, I understand. I'll open another PR to reduce the usage of !important.

[stevenjoezhang]

About !important: #744
So, what's next? Use hide() instead of display: none, or use display: none instead of hide()? The current situation is that the macro hide() is defined, but we barely use it.

[ivan-nginx]

Of course use macro hide(). It's global var.
But in #744 seems some troubles with non-relevant changes.

[ivan-nginx]

Milestone.