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

Justify articles and comments content #1234

Closed
wants to merge 1 commit into from
Closed

Justify articles and comments content #1234

wants to merge 1 commit into from

Conversation

quantumsheep
Copy link

@quantumsheep quantumsheep commented Nov 30, 2018

What type of PR is this?

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

I maybe am an ignorant but why isn't the articles and comments already in justify? It's like a standards in articles and newspapers.

So this PR adds a justification of the articles and comments text.

Mobile & Desktop Screenshots

From Firefox 63.0.3 on Windows

image
image

From Firefox on a 320x568 screen (iPhone 5 size)

screenshot_2018-12-01 the dev community - top this year
image

Added to documentation?

  • docs.dev.to
  • readme
  • no documentation needed

What gif best describes this PR or how it makes you feel?

so much room for activities

@pr-triage pr-triage bot added the PR: unreviewed bot applied label for PR's with no review label Nov 30, 2018
@CLAassistant
Copy link

CLAassistant commented Nov 30, 2018

CLA assistant check
All committers have signed the CLA.

@stephane-rbn
Copy link

I personally don't think that we should justify texts. Medium and freeCodeCamp don't justify their texts either. I think you may find the reasons in this article from Design For Hackers.

@benhalpern
Copy link
Contributor

@stephanyan yep you hit the nail on the head for generally why we don't justify text.

As per that post:

The technology on the web isn’t ready for justification. Sad but true.

Closing the PR, but thanks for the submission.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: unreviewed bot applied label for PR's with no review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants