-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add text_align
option.
#682
Conversation
We need a expert CSS worker 😉 |
Nothing know about css, but 👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems good, but how about add global align option, not only for mobile devices?
mobile_text_justify
option.
@ivan-nginx How do you like options like this:
|
Yep. Seems good. |
Updated. I'll add the docs |
mobile_text_justify
option.text_align
option.
Maybe not Default value is |
@ivan-nginx No need to concern about it, the default value doesn't change in this PR: |
I know what there is no breaking change. I just suggest to think about justify by default and as for me, I always prefer turn off justify as on Desktop and for Mobiles too. I'm talking about |
I'll add a notice in the docs, about the difference between those options. |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue resolved: #73, Roadmap
What is the new behavior?
How to use?
In NexT
_config.yml
:Does this PR introduce a breaking change?