Skip to content

Conversation

Link2Twenty
Copy link
Contributor

What type of PR is this? (check all applicable)

  • Refactor
  • Feature
  • Bug Fix
  • Documentation Update

Description

The 3 hardest problems programmers face are

  1. Naming things
  2. Off by 1 errors

.sidebar-profile-snapshot width was wider than the rest of the sidebar by 1px.

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Before:
image

After:
image

Added to documentation?

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

Width was 1px too wide
Copy link
Contributor

@maestromac maestromac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@maestromac maestromac merged commit 354f7f8 into forem:master Mar 21, 2019
@pr-triage pr-triage bot added the PR: merged bot applied label for PR's that are merged label Mar 21, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: merged bot applied label for PR's that are merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants